Re: [PATCH] gpio: mpsse: support bryx radio interface kit

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

Hi Mary,

kernel test robot noticed the following build warnings:

[auto build test WARNING on brgl/gpio/for-next]
[also build test WARNING on linus/master v6.17-rc5 next-20250909]
[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/Mary-Strodl/gpio-mpsse-support-bryx-radio-interface-kit/20250909-014744
base:   https://git.kernel.org/pub/scm/linux/kernel/git/brgl/linux.git gpio/for-next
patch link:    https://lore.kernel.org/r/20250908173804.3816149-1-mstrodl%40csh.rit.edu
patch subject: [PATCH] gpio: mpsse: support bryx radio interface kit
config: m68k-allmodconfig (https://download.01.org/0day-ci/archive/20250909/202509092305.ncd9mzaZ-lkp@intel.com/config)
compiler: m68k-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250909/202509092305.ncd9mzaZ-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/202509092305.ncd9mzaZ-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/device.h:15,
                    from include/linux/usb.h:19,
                    from drivers/gpio/gpio-mpsse.c:13:
   drivers/gpio/gpio-mpsse.c: In function 'mpsse_ensure_supported':
>> drivers/gpio/gpio-mpsse.c:211:25: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'int' [-Wformat=]
     211 |                         "mpsse: GPIO %ld doesn't support %s\n",
         |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/gpio/gpio-mpsse.c:210:17: note: in expansion of macro 'dev_err'
     210 |                 dev_err(&priv->udev->dev,
         |                 ^~~~~~~
   drivers/gpio/gpio-mpsse.c:211:40: note: format string is defined here
     211 |                         "mpsse: GPIO %ld doesn't support %s\n",
         |                                      ~~^
         |                                        |
         |                                        long int
         |                                      %d


vim +211 drivers/gpio/gpio-mpsse.c

   193	
   194	static int mpsse_ensure_supported(struct gpio_chip *chip,
   195					  unsigned long *mask, int direction)
   196	{
   197		unsigned long supported, unsupported;
   198		char *type = "input";
   199		struct mpsse_priv *priv = gpiochip_get_data(chip);
   200	
   201		supported = priv->dir_in;
   202		if (direction == GPIO_LINE_DIRECTION_OUT) {
   203			supported = priv->dir_out;
   204			type = "output";
   205		}
   206	
   207		/* An invalid bit was in the provided mask */
   208		unsupported = *mask & supported;
   209		if (unsupported) {
   210			dev_err(&priv->udev->dev,
 > 211				"mpsse: GPIO %ld doesn't support %s\n",
   212				find_first_bit(&unsupported, sizeof(unsupported) * 8),
   213				type);
   214			return -EOPNOTSUPP;
   215		}
   216	
   217		return 0;
   218	}
   219	

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