CHECK: Macro argument 'iter' may be better as '(iter)' to avoid precedence issues #55: FILE: drivers/net/ethernet/intel/idpf/idpf.h:486: +#define idpf_for_each_vport(adapter, iter) \ + for (struct idpf_vport **__##iter = &(adapter)->vports[0], \ + *iter = (adapter)->max_vports ? *__##iter : NULL; \ + iter; \ + iter = (++__##iter) < &(adapter)->vports[(adapter)->max_vports] ? \ + *__##iter : NULL) CHECK: spinlock_t definition without comment #214: FILE: drivers/net/ethernet/intel/idpf/idpf_ptp.h:149: + spinlock_t latches_lock; CHECK: spinlock_t definition without comment #215: FILE: drivers/net/ethernet/intel/idpf/idpf_ptp.h:150: + spinlock_t status_lock; total: 0 errors, 0 warnings, 3 checks, 428 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 bca81aef6b6c ("idpf: add Tx timestamp capabilities negotiation") 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, 3 checks, 428 lines checked