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>] [day] [month] [year] [list]
Message-ID: <202502132121.B7Z5mQ2y-lkp@intel.com>
Date: Thu, 13 Feb 2025 21:55:42 +0800
From: kernel test robot <lkp@...el.com>
To: Frank Li <Frank.Li@....com>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
	Vinod Koul <vkoul@...nel.org>
Subject: drivers/dma/fsl-qdma.c:170: warning: Function parameter or struct
 member '__reserved2' not described in 'fsl_qdma_format'

Hi Frank,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   4dc1d1bec89864d8076e5ab314f86f46442bfb02
commit: 1878840a0328dac1c85d29fee31456ec26fcc01c dmaengine: fsl-qdma: add __iomem and struct in union to fix sparse warning
date:   12 months ago
config: arm-randconfig-004-20230928 (https://download.01.org/0day-ci/archive/20250213/202502132121.B7Z5mQ2y-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.4.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250213/202502132121.B7Z5mQ2y-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/202502132121.B7Z5mQ2y-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/dma/fsl-qdma.c:170: warning: Function parameter or struct member '__reserved2' not described in 'fsl_qdma_format'
>> drivers/dma/fsl-qdma.c:170: warning: Function parameter or struct member 'cmd' not described in 'fsl_qdma_format'


vim +170 drivers/dma/fsl-qdma.c

b092529e0aa098 Peng Ma   2018-10-30  135  
b092529e0aa098 Peng Ma   2018-10-30  136  #define FSL_QDMA_BLOCK_BASE_OFFSET(fsl_qdma_engine, x)			\
b092529e0aa098 Peng Ma   2018-10-30  137  	(((fsl_qdma_engine)->block_offset) * (x))
b092529e0aa098 Peng Ma   2018-10-30  138  
b092529e0aa098 Peng Ma   2018-10-30  139  /**
b092529e0aa098 Peng Ma   2018-10-30  140   * struct fsl_qdma_format - This is the struct holding describing compound
b092529e0aa098 Peng Ma   2018-10-30  141   *			    descriptor format with qDMA.
b092529e0aa098 Peng Ma   2018-10-30  142   * @status:		    Command status and enqueue status notification.
b092529e0aa098 Peng Ma   2018-10-30  143   * @cfg:		    Frame offset and frame format.
b092529e0aa098 Peng Ma   2018-10-30  144   * @addr_lo:		    Holding the compound descriptor of the lower
b092529e0aa098 Peng Ma   2018-10-30  145   *			    32-bits address in memory 40-bit address.
b092529e0aa098 Peng Ma   2018-10-30  146   * @addr_hi:		    Same as above member, but point high 8-bits in
b092529e0aa098 Peng Ma   2018-10-30  147   *			    memory 40-bit address.
b092529e0aa098 Peng Ma   2018-10-30  148   * @__reserved1:	    Reserved field.
b092529e0aa098 Peng Ma   2018-10-30  149   * @cfg8b_w1:		    Compound descriptor command queue origin produced
b092529e0aa098 Peng Ma   2018-10-30  150   *			    by qDMA and dynamic debug field.
041c4646003e23 Lee Jones 2020-07-14  151   * @data:		    Pointer to the memory 40-bit address, describes DMA
b092529e0aa098 Peng Ma   2018-10-30  152   *			    source information and DMA destination information.
b092529e0aa098 Peng Ma   2018-10-30  153   */
b092529e0aa098 Peng Ma   2018-10-30  154  struct fsl_qdma_format {
b092529e0aa098 Peng Ma   2018-10-30  155  	__le32 status;
b092529e0aa098 Peng Ma   2018-10-30  156  	__le32 cfg;
b092529e0aa098 Peng Ma   2018-10-30  157  	union {
b092529e0aa098 Peng Ma   2018-10-30  158  		struct {
b092529e0aa098 Peng Ma   2018-10-30  159  			__le32 addr_lo;
b092529e0aa098 Peng Ma   2018-10-30  160  			u8 addr_hi;
b092529e0aa098 Peng Ma   2018-10-30  161  			u8 __reserved1[2];
b092529e0aa098 Peng Ma   2018-10-30  162  			u8 cfg8b_w1;
b092529e0aa098 Peng Ma   2018-10-30  163  		} __packed;
1878840a0328da Frank Li  2024-02-19  164  		struct {
1878840a0328da Frank Li  2024-02-19  165  			__le32 __reserved2;
1878840a0328da Frank Li  2024-02-19  166  			__le32 cmd;
1878840a0328da Frank Li  2024-02-19  167  		} __packed;
b092529e0aa098 Peng Ma   2018-10-30  168  		__le64 data;
b092529e0aa098 Peng Ma   2018-10-30  169  	};
b092529e0aa098 Peng Ma   2018-10-30 @170  } __packed;
b092529e0aa098 Peng Ma   2018-10-30  171  

:::::: The code at line 170 was first introduced by commit
:::::: b092529e0aa09829a6404424ce167bf3ce3235e2 dmaengine: fsl-qdma: Add qDMA controller driver for Layerscape SoCs

:::::: TO: Peng Ma <peng.ma@....com>
:::::: CC: Vinod Koul <vkoul@...nel.org>

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ