powerpc/wii: fix resource printk format warnings
authorRandy Dunlap <rdunlap@infradead.org>
Thu, 23 Feb 2023 07:01:14 +0000 (23:01 -0800)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 30 Mar 2023 12:36:30 +0000 (23:36 +1100)
Use "%pa" format specifier for resource_size_t to avoid compiler
printk format warnings.

../arch/powerpc/platforms/embedded6xx/flipper-pic.c: In function 'flipper_pic_init':
../include/linux/kern_levels.h:5:25: error: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'resource_size_t' {aka 'long long unsigned int'} [-Werror=format=]
../arch/powerpc/platforms/embedded6xx/flipper-pic.c:148:9: note: in expansion of macro 'pr_info'
  148 |         pr_info("controller at 0x%08x mapped to 0x%p\n", res.start, io_base);
      |         ^~~~~~~

../arch/powerpc/platforms/embedded6xx/hlwd-pic.c: In function 'hlwd_pic_init':
../include/linux/kern_levels.h:5:25: error: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'resource_size_t' {aka 'long long unsigned int'} [-Werror=format=]
../arch/powerpc/platforms/embedded6xx/hlwd-pic.c:174:9: note: in expansion of macro 'pr_info'
  174 |         pr_info("controller at 0x%08x mapped to 0x%p\n", res.start, io_base);
      |         ^~~~~~~

../arch/powerpc/platforms/embedded6xx/wii.c: In function 'wii_ioremap_hw_regs':
../include/linux/kern_levels.h:5:25: error: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'resource_size_t' {aka 'long long unsigned int'} [-Werror=format=]
../arch/powerpc/platforms/embedded6xx/wii.c:77:17: note: in expansion of macro 'pr_info'
   77 |                 pr_info("%s at 0x%08x mapped to 0x%p\n", name,
      |                 ^~~~~~~

Fixes: 028ee972f032 ("powerpc: gamecube/wii: flipper interrupt controller support")
Fixes: 9c21025c7845 ("powerpc: wii: hollywood interrupt controller support")
Fixes: 5a7ee3198dfa ("powerpc: wii: platform support")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20230223070116.660-3-rdunlap@infradead.org
arch/powerpc/platforms/embedded6xx/flipper-pic.c
arch/powerpc/platforms/embedded6xx/hlwd-pic.c
arch/powerpc/platforms/embedded6xx/wii.c

index 609bda2ad5dd2d3ed692d349855845af21eb0e30..4d9200bdba78c3277217238846e98b490228a173 100644 (file)
@@ -145,7 +145,7 @@ static struct irq_domain * __init flipper_pic_init(struct device_node *np)
        }
        io_base = ioremap(res.start, resource_size(&res));
 
-       pr_info("controller at 0x%08x mapped to 0x%p\n", res.start, io_base);
+       pr_info("controller at 0x%pa mapped to 0x%p\n", &res.start, io_base);
 
        __flipper_quiesce(io_base);
 
index 380b4285cce47f152fa52d5bbe615fcea9867bad..4d2d92de30afd643b3144523f4de62d95093a78f 100644 (file)
@@ -171,7 +171,7 @@ static struct irq_domain *__init hlwd_pic_init(struct device_node *np)
                return NULL;
        }
 
-       pr_info("controller at 0x%08x mapped to 0x%p\n", res.start, io_base);
+       pr_info("controller at 0x%pa mapped to 0x%p\n", &res.start, io_base);
 
        __hlwd_quiesce(io_base);
 
index b54382a8ccc6561284410357b33d71817bf1f54d..cb3be6d6e3398e7873488d4beff6b77ab236f836 100644 (file)
@@ -74,8 +74,8 @@ static void __iomem *__init wii_ioremap_hw_regs(char *name, char *compatible)
 
        hw_regs = ioremap(res.start, resource_size(&res));
        if (hw_regs) {
-               pr_info("%s at 0x%08x mapped to 0x%p\n", name,
-                       res.start, hw_regs);
+               pr_info("%s at 0x%pa mapped to 0x%p\n", name,
+                       &res.start, hw_regs);
        }
 
 out_put: