[stefandoesinger-zx297520:dlink 131/142] drivers/misc/cardreader/infineon_usim.c:697:61: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long'

[stefandoesinger-zx297520:dlink 131/142] drivers/misc/cardreader/infineon_usim.c:697:61: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long'

[kernel test robot]

tree:   https://gitlab.com/stefandoesinger/zx297520-kernel dlink
head:   7998b6e929bb20ed66acafa33720d8e3533e1598
commit: e70fe7818c4a351f59b891ad4189e1454ea4ea2e [131/142] Sloppy APDU
config: sparc64-allmodconfig (https://download.01.org/0day-ci/archive/20261008/202610080635.Tej6yJe8-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261008/202610080635.Tej6yJe8-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/202610080635.Tej6yJe8-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/misc/cardreader/infineon_usim.c:191:14: warning: unused variable 'i' [-Wunused-variable]
     191 |         size_t len, i;
         |                     ^
>> drivers/misc/cardreader/infineon_usim.c:697:61: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long' [-Wformat]
     697 |                 dev_err(usim->dev, "ioctl 0x%08x, powerup %x\n", command, INFINEON_USIM_IOC_POWER_UP);
         |                                                           ~~              ^~~~~~~~~~~~~~~~~~~~~~~~~~
         |                                                           %lx
   include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                                ~~~     ^~~~~~~~~~~
   include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ~~~    ^~~~~~~~~~~
   include/uapi/linux/infineon_usim.h:39:36: note: expanded from macro 'INFINEON_USIM_IOC_POWER_UP'
      39 | #define INFINEON_USIM_IOC_POWER_UP _IOWR(INFINEON_USIM_IOC_MAGIC, 0x0, struct infineon_usim_power_up)
         |                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/sparc/include/uapi/asm/ioctl.h:48:29: note: expanded from macro '_IOWR'
      48 | #define _IOWR(type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),sizeof(size))
         |                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/sparc/include/uapi/asm/ioctl.h:40:9: note: expanded from macro '_IOC'
      40 |         (((dir)  << _IOC_DIRSHIFT) | \
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      41 |          ((type) << _IOC_TYPESHIFT) | \
         |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      42 |          ((nr)   << _IOC_NRSHIFT) | \
         |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      43 |          ((size) << _IOC_SIZESHIFT))
         |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/misc/cardreader/infineon_usim.c:716:13: error: incompatible function pointer types initializing 'int (*)(struct inode *, struct file *)' with an expression of type 'ssize_t (struct inode *, struct file *)' (aka 'long (struct inode *, struct file *)') [-Wincompatible-function-pointer-types]
     716 |         .release = usim_release,
         |                    ^~~~~~~~~~~~
   2 warnings and 1 error generated.


vim +697 drivers/misc/cardreader/infineon_usim.c

   669	
   670	static long usim_ioctl(struct file *file, unsigned int command, unsigned long argument)
   671	{
   672		struct usim_drv *usim = file->private_data;
   673		void __user *argp = (void __user *)argument;
   674		long ret;
   675	
   676		dev_dbg(usim->dev, "USIM IOCTL 0x%x\n", command);
   677		mutex_lock(&usim->lock);
   678	
   679		switch (command) {
   680		case INFINEON_USIM_IOC_POWER_UP:
   681			ret = usim_power_up(usim, argp);
   682			break;
   683	
   684		case INFINEON_USIM_IOC_POWER_DOWN:
   685			ret = usim_power_down(usim);
   686			break;
   687	
   688		case INFINEON_USIM_IOC_RESET:
   689			ret = usim_reset(usim, argp);
   690			break;
   691	
   692		case INFINEON_USIM_IOC_TRANSMIT:
   693			ret = usim_transmit(usim, argp);
   694			break;
   695	
   696		default:
 > 697			dev_err(usim->dev, "ioctl 0x%08x, powerup %x\n", command, INFINEON_USIM_IOC_POWER_UP);
   698			ret = -ENOTTY;
   699			break;
   700		}
   701	
   702		mutex_unlock(&usim->lock);
   703		dev_dbg(usim->dev, "ioctl 0x%x ret %ld\n", command, ret);
   704	
   705		return ret;
   706	}
   707	

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