CHECK: Unnecessary parentheses around t->if_lock #46: FILE: net/core/pktgen.c:234: +#define if_lock(t) mutex_lock(&(t->if_lock)) CHECK: Macro argument 't' may be better as '(t)' to avoid precedence issues #46: FILE: net/core/pktgen.c:234: +#define if_lock(t) mutex_lock(&(t->if_lock)) CHECK: Unnecessary parentheses around t->if_lock #47: FILE: net/core/pktgen.c:235: +#define if_unlock(t) mutex_unlock(&(t->if_lock)) CHECK: Macro argument 't' may be better as '(t)' to avoid precedence issues #47: FILE: net/core/pktgen.c:235: +#define if_unlock(t) mutex_unlock(&(t->if_lock)) total: 0 errors, 0 warnings, 4 checks, 18 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 0289fb3886bb ("net: pktgen: fix code style (WARNING: macros should not use a trailing semicolon)") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE 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, 4 checks, 18 lines checked