Message ID | aa6055769b92a5d8685b8d0adab99c48a0b0ef4b.1631956926.git.christophe.leroy@csgroup.eu (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | powerpc/476: Fix sparse report | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 8 jobs. |
snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 8 jobs. |
snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 7 jobs. |
snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 24 jobs. |
snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
Thanks, looks reasonable. Reviewed-by: Alistair Popple <alistair@popple.id.au> On Saturday, 18 September 2021 7:22:32 PM AEST Christophe Leroy wrote: > arch/powerpc/platforms/44x/ppc476.c:236:17: warning: cast removes address space '__iomem' of expression > arch/powerpc/platforms/44x/ppc476.c:241:34: warning: incorrect type in argument 1 (different address spaces) > arch/powerpc/platforms/44x/ppc476.c:241:34: expected void const volatile [noderef] __iomem *addr > arch/powerpc/platforms/44x/ppc476.c:241:34: got unsigned char [usertype] * > arch/powerpc/platforms/44x/ppc476.c:243:17: warning: incorrect type in argument 1 (different address spaces) > arch/powerpc/platforms/44x/ppc476.c:243:17: expected void volatile [noderef] __iomem *addr > arch/powerpc/platforms/44x/ppc476.c:243:17: got unsigned char [usertype] *[assigned] fpga > > Mark 'fpga' pointer as __iomem. > > Reported-by: kernel test robot <lkp@intel.com> > Fixes: ab9a4183fddf ("powerpc: Update currituck pci/usb fixup for new board revision") > Cc: Alistair Popple <alistair@popple.id.au> > Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu> > --- > arch/powerpc/platforms/44x/ppc476.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/platforms/44x/ppc476.c b/arch/powerpc/platforms/44x/ppc476.c > index 07f7e3ce67b5..fb7db5cedd4e 100644 > --- a/arch/powerpc/platforms/44x/ppc476.c > +++ b/arch/powerpc/platforms/44x/ppc476.c > @@ -219,7 +219,7 @@ static int board_rev = -1; > static int __init ppc47x_get_board_rev(void) > { > int reg; > - u8 *fpga; > + u8 __iomem *fpga; > struct device_node *np = NULL; > > if (of_machine_is_compatible("ibm,currituck")) { > @@ -233,7 +233,7 @@ static int __init ppc47x_get_board_rev(void) > if (!np) > goto fail; > > - fpga = (u8 *) of_iomap(np, 0); > + fpga = of_iomap(np, 0); > of_node_put(np); > if (!fpga) > goto fail; >
On Sat, 18 Sep 2021 11:22:32 +0200, Christophe Leroy wrote: > arch/powerpc/platforms/44x/ppc476.c:236:17: warning: cast removes address space '__iomem' of expression > arch/powerpc/platforms/44x/ppc476.c:241:34: warning: incorrect type in argument 1 (different address spaces) > arch/powerpc/platforms/44x/ppc476.c:241:34: expected void const volatile [noderef] __iomem *addr > arch/powerpc/platforms/44x/ppc476.c:241:34: got unsigned char [usertype] * > arch/powerpc/platforms/44x/ppc476.c:243:17: warning: incorrect type in argument 1 (different address spaces) > arch/powerpc/platforms/44x/ppc476.c:243:17: expected void volatile [noderef] __iomem *addr > arch/powerpc/platforms/44x/ppc476.c:243:17: got unsigned char [usertype] *[assigned] fpga > > [...] Applied to powerpc/next. [1/1] powerpc/476: Fix sparse report https://git.kernel.org/powerpc/c/494f238a3861863d908af7b98a369f6d8a986c85 cheers
diff --git a/arch/powerpc/platforms/44x/ppc476.c b/arch/powerpc/platforms/44x/ppc476.c index 07f7e3ce67b5..fb7db5cedd4e 100644 --- a/arch/powerpc/platforms/44x/ppc476.c +++ b/arch/powerpc/platforms/44x/ppc476.c @@ -219,7 +219,7 @@ static int board_rev = -1; static int __init ppc47x_get_board_rev(void) { int reg; - u8 *fpga; + u8 __iomem *fpga; struct device_node *np = NULL; if (of_machine_is_compatible("ibm,currituck")) { @@ -233,7 +233,7 @@ static int __init ppc47x_get_board_rev(void) if (!np) goto fail; - fpga = (u8 *) of_iomap(np, 0); + fpga = of_iomap(np, 0); of_node_put(np); if (!fpga) goto fail;
arch/powerpc/platforms/44x/ppc476.c:236:17: warning: cast removes address space '__iomem' of expression arch/powerpc/platforms/44x/ppc476.c:241:34: warning: incorrect type in argument 1 (different address spaces) arch/powerpc/platforms/44x/ppc476.c:241:34: expected void const volatile [noderef] __iomem *addr arch/powerpc/platforms/44x/ppc476.c:241:34: got unsigned char [usertype] * arch/powerpc/platforms/44x/ppc476.c:243:17: warning: incorrect type in argument 1 (different address spaces) arch/powerpc/platforms/44x/ppc476.c:243:17: expected void volatile [noderef] __iomem *addr arch/powerpc/platforms/44x/ppc476.c:243:17: got unsigned char [usertype] *[assigned] fpga Mark 'fpga' pointer as __iomem. Reported-by: kernel test robot <lkp@intel.com> Fixes: ab9a4183fddf ("powerpc: Update currituck pci/usb fixup for new board revision") Cc: Alistair Popple <alistair@popple.id.au> Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu> --- arch/powerpc/platforms/44x/ppc476.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)