Re: [PATCH V2 4/5] iio: imu: inv_icm42600: Add support for icm42607

[kernel test robot <lkp@intel.com>]

Hi Chris,

kernel test robot noticed the following build warnings:

[auto build test WARNING on rockchip/for-next]
[also build test WARNING on linus/master v7.0-rc4 next-20260320]
[cannot apply to jic23-iio/togreg]
[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/Chris-Morgan/dt-bindings-iio-imu-add-icm42607/20260320-061927
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git for-next
patch link:    https://lore.kernel.org/r/20260319182956.146976-5-macroalpha82%40gmail.com
patch subject: [PATCH V2 4/5] iio: imu: inv_icm42600: Add support for icm42607
config: i386-randconfig-r134-20260320 (https://download.01.org/0day-ci/archive/20260321/202603210032.xLBroWQu-lkp@intel.com/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
sparse: v0.6.5-rc1
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260321/202603210032.xLBroWQu-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/202603210032.xLBroWQu-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/iio/imu/inv_icm42600/inv_icm42600_core.c: In function 'inv_icm42607_hw_suspend':
>> drivers/iio/imu/inv_icm42600/inv_icm42600_core.c:1150:13: warning: variable 'accel_conf' set but not used [-Wunused-but-set-variable]
    1150 |         int accel_conf;
         |             ^~~~~~~~~~
--
>> drivers/iio/imu/inv_icm42600/inv_icm42600_accel.c:108:18: warning: 'inv_icm42607_accel_filter_values' defined but not used [-Wunused-const-variable=]
     108 | static const int inv_icm42607_accel_filter_values[] = {
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


vim +/accel_conf +1150 drivers/iio/imu/inv_icm42600/inv_icm42600_core.c

  1144	
  1145	static int inv_icm42607_hw_suspend(struct device *dev)
  1146	{
  1147		struct inv_icm42600_state *st = dev_get_drvdata(dev);
  1148		struct device *accel_dev;
  1149		bool wakeup;
> 1150		int accel_conf;
  1151		int ret = 0;
  1152	
  1153		guard(mutex)(&st->lock);
  1154	
  1155		st->suspended.gyro = st->conf.gyro.mode;
  1156		st->suspended.accel = st->conf.accel.mode;
  1157		st->suspended.temp = st->conf.temp_en;
  1158		if (pm_runtime_suspended(dev))
  1159			return 0;
  1160	
  1161		if (st->fifo.on) {
  1162			ret = regmap_write(st->map, INV_ICM42607_REG_FIFO_CONFIG1,
  1163					   INV_ICM42607_FIFO_CONFIG1_BYPASS);
  1164			if (ret)
  1165				return ret;
  1166		}
  1167	
  1168		/* keep chip on and wake-up capable if APEX and wakeup on */
  1169		accel_dev = &st->indio_accel->dev;
  1170		wakeup = st->apex.on && device_may_wakeup(accel_dev);
  1171		if (wakeup) {
  1172			/* keep accel on and setup irq for wakeup */
  1173			accel_conf = st->conf.accel.mode;
  1174			enable_irq_wake(st->irq);
  1175			disable_irq(st->irq);
  1176		} else {
  1177			/* disable APEX features and accel if wakeup disabled */
  1178			if (st->apex.wom.enable) {
  1179				ret = inv_icm42607_disable_wom(st);
  1180				if (ret)
  1181					return ret;
  1182			}
  1183			accel_conf = INV_ICM42600_SENSOR_MODE_OFF;
  1184		}
  1185	
  1186		ret = inv_icm42607_set_pwr_mgmt0(st, INV_ICM42600_SENSOR_MODE_OFF,
  1187						 INV_ICM42600_SENSOR_MODE_OFF, false,
  1188						 NULL);
  1189		if (ret)
  1190			return ret;
  1191	
  1192		if (!wakeup)
  1193			regulator_disable(st->vddio_supply);
  1194	
  1195		return 0;
  1196	}
  1197	

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

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip