Re: [PATCH 2/2] drivers: i3c: Add driver for NXP P3H2x4x i3c-hub device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Aman,

kernel test robot noticed the following build warnings:

[auto build test WARNING on robh/for-next]
[also build test WARNING on linus/master v6.14-rc2 next-20250213]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Aman-Kumar-Pandey/drivers-i3c-Add-driver-for-NXP-P3H2x4x-i3c-hub-device/20250212-213659
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link:    https://lore.kernel.org/r/20250212132227.1348374-2-aman.kumarpandey%40nxp.com
patch subject: [PATCH 2/2] drivers: i3c: Add driver for NXP P3H2x4x i3c-hub device
config: loongarch-allmodconfig (https://download.01.org/0day-ci/archive/20250214/202502140107.5TZoA3GU-lkp@xxxxxxxxx/config)
compiler: loongarch64-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250214/202502140107.5TZoA3GU-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202502140107.5TZoA3GU-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

   In file included from drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub_i3c.c:7:
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:351:37: warning: 'io_strength_settings' defined but not used [-Wunused-const-variable=]
     351 | static const struct p3h2x4x_setting io_strength_settings[] = {
         |                                     ^~~~~~~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:345:37: warning: 'tp_pullup_settings' defined but not used [-Wunused-const-variable=]
     345 | static const struct p3h2x4x_setting tp_pullup_settings[] = {
         |                                     ^~~~~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:337:37: warning: 'tp_mode_settings' defined but not used [-Wunused-const-variable=]
     337 | static const struct p3h2x4x_setting tp_mode_settings[] = {
         |                                     ^~~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:329:37: warning: 'pullup_settings' defined but not used [-Wunused-const-variable=]
     329 | static const struct p3h2x4x_setting pullup_settings[] = {
         |                                     ^~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:321:37: warning: 'ldo_volt_settings' defined but not used [-Wunused-const-variable=]
     321 | static const struct p3h2x4x_setting ldo_volt_settings[] = {
         |                                     ^~~~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:315:37: warning: 'ldo_en_settings' defined but not used [-Wunused-const-variable=]
     315 | static const struct p3h2x4x_setting ldo_en_settings[] = {
         |                                     ^~~~~~~~~~~~~~~
>> drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h:309:37: warning: 'ibi_en_settings' defined but not used [-Wunused-const-variable=]
     309 | static const struct p3h2x4x_setting ibi_en_settings[] = {
         |                                     ^~~~~~~~~~~~~~~


vim +/io_strength_settings +351 drivers/i3c/p3h2x4x/p3h2x4x_i3c_hub.h

   307	
   308	/* IBI enable/disable settings */
 > 309	static const struct p3h2x4x_setting ibi_en_settings[] = {
   310		{ "disabled",	P3H2x4x_IBI_DISABLED },
   311		{ "enabled",	P3H2x4x_IBI_ENABLED },
   312	};
   313	
   314	/* LDO enable/disable settings */
 > 315	static const struct p3h2x4x_setting ldo_en_settings[] = {
   316		{ "disabled",	P3H2x4x_LDO_DISABLED },
   317		{ "enabled",	P3H2x4x_LDO_ENABLED },
   318	};
   319	
   320	/* LDO voltage settings */
 > 321	static const struct p3h2x4x_setting ldo_volt_settings[] = {
   322		{ "1.0V",	P3H2x4x_LDO_VOLT_1_0V },
   323		{ "1.1V",	P3H2x4x_LDO_VOLT_1_1V },
   324		{ "1.2V",	P3H2x4x_LDO_VOLT_1_2V },
   325		{ "1.8V",	P3H2x4x_LDO_VOLT_1_8V },
   326	};
   327	
   328	/* target port pull-up settings */
 > 329	static const struct p3h2x4x_setting pullup_settings[] = {
   330		{ "250R",		P3H2x4x_TP_PULLUP_250R },
   331		{ "500R",		P3H2x4x_TP_PULLUP_500R },
   332		{ "1000R",		P3H2x4x_TP_PULLUP_1000R },
   333		{ "2000R",		P3H2x4x_TP_PULLUP_2000R },
   334	};
   335	
   336	/* target port mode settings */
 > 337	static const struct p3h2x4x_setting tp_mode_settings[] = {
   338		{ "i3c",		P3H2x4x_TP_MODE_I3C },
   339		{ "smbus",		P3H2x4x_TP_MODE_SMBUS },
   340		{ "gpio",		P3H2x4x_TP_MODE_GPIO },
   341		{ "i2c",		P3H2x4x_TP_MODE_I2C },
   342	};
   343	
   344	/* pull-up enable/disable settings */
 > 345	static const struct p3h2x4x_setting tp_pullup_settings[] = {
   346		{ "disabled",	P3H2x4x_TP_PULLUP_DISABLED },
   347		{ "enabled",	P3H2x4x_TP_PULLUP_ENABLED },
   348	};
   349	
   350	/*  IO strenght settings */
 > 351	static const struct p3h2x4x_setting io_strength_settings[] = {
   352		{ "20Ohms",		P3H2x4x_IO_STRENGTH_20_OHM },
   353		{ "30Ohms",		P3H2x4x_IO_STRENGTH_30_OHM },
   354		{ "40Ohms",		P3H2x4x_IO_STRENGTH_40_OHM },
   355		{ "50Ohms",		P3H2x4x_IO_STRENGTH_50_OHM },
   356	};
   357	

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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux