Hi Andy, On Mon, Sep 16, 2024 at 12:07:28PM GMT, Andy Shevchenko wrote: > On Sat, Sep 14, 2024 at 02:56:19PM +0800, kernel test robot wrote: > > 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: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20240914/202409141436.QFCDQrRF-lkp@xxxxxxxxx/config) > > compiler: alpha-linux-gcc (GCC) 13.3.0 > > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240914/202409141436.QFCDQrRF-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/202409141436.QFCDQrRF-lkp@xxxxxxxxx/ > > > > All warnings (new ones prefixed by >>): > > > > drivers/i2c/busses/i2c-isch.c: In function 'smbus_sch_probe': > > >> drivers/i2c/busses/i2c-isch.c:296:42: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'resource_size_t' {aka 'long long unsigned int'} [-Wformat=] > > 296 | "SMBus SCH adapter at %04x", res->start); > > | ~~~^ ~~~~~~~~~~ > > | | | > > | | resource_size_t {aka long long unsigned int} > > | unsigned int > > | %04llx > > Yeah, this should be something like %pa, but the problem with that that it > always uses the same, fixed-width format with a prefix. We don't want this. But > to make sure we have proper specifier we need to introduce a temporary variable > and assign the resource start address to it and then use that variable in here. > I'll update this in v2 and send it after we have v6.12-rc1 is out. Feel free to send it, I will apply it in i2c/i2c-host-for-6.12, that's where I'm collecting the next patches. Andi
