tree: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git for-leds-next head: b9bd590268485652b928110b5543057543b5d02b commit: efd0d1cbb8c5dd1049922e839fa7d85811facd53 [61/62] leds: leds-lp55xx: Convert mutex lock/unlock to guard API config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20240709/202407092352.NAcaYfjI-lkp@xxxxxxxxx/config) compiler: sh4-linux-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240709/202407092352.NAcaYfjI-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/202407092352.NAcaYfjI-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): drivers/leds/leds-lp5521.c: In function 'lp5521_selftest': >> drivers/leds/leds-lp5521.c:189:33: error: macro "guard" passed 2 arguments, but takes just 1 189 | guard(mutex, &chip->lock); | ^ In file included from drivers/leds/leds-lp5521.c:12: include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | >> drivers/leds/leds-lp5521.c:189:9: error: 'guard' undeclared (first use in this function) 189 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp5521.c:189:9: note: each undeclared identifier is reported only once for each function it appears in -- drivers/leds/leds-lp5523.c: In function 'lp5523_selftest': >> drivers/leds/leds-lp5523.c:192:33: error: macro "guard" passed 2 arguments, but takes just 1 192 | guard(mutex, &chip->lock); | ^ In file included from drivers/leds/leds-lp5523.c:12: include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | >> drivers/leds/leds-lp5523.c:192:9: error: 'guard' undeclared (first use in this function) 192 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp5523.c:192:9: note: each undeclared identifier is reported only once for each function it appears in -- drivers/leds/leds-lp5562.c: In function 'lp5562_led_brightness': >> drivers/leds/leds-lp5562.c:175:33: error: macro "guard" passed 2 arguments, but takes just 1 175 | guard(mutex, &chip->lock); | ^ In file included from drivers/leds/leds-lp5562.c:10: include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | >> drivers/leds/leds-lp5562.c:175:9: error: 'guard' undeclared (first use in this function) 175 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp5562.c:175:9: note: each undeclared identifier is reported only once for each function it appears in drivers/leds/leds-lp5562.c: In function 'lp5562_store_pattern': drivers/leds/leds-lp5562.c:272:33: error: macro "guard" passed 2 arguments, but takes just 1 272 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp5562.c:272:9: error: 'guard' undeclared (first use in this function) 272 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp5562.c: In function 'lp5562_store_engine_mux': drivers/leds/leds-lp5562.c:324:33: error: macro "guard" passed 2 arguments, but takes just 1 324 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp5562.c:324:9: error: 'guard' undeclared (first use in this function) 324 | guard(mutex, &chip->lock); | ^~~~~ -- drivers/leds/leds-lp5569.c: In function 'lp5569_selftest': >> drivers/leds/leds-lp5569.c:400:33: error: macro "guard" passed 2 arguments, but takes just 1 400 | guard(mutex, &chip->lock); | ^ In file included from drivers/leds/leds-lp5569.c:7: include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | >> drivers/leds/leds-lp5569.c:400:9: error: 'guard' undeclared (first use in this function) 400 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp5569.c:400:9: note: each undeclared identifier is reported only once for each function it appears in -- drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_led_brightness': >> drivers/leds/leds-lp55xx-common.c:276:33: error: macro "guard" passed 2 arguments, but takes just 1 276 | guard(mutex, &chip->lock); | ^ In file included from drivers/leds/leds-lp55xx-common.c:13: include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | >> drivers/leds/leds-lp55xx-common.c:276:9: error: 'guard' undeclared (first use in this function) 276 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c:276:9: note: each undeclared identifier is reported only once for each function it appears in drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_multicolor_brightness': drivers/leds/leds-lp55xx-common.c:291:33: error: macro "guard" passed 2 arguments, but takes just 1 291 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:291:9: error: 'guard' undeclared (first use in this function) 291 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'led_current_store': drivers/leds/leds-lp55xx-common.c:409:33: error: macro "guard" passed 2 arguments, but takes just 1 409 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:409:9: error: 'guard' undeclared (first use in this function) 409 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'run_engine_store': drivers/leds/leds-lp55xx-common.c:637:33: error: macro "guard" passed 2 arguments, but takes just 1 637 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:637:9: error: 'guard' undeclared (first use in this function) 637 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_store_engine_mode': drivers/leds/leds-lp55xx-common.c:676:33: error: macro "guard" passed 2 arguments, but takes just 1 676 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:676:9: error: 'guard' undeclared (first use in this function) 676 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_store_engine_load': drivers/leds/leds-lp55xx-common.c:704:33: error: macro "guard" passed 2 arguments, but takes just 1 704 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:704:9: error: 'guard' undeclared (first use in this function) 704 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_store_engine_leds': drivers/leds/leds-lp55xx-common.c:803:33: error: macro "guard" passed 2 arguments, but takes just 1 803 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:803:9: error: 'guard' undeclared (first use in this function) 803 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c:798:17: warning: unused variable 'ret' [-Wunused-variable] 798 | ssize_t ret; | ^~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_show_master_fader': drivers/leds/leds-lp55xx-common.c:827:33: error: macro "guard" passed 2 arguments, but takes just 1 827 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:827:9: error: 'guard' undeclared (first use in this function) 827 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_store_master_fader': drivers/leds/leds-lp55xx-common.c:851:33: error: macro "guard" passed 2 arguments, but takes just 1 851 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:851:9: error: 'guard' undeclared (first use in this function) 851 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_show_master_fader_leds': drivers/leds/leds-lp55xx-common.c:870:33: error: macro "guard" passed 2 arguments, but takes just 1 870 | guard(mutex, &chip->lock); | ^ include/linux/cleanup.h:163: note: macro "guard" defined here 163 | #define guard(_name) \ | drivers/leds/leds-lp55xx-common.c:870:9: error: 'guard' undeclared (first use in this function) 870 | guard(mutex, &chip->lock); | ^~~~~ drivers/leds/leds-lp55xx-common.c: In function 'lp55xx_store_master_fader_leds': drivers/leds/leds-lp55xx-common.c:901:33: error: macro "guard" passed 2 arguments, but takes just 1 901 | guard(mutex, &chip->lock); | ^ vim +/guard +189 drivers/leds/leds-lp5521.c 180 181 static ssize_t lp5521_selftest(struct device *dev, 182 struct device_attribute *attr, 183 char *buf) 184 { 185 struct lp55xx_led *led = i2c_get_clientdata(to_i2c_client(dev)); 186 struct lp55xx_chip *chip = led->chip; 187 int ret; 188 > 189 guard(mutex, &chip->lock); 190 191 ret = lp5521_run_selftest(chip, buf); 192 193 return sysfs_emit(buf, "%s\n", ret ? "FAIL" : "OK"); 194 } 195 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
