CHECK: Macro argument 'len' may be better as '(len)' to avoid precedence issues #82: FILE: tools/testing/selftests/bpf/progs/test_tc_tunnel.c:26: +#define BPF_F_ADJ_ROOM_ENCAP_L2(len) \ + (((__u64)len & BPF_ADJ_ROOM_ENCAP_L2_MASK) \ + << BPF_ADJ_ROOM_ENCAP_L2_SHIFT) CHECK: spaces preferred around that '<<' (ctx:VxV) #97: FILE: tools/testing/selftests/bpf/progs/test_tc_tunnel.c:39: +#define VXLAN_FLAGS bpf_htonl(1<<27) ^ total: 0 errors, 0 warnings, 2 checks, 127 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit 887f0f183e90 ("selftests/bpf: make test_tc_tunnel.bpf.c compatible with big endian platforms") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 0 warnings, 2 checks, 127 lines checked