linux-kernel - Re: drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'timing0' not described in 'mxs_i2c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Z2i/Gqg6TpWhKiQc@rli9-mobl>
Date: Mon, 23 Dec 2024 09:38:34 +0800
From: Philip Li <philip.li@...el.com>
To: kernel test robot <lkp@...el.com>
CC: Krzysztof Kozlowski <krzk@...nel.org>, <oe-kbuild-all@...ts.linux.dev>,
	<linux-kernel@...r.kernel.org>, Wolfram Sang <wsa-dev@...g-engineering.com>
Subject: Re: drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or
 struct member 'timing0' not described in 'mxs_i2c_dev'

On Mon, Dec 23, 2024 at 09:19:58AM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   4bbf9020becbfd8fc2c3da790855b7042fad455b
> commit: 4a2d5f663dab6614772d8e28ca190b127ba46d9d i2c: Enable compile testing for more drivers
> date:   4 years, 11 months ago
> config: sparc-randconfig-001-20241212 (https://download.01.org/0day-ci/archive/20241223/202412230939.8AJO139S-lkp@intel.com/config)
> compiler: sparc-linux-gcc (GCC) 12.4.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241223/202412230939.8AJO139S-lkp@intel.com/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@...el.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202412230939.8AJO139S-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):

Kindly ignore this report for the wrong bad commit that is not the root cause.

Thanks

> 
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'timing0' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'timing1' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'timing2' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'dmach' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'pio_data' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'addr_data' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'sg_io' not described in 'mxs_i2c_dev'
> >> drivers/i2c/busses/i2c-mxs.c:130: warning: Function parameter or struct member 'dma_read' not described in 'mxs_i2c_dev'
> 
> 
> vim +130 drivers/i2c/busses/i2c-mxs.c
> 
> 616228a164d436 Juergen Beisert 2013-09-30  101  
> a8da7fecb60102 Wolfram Sang    2011-02-16  102  /**
> a8da7fecb60102 Wolfram Sang    2011-02-16  103   * struct mxs_i2c_dev - per device, private MXS-I2C data
> a8da7fecb60102 Wolfram Sang    2011-02-16  104   *
> a8da7fecb60102 Wolfram Sang    2011-02-16  105   * @dev: driver model device node
> 616228a164d436 Juergen Beisert 2013-09-30  106   * @dev_type: distinguish i.MX23/i.MX28 features
> a8da7fecb60102 Wolfram Sang    2011-02-16  107   * @regs: IO registers pointer
> a8da7fecb60102 Wolfram Sang    2011-02-16  108   * @cmd_complete: completion object for transaction wait
> a8da7fecb60102 Wolfram Sang    2011-02-16  109   * @cmd_err: error code for last transaction
> a8da7fecb60102 Wolfram Sang    2011-02-16  110   * @adapter: i2c subsystem adapter node
> a8da7fecb60102 Wolfram Sang    2011-02-16  111   */
> a8da7fecb60102 Wolfram Sang    2011-02-16  112  struct mxs_i2c_dev {
> a8da7fecb60102 Wolfram Sang    2011-02-16  113  	struct device *dev;
> 616228a164d436 Juergen Beisert 2013-09-30  114  	enum mxs_i2c_devtype dev_type;
> a8da7fecb60102 Wolfram Sang    2011-02-16  115  	void __iomem *regs;
> a8da7fecb60102 Wolfram Sang    2011-02-16  116  	struct completion cmd_complete;
> 0f40cbc4f85e13 Fabio Estevam   2013-01-07  117  	int cmd_err;
> a8da7fecb60102 Wolfram Sang    2011-02-16  118  	struct i2c_adapter adapter;
> 626f0a2ff6bc98 Marek Vasut     2012-11-30  119  
> 626f0a2ff6bc98 Marek Vasut     2012-11-30  120  	uint32_t timing0;
> 626f0a2ff6bc98 Marek Vasut     2012-11-30  121  	uint32_t timing1;
> 869c6a3ededbb1 Lothar Waßmann  2013-07-05  122  	uint32_t timing2;
> 62885f59a26195 Marek Vasut     2012-08-24  123  
> 62885f59a26195 Marek Vasut     2012-08-24  124  	/* DMA support components */
> 62885f59a26195 Marek Vasut     2012-08-24  125  	struct dma_chan			*dmach;
> 62885f59a26195 Marek Vasut     2012-08-24  126  	uint32_t			pio_data[2];
> 62885f59a26195 Marek Vasut     2012-08-24  127  	uint32_t			addr_data;
> 62885f59a26195 Marek Vasut     2012-08-24  128  	struct scatterlist		sg_io[2];
> 62885f59a26195 Marek Vasut     2012-08-24  129  	bool				dma_read;
> a8da7fecb60102 Wolfram Sang    2011-02-16 @130  };
> a8da7fecb60102 Wolfram Sang    2011-02-16  131  
> 
> :::::: The code at line 130 was first introduced by commit
> :::::: a8da7fecb6010222fe495cc2b4330b99bca20c61 i2c: add driver for Freescale i.MX28
> 
> :::::: TO: Wolfram Sang <w.sang@...gutronix.de>
> :::::: CC: Ben Dooks <ben-linux@...ff.org>
> 
> -- 
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
>