Re: drivers/memory/emif.c:70: warning: Function parameter or struct member 'lpmode' not described in 'emif_data'

[Philip Li <philip.li@intel.com>]

On Sat, Dec 28, 2024 at 05:55:47PM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   fd0584d220fe285dc45be43eede55df89ad6a3d9
> commit: ea0c0ad6b6eb36726088991d97a55b99cae456d0 memory: Enable compile testing for most of the drivers
> date:   4 years, 4 months ago
> config: sparc-randconfig-001-20241212 (https://download.01.org/0day-ci/archive/20241228/202412281715.ZOR40f8d-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/20241228/202412281715.ZOR40f8d-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/202412281715.ZOR40f8d-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):

Sorry, please ignore this report, it is not the actual cause.

> 
> >> drivers/memory/emif.c:70: warning: Function parameter or struct member 'lpmode' not described in 'emif_data'
>    drivers/memory/emif.c:70: warning: Function parameter or struct member 'irq_state' not described in 'emif_data'
>    drivers/memory/emif.c:70: warning: Function parameter or struct member 'addressing' not described in 'emif_data'
> 
> 
> vim +70 drivers/memory/emif.c
> 
> 7ec944538dde3d Aneesh V 2012-04-27  30  
> 7ec944538dde3d Aneesh V 2012-04-27  31  /**
> 7ec944538dde3d Aneesh V 2012-04-27  32   * struct emif_data - Per device static data for driver's use
> 7ec944538dde3d Aneesh V 2012-04-27  33   * @duplicate:			Whether the DDR devices attached to this EMIF
> 7ec944538dde3d Aneesh V 2012-04-27  34   *				instance are exactly same as that on EMIF1. In
> 7ec944538dde3d Aneesh V 2012-04-27  35   *				this case we can save some memory and processing
> 7ec944538dde3d Aneesh V 2012-04-27  36   * @temperature_level:		Maximum temperature of LPDDR2 devices attached
> 7ec944538dde3d Aneesh V 2012-04-27  37   *				to this EMIF - read from MR4 register. If there
> 7ec944538dde3d Aneesh V 2012-04-27  38   *				are two devices attached to this EMIF, this
> 7ec944538dde3d Aneesh V 2012-04-27  39   *				value is the maximum of the two temperature
> 7ec944538dde3d Aneesh V 2012-04-27  40   *				levels.
> 7ec944538dde3d Aneesh V 2012-04-27  41   * @node:			node in the device list
> 7ec944538dde3d Aneesh V 2012-04-27  42   * @base:			base address of memory-mapped IO registers.
> 7ec944538dde3d Aneesh V 2012-04-27  43   * @dev:			device pointer.
> a93de288aad3b0 Aneesh V 2012-04-27  44   * @addressing			table with addressing information from the spec
> a93de288aad3b0 Aneesh V 2012-04-27  45   * @regs_cache:			An array of 'struct emif_regs' that stores
> a93de288aad3b0 Aneesh V 2012-04-27  46   *				calculated register values for different
> a93de288aad3b0 Aneesh V 2012-04-27  47   *				frequencies, to avoid re-calculating them on
> a93de288aad3b0 Aneesh V 2012-04-27  48   *				each DVFS transition.
> a93de288aad3b0 Aneesh V 2012-04-27  49   * @curr_regs:			The set of register values used in the last
> a93de288aad3b0 Aneesh V 2012-04-27  50   *				frequency change (i.e. corresponding to the
> a93de288aad3b0 Aneesh V 2012-04-27  51   *				frequency in effect at the moment)
> 7ec944538dde3d Aneesh V 2012-04-27  52   * @plat_data:			Pointer to saved platform data.
> aac10aaa8cc65a Aneesh V 2012-04-27  53   * @debugfs_root:		dentry to the root folder for EMIF in debugfs
> e6b42eb6a66c18 Aneesh V 2012-08-17  54   * @np_ddr:			Pointer to ddr device tree node
> 7ec944538dde3d Aneesh V 2012-04-27  55   */
> 7ec944538dde3d Aneesh V 2012-04-27  56  struct emif_data {
> 7ec944538dde3d Aneesh V 2012-04-27  57  	u8				duplicate;
> 7ec944538dde3d Aneesh V 2012-04-27  58  	u8				temperature_level;
> a93de288aad3b0 Aneesh V 2012-04-27  59  	u8				lpmode;
> 7ec944538dde3d Aneesh V 2012-04-27  60  	struct list_head		node;
> a93de288aad3b0 Aneesh V 2012-04-27  61  	unsigned long			irq_state;
> 7ec944538dde3d Aneesh V 2012-04-27  62  	void __iomem			*base;
> 7ec944538dde3d Aneesh V 2012-04-27  63  	struct device			*dev;
> a93de288aad3b0 Aneesh V 2012-04-27  64  	const struct lpddr2_addressing	*addressing;
> a93de288aad3b0 Aneesh V 2012-04-27  65  	struct emif_regs		*regs_cache[EMIF_MAX_NUM_FREQUENCIES];
> a93de288aad3b0 Aneesh V 2012-04-27  66  	struct emif_regs		*curr_regs;
> 7ec944538dde3d Aneesh V 2012-04-27  67  	struct emif_platform_data	*plat_data;
> aac10aaa8cc65a Aneesh V 2012-04-27  68  	struct dentry			*debugfs_root;
> e6b42eb6a66c18 Aneesh V 2012-08-17  69  	struct device_node		*np_ddr;
> 7ec944538dde3d Aneesh V 2012-04-27 @70  };
> 7ec944538dde3d Aneesh V 2012-04-27  71  
> 
> :::::: The code at line 70 was first introduced by commit
> :::::: 7ec944538dde3d7f490bd4d2619051789db5c3c3 memory: emif: add basic infrastructure for EMIF driver
> 
> :::::: TO: Aneesh V <aneesh@ti.com>
> :::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
> -- 
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
>