Re: [PATCH v1 2/2] leds: Add LED1202 I2C driver

From: kernel test robot
Date: Tue Jun 18 2024 - 08:25:02 EST


Hi Vicentiu,

kernel test robot noticed the following build errors:

[auto build test ERROR on lee-leds/for-leds-next]
[also build test ERROR on robh/for-next linus/master v6.10-rc4 next-20240617]
[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/Vicentiu-Galanopulo/leds-Add-LED1202-I2C-driver/20240618-052035
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git for-leds-next
patch link: https://lore.kernel.org/r/ZnCoBAG2TNiIz1xE%40admins-Air
patch subject: [PATCH v1 2/2] leds: Add LED1202 I2C driver
config: openrisc-allyesconfig (https://download.01.org/0day-ci/archive/20240618/202406181953.mXAkCdSy-lkp@xxxxxxxxx/config)
compiler: or1k-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240618/202406181953.mXAkCdSy-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/202406181953.mXAkCdSy-lkp@xxxxxxxxx/

All errors (new ones prefixed by >>):

In file included from include/linux/container_of.h:5,
from include/linux/list.h:5,
from include/linux/module.h:12,
from drivers/leds/leds-led1202.c:7:
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:396:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
396 | BUILD_BUG_ON_ZERO((perms) > 0777) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:373:8: note: in expansion of macro 'DEVICE_ATTR'
373 | static DEVICE_ATTR(led_device_regsdump, 0x00444, ll1202_show_all_registers,
| ^~~~~~~~~~~
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:399:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
399 | BUILD_BUG_ON_ZERO((((perms) >> 3) & 4) < ((perms) & 4)) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:373:8: note: in expansion of macro 'DEVICE_ATTR'
373 | static DEVICE_ATTR(led_device_regsdump, 0x00444, ll1202_show_all_registers,
| ^~~~~~~~~~~
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:396:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
396 | BUILD_BUG_ON_ZERO((perms) > 0777) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:375:8: note: in expansion of macro 'DEVICE_ATTR'
375 | static DEVICE_ATTR(patt_sequence_repetition, 0x00444 | 0x00200,
| ^~~~~~~~~~~
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:399:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
399 | BUILD_BUG_ON_ZERO((((perms) >> 3) & 4) < ((perms) & 4)) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:375:8: note: in expansion of macro 'DEVICE_ATTR'
375 | static DEVICE_ATTR(patt_sequence_repetition, 0x00444 | 0x00200,
| ^~~~~~~~~~~
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:396:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
396 | BUILD_BUG_ON_ZERO((perms) > 0777) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:378:8: note: in expansion of macro 'DEVICE_ATTR'
378 | static DEVICE_ATTR(current_mA, 0x00444, ll1202_show_channel_mA_current, NULL);
| ^~~~~~~~~~~
>> include/linux/build_bug.h:16:51: error: negative width in bit-field '<anonymous>'
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/kernel.h:399:10: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
399 | BUILD_BUG_ON_ZERO((((perms) >> 3) & 4) < ((perms) & 4)) + \
| ^~~~~~~~~~~~~~~~~
include/linux/sysfs.h:221:26: note: in expansion of macro 'VERIFY_OCTAL_PERMISSIONS'
221 | .mode = VERIFY_OCTAL_PERMISSIONS(_mode) }, \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/device.h:158:52: note: in expansion of macro '__ATTR'
158 | struct device_attribute dev_attr_##_name = __ATTR(_name, _mode, _show, _store)
| ^~~~~~
drivers/leds/leds-led1202.c:378:8: note: in expansion of macro 'DEVICE_ATTR'
378 | static DEVICE_ATTR(current_mA, 0x00444, ll1202_show_channel_mA_current, NULL);
| ^~~~~~~~~~~


vim +16 include/linux/build_bug.h

bc6245e5efd70c Ian Abbott 2017-07-10 6
bc6245e5efd70c Ian Abbott 2017-07-10 7 #ifdef __CHECKER__
bc6245e5efd70c Ian Abbott 2017-07-10 8 #define BUILD_BUG_ON_ZERO(e) (0)
bc6245e5efd70c Ian Abbott 2017-07-10 9 #else /* __CHECKER__ */
bc6245e5efd70c Ian Abbott 2017-07-10 10 /*
bc6245e5efd70c Ian Abbott 2017-07-10 11 * Force a compilation error if condition is true, but also produce a
8788994376d84d Rikard Falkeborn 2019-12-04 12 * result (of value 0 and type int), so the expression can be used
bc6245e5efd70c Ian Abbott 2017-07-10 13 * e.g. in a structure initializer (or where-ever else comma expressions
bc6245e5efd70c Ian Abbott 2017-07-10 14 * aren't permitted).
bc6245e5efd70c Ian Abbott 2017-07-10 15 */
8788994376d84d Rikard Falkeborn 2019-12-04 @16 #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
527edbc18a70e7 Masahiro Yamada 2019-01-03 17 #endif /* __CHECKER__ */
527edbc18a70e7 Masahiro Yamada 2019-01-03 18

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki