Re: drivers/hid/i2c-hid/i2c-hid-core.c:867:4: warning: format specifies type 'unsigned short' but the argument has type 'int'

[Rong Chen]

On 11/30/21 08:19, Doug Anderson wrote:
> <Beep> <Boop> <Bop> (translates as "Hello Mr. Robot"),
>
> On Thu, Nov 25, 2021 at 10:44 PM kernel test robot <lkp@xxxxxxxxx> wrote:
> > Hi Douglas,
> >
> > First bad commit (maybe != root cause):
> >
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head:   a4849f6000e29235a2707f22e39da6b897bb9543
> > commit: c1ed18c11bdb80eced208a61d40b1988f36a014f HID: i2c-hid: Introduce goodix-i2c-hid using i2c-hid core
> > date:   10 months ago
> > config: arm64-randconfig-r034-20211116 (https://download.01.org/0day-ci/archive/20211126/202111261447.lxHTeAhi-lkp@xxxxxxxxx/config)
> > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project fbe72e41b99dc7994daac300d208a955be3e4a0a)
> > reproduce (this is a W=1 build):
> >          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >          chmod +x ~/bin/make.cross
> >          # install arm64 cross compiling tool for clang build
> >          # apt-get install binutils-aarch64-linux-gnu
> >          # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c1ed18c11bdb80eced208a61d40b1988f36a014f
> >          git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> >          git fetch --no-tags linus master
> >          git checkout c1ed18c11bdb80eced208a61d40b1988f36a014f
> >          # save the config file to linux build tree
> >          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=arm64
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> >
> > All warnings (new ones prefixed by >>):
> >
> > > > drivers/hid/i2c-hid/i2c-hid-core.c:867:4: warning: format specifies type 'unsigned short' but the argument has type 'int' [-Wformat]
> >                             le16_to_cpu(hdesc->bcdVersion));
> >                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >     include/linux/dev_printk.h:112:32: note: expanded from macro 'dev_err'
> >             _dev_err(dev, dev_fmt(fmt), ##__VA_ARGS__)
> >                                   ~~~     ^~~~~~~~~~~
> >     include/linux/byteorder/generic.h:91:21: note: expanded from macro 'le16_to_cpu'
> >     #define le16_to_cpu __le16_to_cpu
> >                         ^
> >     include/uapi/linux/byteorder/big_endian.h:36:26: note: expanded from macro '__le16_to_cpu'
> >     #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
> >                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >     include/uapi/linux/swab.h:105:2: note: expanded from macro '__swab16'
> >             (__builtin_constant_p((__u16)(x)) ?     \
> >             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> May I kindly point to the reply I gave to the same error last month
> when you mailed it out?
>
> https://lore.kernel.org/r/CAD=FV=WkUEkkkfGa+6QJSBvj8EgVrnGYYbd6RrC_5HdTue=mDw@xxxxxxxxxxxxxx/

Hi Doug,

Thanks for the feedback, we have added the warnings to our ignore list.

Best Regards,

Rong Chen