CHECK: Macro argument 'dev' may be better as '(dev)' to avoid precedence issues #168: FILE: include/linux/netdevice.h:3451: +#define netif_rx_mode_for_each_uc_addr(dev, ha_addr, idx) \ + for (ha_addr = dev->rx_mode_ctx->ready->uc_addrs, idx = 0; \ + idx < dev->rx_mode_ctx->ready->uc_count; \ + ha_addr += dev->addr_len, idx++) CHECK: Macro argument 'dev' may be better as '(dev)' to avoid precedence issues #173: FILE: include/linux/netdevice.h:3456: +#define netif_rx_mode_for_each_mc_addr(dev, ha_addr, idx) \ + for (ha_addr = dev->rx_mode_ctx->ready->mc_addrs, idx = 0; \ + idx < dev->rx_mode_ctx->ready->mc_count; \ + ha_addr += dev->addr_len, idx++) total: 0 errors, 0 warnings, 2 checks, 391 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 ed4ea99f3f4a ("net: refactor set_rx_mode into snapshot and deferred I/O") 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, 391 lines checked