Re: [PATCH] scripts/kernel-doc: restore warning for Excess struct/union

From: Randy Dunlap
Date: Fri Dec 15 2023 - 00:39:15 EST



On 12/14/23 21:35, kernel test robot wrote:
> Hi Randy,
>
> kernel test robot noticed the following build warnings:
>

That's great. Thanks.

> [auto build test WARNING on lwn/docs-next]
> [also build test WARNING on linus/master v6.7-rc5 next-20231214]
> [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/Randy-Dunlap/scripts-kernel-doc-restore-warning-for-Excess-struct-union/20231214-150722
> base: git://git.lwn.net/linux.git docs-next
> patch link: https://lore.kernel.org/r/20231214070200.24405-1-rdunlap%40infradead.org
> patch subject: [PATCH] scripts/kernel-doc: restore warning for Excess struct/union
> config: arm-randconfig-r081-20231214 (https://download.01.org/0day-ci/archive/20231215/202312151356.OZmUHKRx-lkp@xxxxxxxxx/config)
> compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231215/202312151356.OZmUHKRx-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/202312151356.OZmUHKRx-lkp@xxxxxxxxx/
>
> All warnings (new ones prefixed by >>):
>
>>> drivers/gpio/gpio-syscon.c:46: warning: Excess struct member 'compatible' description in 'syscon_gpio_data'
> --
>>> drivers/gpio/gpio-xilinx.c:75: warning: Excess struct member 'irqchip' description in 'xgpio_instance'
> --
>>> net/tipc/link.c:228: warning: Excess struct member 'media_addr' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'timer' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'refcnt' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'proto_msg' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'pmsg' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'backlog_limit' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'exp_msg_count' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'reset_rcv_checkpt' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'transmitq' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'snt_nxt' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'deferred_queue' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'unacked_window' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'next_out' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'long_msg_seq_no' description in 'tipc_link'
>>> net/tipc/link.c:228: warning: Excess struct member 'bc_rcvr' description in 'tipc_link'
> --
>>> net/tipc/node.c:150: warning: Excess struct member 'inputq' description in 'tipc_node'
>>> net/tipc/node.c:150: warning: Excess struct member 'namedq' description in 'tipc_node'
> --
>>> net/tipc/socket.c:143: warning: Excess struct member 'blocking_link' description in 'tipc_sock'
> --
>>> drivers/hte/hte.c:90: warning: Excess struct member 'ei' description in 'hte_device'
> --
>>> drivers/i2c/i2c-atr.c:98: warning: Excess struct member 'adapter' description in 'i2c_atr'
> --
>>> drivers/leds/leds-lm3697.c:93: warning: Excess struct member 'leds' description in 'lm3697'
> --
>>> drivers/leds/leds-mlxreg.c:42: warning: Excess struct member 'led_data' description in 'mlxreg_led_data'
> --
>>> drivers/of/property.c:1240: warning: Excess struct member 'parse_prop.np' description in 'supplier_bindings'
>>> drivers/of/property.c:1240: warning: Excess struct member 'parse_prop.prop_name' description in 'supplier_bindings'
>>> drivers/of/property.c:1240: warning: Excess struct member 'parse_prop.index' description in 'supplier_bindings'
> --
>>> drivers/reset/core.c:64: warning: Excess struct member 'rstc' description in 'reset_control_array'
> ..
>
>
> vim +46 drivers/gpio/gpio-syscon.c
>
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 23
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 24 /**
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 25 * struct syscon_gpio_data - Configuration for the device.
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 26 * @compatible: SYSCON driver compatible string.
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 27 * @flags: Set of GPIO_SYSCON_FEAT_ flags:
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 28 * GPIO_SYSCON_FEAT_IN: GPIOs supports input,
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 29 * GPIO_SYSCON_FEAT_OUT: GPIOs supports output,
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 30 * GPIO_SYSCON_FEAT_DIR: GPIOs supports switch direction.
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 31 * @bit_count: Number of bits used as GPIOs.
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 32 * @dat_bit_offset: Offset (in bits) to the first GPIO bit.
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 33 * @dir_bit_offset: Optional offset (in bits) to the first bit to switch
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 34 * GPIO direction (Used with GPIO_SYSCON_FEAT_DIR flag).
> 5f3beb67b5d1c5 Lee Jones 2020-06-30 35 * @set: HW specific callback to assigns output value
> 2c341d62eb4b69 Grygorii Strashko 2014-09-03 36 * for signal "offset"
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 37 */
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 38
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 39 struct syscon_gpio_data {
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 40 unsigned int flags;
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 41 unsigned int bit_count;
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 42 unsigned int dat_bit_offset;
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 43 unsigned int dir_bit_offset;
> 2c341d62eb4b69 Grygorii Strashko 2014-09-03 44 void (*set)(struct gpio_chip *chip,
> 2c341d62eb4b69 Grygorii Strashko 2014-09-03 45 unsigned offset, int value);
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 @46 };
> 6a8a0c1d87377c Alexander Shiyan 2014-03-11 47
>

--
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html