Re: [PATCH 06/18] mfd: adp5585: add support for adp5589

["Nuno Sá" <noname.nuno@gmail.com>]

On Fri, 2025-03-14 at 17:23 +0800, kernel test robot wrote:
> Hi Nuno,
> 
> kernel test robot noticed the following build warnings:
> 
> [auto build test WARNING on 4d395cb071a343196ca524d3694790f06978fe91]
> 
> url:   
> https://github.com/intel-lab-lkp/linux/commits/Nuno-S-via-B4-Relay/dt-bindings-mfd-adp5585-ease-on-the-required-properties/20250313-222511
> base:   4d395cb071a343196ca524d3694790f06978fe91
> patch link:   
> https://lore.kernel.org/r/20250313-dev-adp5589-fw-v1-6-20e80d4bd4ea%40analog.com
> patch subject: [PATCH 06/18] mfd: adp5585: add support for adp5589
> config: s390-randconfig-001-20250314
> (https://download.01.org/0day-ci/archive/20250314/202503141715.1e4xyKyq-lkp@in
> tel.com/config)
> compiler: s390-linux-gcc (GCC) 14.2.0
> reproduce (this is a W=1 build):
> (https://download.01.org/0day-ci/archive/20250314/202503141715.1e4xyKyq-lkp@in
> tel.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/202503141715.1e4xyKyq-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):
> 
> > > drivers/mfd/adp5585.c:249:34: warning: 'adp5585_02_info' defined but not
> > > used [-Wunused-const-variable=]
>      249 | static const struct adp5585_info adp5585_02_info = {
>          |                                  ^~~~~~~~~~~~~~~
> > > drivers/mfd/adp5585.c:239:34: warning: 'adp5585_01_info' defined but not
> > > used [-Wunused-const-variable=]
>      239 | static const struct adp5585_info adp5585_01_info = {
>          |                                  ^~~~~~~~~~~~~~~
> > > drivers/mfd/adp5585.c:229:34: warning: 'adp5585_info' defined but not used
> > > [-Wunused-const-variable=]
>      229 | static const struct adp5585_info adp5585_info = {
>          |                                  ^~~~~~~~~~~~
> > > drivers/mfd/adp5585.c:48:41: warning: 'adp5589_volatile_regs' defined but
> > > not used [-Wunused-const-variable=]
>       48 | static const struct regmap_access_table adp5589_volatile_regs = {
>          |                                         ^~~~~~~~~~~~~~~~~~~~~
> 

ouchhh!!!

will fix in v2

- Nuno Sá
> 
> vim +/adp5585_02_info +249 drivers/mfd/adp5585.c
> 
>    228	
>  > 229	static const struct adp5585_info adp5585_info = {
>    230		.adp5585_devs = adp5585_devs,
>    231		.regmap_config = &adp5585_regmap_configs[ADP5585_REGMAP_00],
>    232		.n_devs = ARRAY_SIZE(adp5585_devs),
>    233		.id = ADP5585_MAN_ID_VALUE,
>    234		.regs = &adp5585_regs,
>    235		.max_rows = ADP5585_MAX_ROW_NUM,
>    236		.max_cols = ADP5585_MAX_COL_NUM,
>    237	};
>    238	
>  > 239	static const struct adp5585_info adp5585_01_info = {
>    240		.adp5585_devs = adp5585_devs,
>    241		.regmap_config = &adp5585_regmap_configs[ADP5585_REGMAP_00],
>    242		.n_devs = ARRAY_SIZE(adp5585_devs),
>    243		.id = ADP5585_MAN_ID_VALUE,
>    244		.regs = &adp5585_regs,
>    245		.max_rows = ADP5585_MAX_ROW_NUM,
>    246		.max_cols = ADP5585_MAX_COL_NUM,
>    247	};
>    248	
>  > 249	static const struct adp5585_info adp5585_02_info = {
>    250		.adp5585_devs = adp5585_devs,
>    251		.regmap_config = &adp5585_regmap_configs[ADP5585_REGMAP_02],
>    252		.n_devs = ARRAY_SIZE(adp5585_devs),
>    253		.id = ADP5585_MAN_ID_VALUE,
>    254		.regs = &adp5585_regs,
>    255		.max_rows = ADP5585_MAX_ROW_NUM,
>    256		.max_cols = ADP5585_MAX_COL_NUM,
>    257	};
>    258	
>