CHECK: Alignment should match open parenthesis #170: FILE: drivers/net/ethernet/intel/ice/ice_txrx.h:140: + struct_group(tx, + u64 tx_restart_q; CHECK: Alignment should match open parenthesis #175: FILE: drivers/net/ethernet/intel/ice/ice_txrx.h:145: + struct_group(rx, + u64 rx_non_eop_descs; CHECK: Macro argument 'member' may be better as '(member)' to avoid precedence issues #193: FILE: drivers/net/ethernet/intel/ice/ice_txrx.h:163: +#define ice_stats_read(stats, member) ({ \ + struct ice_ring_stats *__stats = (stats); \ + __stats->member; \ +}) CHECK: Macro argument 'member' may be better as '(member)' to avoid precedence issues #206: FILE: drivers/net/ethernet/intel/ice/ice_txrx.h:176: +#define ice_stats_inc(stats, member) do { \ + struct ice_ring_stats *__stats = (stats); \ + __stats->member++; \ +} while (0) total: 0 errors, 0 warnings, 4 checks, 186 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 1f29794dca80 ("ice: shorten ring stat names and add accessors") 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, 4 checks, 186 lines checked