Re: linux-next: build warning after merge of the powerpc tree
From: Michael Ellerman
Date: Fri Apr 16 2021 - 06:54:33 EST
Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> writes:
> Hi all,
>
> After merging the powerpc tree, today's linux-next build (powerpc
> allyesconfig) produced this warning:
>
> In file included from include/linux/device.h:15,
> from arch/powerpc/include/asm/io.h:27,
> from include/linux/io.h:13,
> from include/linux/irq.h:20,
> from arch/powerpc/include/asm/hardirq.h:6,
> from include/linux/hardirq.h:11,
> from include/linux/highmem.h:10,
> from include/linux/bio.h:8,
> from include/linux/libnvdimm.h:14,
> from arch/powerpc/platforms/pseries/papr_scm.c:12:
> arch/powerpc/platforms/pseries/papr_scm.c: In function 'papr_scm_pmem_flush':
> arch/powerpc/platforms/pseries/papr_scm.c:144:26: warning: format '%lld' expects argument of type 'long long int', but argument 3 has type 'long int' [-Wformat=]
> 144 | dev_err(&p->pdev->dev, "flush error: %lld", rc);
> | ^~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
> 19 | #define dev_fmt(fmt) fmt
> | ^~~
> arch/powerpc/platforms/pseries/papr_scm.c:144:3: note: in expansion of macro 'dev_err'
> 144 | dev_err(&p->pdev->dev, "flush error: %lld", rc);
> | ^~~~~~~
> arch/powerpc/platforms/pseries/papr_scm.c:144:43: note: format string is defined here
> 144 | dev_err(&p->pdev->dev, "flush error: %lld", rc);
> | ~~~^
> | |
> | long long int
> | %ld
>
> Introduced by commit
>
> 75b7c05ebf90 ("powerpc/papr_scm: Implement support for H_SCM_FLUSH hcall")
My bad.
cheers