Hi Martino, kernel test robot noticed the following build warnings: [auto build test WARNING on hid/for-next] [also build test WARNING on linus/master v6.5 next-20230906] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Martino-Fontana/HID-nintendo-cleanup-LED-code/20230906-221818 base: https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next patch link: https://lore.kernel.org/r/20230906141533.36921-1-tinozzo123%40gmail.com patch subject: [PATCH v3] HID: nintendo: cleanup LED code config: arc-randconfig-r033-20230907 (https://download.01.org/0day-ci/archive/20230907/202309070615.yjnCY1YM-lkp@xxxxxxxxx/config) compiler: arceb-elf-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230907/202309070615.yjnCY1YM-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202309070615.yjnCY1YM-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): In file included from arch/arc/include/asm/ptrace.h:11, from arch/arc/include/asm/unaligned.h:12, from drivers/hid/hid-nintendo.c:26: drivers/hid/hid-nintendo.c: In function 'joycon_leds_create': >> include/linux/compiler.h:55:23: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 55 | #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) ) | ^~ drivers/hid/hid-nintendo.c:1954:17: note: in expansion of macro 'if' 1954 | if (ret) | ^~ drivers/hid/hid-nintendo.c:1956:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 1956 | return ret; | ^~~~~~ >> include/linux/compiler.h:55:23: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 55 | #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) ) | ^~ drivers/hid/hid-nintendo.c:1986:17: note: in expansion of macro 'if' 1986 | if (ret) | ^~ drivers/hid/hid-nintendo.c:1988:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 1988 | return ret; | ^~~~~~ vim +/if +55 include/linux/compiler.h 2bcd521a684cc9 Steven Rostedt 2008-11-21 49 2bcd521a684cc9 Steven Rostedt 2008-11-21 50 #ifdef CONFIG_PROFILE_ALL_BRANCHES 2bcd521a684cc9 Steven Rostedt 2008-11-21 51 /* 2bcd521a684cc9 Steven Rostedt 2008-11-21 52 * "Define 'is'", Bill Clinton 2bcd521a684cc9 Steven Rostedt 2008-11-21 53 * "Define 'if'", Steven Rostedt 2bcd521a684cc9 Steven Rostedt 2008-11-21 54 */ a15fd609ad53a6 Linus Torvalds 2019-03-20 @55 #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) ) a15fd609ad53a6 Linus Torvalds 2019-03-20 56 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
