From: Barry Song <v-songbaohua@xxxxxxxx> -v7: * collect ack of Joe for checkpatch.pl, thanks! * fixed an indentation per Joe -v6: * collect ack of Joe, thanks! * refine docs according to Jonathan, thanks! * add checkpatch doc according to Joe, thanks! v6 link: https://lore.kernel.org/lkml/20240506014606.8638-1-21cnbao@xxxxxxxxx/ -v5: * Simplify the code for Patch 2 according to Joe's suggestions. * add s-o-b of Barry according to Jeff Johnson v5 link: https://lore.kernel.org/all/20240401012120.6052-1-21cnbao@xxxxxxxxx/ -v4: * fix Xining's email address, s/ma.xxn@xxxxxxxxxxx/mac.xxn@xxxxxxxxxxx/g * fix some false positives of checkpatch.pl * downgrade from ERROR to WARN in checkpatch.pl Thanks for Joe's comments! v4 link: https://lore.kernel.org/all/20240328022136.5789-1-21cnbao@xxxxxxxxx/ -v3: https://lore.kernel.org/all/20240322084937.66018-1-21cnbao@xxxxxxxxx/ A function-like macro could result in build warnings such as "unused variable." This patchset updates the guidance to recommend always using a static inline function instead and also provides checkpatch support for this new rule. Barry Song (1): Documentation: coding-style: ask function-like macros to evaluate parameters Xining Xu (1): scripts: checkpatch: check unused parameters for function-like macro Documentation/dev-tools/checkpatch.rst | 14 ++++++++++++++ Documentation/process/coding-style.rst | 23 +++++++++++++++++++++++ scripts/checkpatch.pl | 6 ++++++ 3 files changed, 43 insertions(+) -- 2.34.1
