Re: [PATCH v5 4/8] i2c: amd-asf: Add ACPI support for AMD ASF Controller

Re: [PATCH v5 4/8] i2c: amd-asf: Add ACPI support for AMD ASF Controller

[kernel test robot]

Hi Shyam,

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 next-20240919]
[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/Shyam-Sundar-S-K/i2c-piix4-Change-the-parameter-list-of-piix4_transaction-function/20240913-201353
base:   https://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host
patch link:    https://lore.kernel.org/r/20240913121110.1611340-5-Shyam-sundar.S-k%40amd.com
patch subject: [PATCH v5 4/8] i2c: amd-asf: Add ACPI support for AMD ASF Controller
config: arm-randconfig-003-20240919 (https://download.01.org/0day-ci/archive/20240919/202409192333.P0Nve307-lkp@intel.com/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 8663a75fa2f31299ab8d1d90288d9df92aadee88)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240919/202409192333.P0Nve307-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/202409192333.P0Nve307-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from drivers/i2c/busses/i2c-amd-asf-plat.c:15:
   In file included from include/linux/i2c.h:19:
   In file included from include/linux/regulator/consumer.h:35:
   In file included from include/linux/suspend.h:5:
   In file included from include/linux/swap.h:9:
   In file included from include/linux/memcontrol.h:21:
   In file included from include/linux/mm.h:2232:
   include/linux/vmstat.h:517:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
     517 |         return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
         |                               ~~~~~~~~~~~ ^ ~~~
>> drivers/i2c/busses/i2c-amd-asf-plat.c:52:56: warning: format specifies type 'unsigned long long' but the argument has type 'resource_size_t' (aka 'unsigned int') [-Wformat]
      52 |                  "SMBus ASF adapter%s at 0x%llx", amd_asf_port_name, asf_dev->port_addr->start);
         |                                            ~~~~                      ^~~~~~~~~~~~~~~~~~~~~~~~~
         |                                            %x
   2 warnings generated.


vim +52 drivers/i2c/busses/i2c-amd-asf-plat.c

    30	
    31	static int amd_asf_probe(struct platform_device *pdev)
    32	{
    33		struct amd_asf_dev *asf_dev;
    34	
    35		asf_dev = devm_kzalloc(&pdev->dev, sizeof(*asf_dev), GFP_KERNEL);
    36		if (!asf_dev)
    37			return dev_err_probe(&pdev->dev, -ENOMEM, "Failed to allocate memory\n");
    38	
    39		asf_dev->dev = &pdev->dev;
    40		asf_dev->mmio_cfg.use_mmio = true;
    41		platform_set_drvdata(pdev, asf_dev);
    42	
    43		asf_dev->port_addr = platform_get_resource(pdev, IORESOURCE_IO, 0);
    44		if (!asf_dev->port_addr)
    45			return dev_err_probe(&pdev->dev, -EINVAL, "missing IO resources\n");
    46	
    47		asf_dev->adap.owner = THIS_MODULE;
    48		asf_dev->adap.dev.parent = &pdev->dev;
    49	
    50		i2c_set_adapdata(&asf_dev->adap, asf_dev);
    51		snprintf(asf_dev->adap.name, sizeof(asf_dev->adap.name),
  > 52			 "SMBus ASF adapter%s at 0x%llx", amd_asf_port_name, asf_dev->port_addr->start);
    53	
    54		return devm_i2c_add_adapter(&pdev->dev, &asf_dev->adap);
    55	}
    56	

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