From: Scott Wood Date: Thu, 12 Mar 2015 03:13:46 +0000 (-0500) Subject: powerpc/32: %pF is only for function pointers X-Git-Tag: v5.15~15963^2~75 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cc83458d3aa01d69ccb2976e1323c92a6caae540;p=platform%2Fkernel%2Flinux-starfive.git powerpc/32: %pF is only for function pointers Use %pS for actual addresses, otherwise you'll get bad output on arches like ppc64 where %pF expects a function descriptor. Even on other architectures, refrain from setting a bad example that people copy. Signed-off-by: Scott Wood Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Michael Ellerman --- diff --git a/arch/powerpc/mm/pgtable_32.c b/arch/powerpc/mm/pgtable_32.c index 03b1a3b..24f304a 100644 --- a/arch/powerpc/mm/pgtable_32.c +++ b/arch/powerpc/mm/pgtable_32.c @@ -221,7 +221,7 @@ __ioremap_caller(phys_addr_t addr, unsigned long size, unsigned long flags, */ if (mem_init_done && (p < virt_to_phys(high_memory)) && !(__allow_ioremap_reserved && memblock_is_region_reserved(p, size))) { - printk("__ioremap(): phys addr 0x%llx is RAM lr %pf\n", + printk("__ioremap(): phys addr 0x%llx is RAM lr %ps\n", (unsigned long long)p, __builtin_return_address(0)); return NULL; }