Re: [mtd:mtd/next 10/11] drivers/mtd/mtdoops.c:244:39: warning: format specifies type 'long' but the argument has type 'unsigned int'

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Jean-Marc,

jmeurin@google.com wrote on Thu, 21 Apr 2022 15:22:26 -0700:

> On Thu, Apr 21, 2022 at 6:48 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> >
> > Hi Jean-Marc,
> >
> > lkp@intel.com wrote on Thu, 21 Apr 2022 21:22:47 +0800:
> >  
> > > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next
> > > head:   f812679fab605b3d5b853ed24a81dabb222ea23a
> > > commit: 0629fcef16d703d384f76cb2c2c3a119a9149a34 [10/11] mtd: mtdoops: Create a header structure for the saved mtdoops.
> > > config: hexagon-randconfig-r045-20220420 (https://download.01.org/0day-ci/archive/20220421/202204212106.9R1ylUB7-lkp@intel.com/config)
> > > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project bac6cd5bf85669e3376610cfc4c4f9ca015e7b9b)
> > > 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
> > >         # https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git/commit/?id=0629fcef16d703d384f76cb2c2c3a119a9149a34
> > >         git remote add mtd https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
> > >         git fetch --no-tags mtd mtd/next
> > >         git checkout 0629fcef16d703d384f76cb2c2c3a119a9149a34
> > >         # save the config file
> > >         mkdir build_dir && cp config build_dir/.config
> > >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/mtd/
> > >
> > > If you fix the issue, kindly add following tag as appropriate
> > > Reported-by: kernel test robot <lkp@intel.com>
> > >
> > > All warnings (new ones prefixed by >>):
> > >  
> > > >> drivers/mtd/mtdoops.c:244:39: warning: format specifies type 'long' but the argument has type 'unsigned int' [-Wformat]  
> > >                                   page * record_size, retlen, sizeof(hdr), ret);
> > >                                                               ^~~~~~~~~~~
> > >    include/linux/printk.h:446:60: note: expanded from macro 'printk'
> > >    #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
> > >                                                        ~~~    ^~~~~~~~~~~
> > >    include/linux/printk.h:418:19: note: expanded from macro 'printk_index_wrap'
> > >                    _p_func(_fmt, ##__VA_ARGS__);                           \
> > >                            ~~~~    ^~~~~~~~~~~
> > >    1 warning generated.  
> >
> > I've dropped the series for now. Please have a look at the reports and
> > propose a new version when this is fixed?  
> 
> Sorry about that, I had made that change based on this suggestion
> https://lore.kernel.org/all/202203310648.it4f2xXD-lkp@intel.com/ :-)

Did you verify with

> > >         # save the config file
> > >         mkdir build_dir && cp config build_dir/.config
> > >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/mtd/

That the errors were actually gone? You seem to suggest that you didn't
:)

Thanks,
Miquèl