Re: [PATCH v1 04/12] i2c: isch: Switch to memory mapped IO accessors

From: kernel test robot
Date: Fri Sep 13 2024 - 20:10:53 EST


Hi Andy,

kernel test robot noticed the following build warnings:

[auto build test WARNING on andi-shyti/i2c/i2c-host]
[also build test WARNING on linus/master v6.11-rc7]
[cannot apply to next-20240913]
[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/Andy-Shevchenko/i2c-isch-Add-missed-else/20240912-002224
base: https://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host
patch link: https://lore.kernel.org/r/20240911154820.2846187-5-andriy.shevchenko%40linux.intel.com
patch subject: [PATCH v1 04/12] i2c: isch: Switch to memory mapped IO accessors
config: i386-buildonly-randconfig-001-20240913 (https://download.01.org/0day-ci/archive/20240914/202409140743.kKVc8T3C-lkp@xxxxxxxxx/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240914/202409140743.kKVc8T3C-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/202409140743.kKVc8T3C-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

>> drivers/i2c/busses/i2c-isch.c:296:32: warning: format specifies type 'unsigned int' but the argument has type 'resource_size_t' (aka 'unsigned long long') [-Wformat]
296 | "SMBus SCH adapter at %04x", res->start);
| ~~~~ ^~~~~~~~~~
| %04llx
1 warning generated.


vim +296 drivers/i2c/busses/i2c-isch.c

276
277 static int smbus_sch_probe(struct platform_device *dev)
278 {
279 struct resource *res;
280 int retval;
281
282 res = platform_get_resource(dev, IORESOURCE_IO, 0);
283 if (!res)
284 return -EBUSY;
285
286 sch_smba = devm_ioport_map(&dev->dev, res->start, resource_size(res));
287 if (!sch_smba) {
288 dev_err(&dev->dev, "SMBus region %pR already in use!\n", res);
289 return -EBUSY;
290 }
291
292 /* set up the sysfs linkage to our parent device */
293 sch_adapter.dev.parent = &dev->dev;
294
295 snprintf(sch_adapter.name, sizeof(sch_adapter.name),
> 296 "SMBus SCH adapter at %04x", res->start);
297
298 retval = i2c_add_adapter(&sch_adapter);
299 if (retval)
300 sch_smba = NULL;
301
302 return retval;
303 }
304

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