mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Frank Li <Frank.Li@nxp.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
	Vinod Koul <vkoul@kernel.org>
Subject: drivers/dma/fsl-qdma.c:170: warning: Function parameter or struct member '__reserved2' not described in 'fsl_qdma_format'
Date: Thu, 13 Feb 2025 21:55:42 +0800	[thread overview]
Message-ID: <202502132121.B7Z5mQ2y-lkp@intel.com> (raw)

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@intel.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@nxp.com>
:::::: CC: Vinod Koul <vkoul@kernel.org>

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

                 reply	other threads:[~2025-02-13 13:56 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202502132121.B7Z5mQ2y-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=Frank.Li@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=vkoul@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®