mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] powerpc/mpc52xx: Remove redundant 0x prefix in mpc52xx_pci_setup()
@ 2026-11-04 17:28 Thorsten Blum
  2026-11-04 17:42 ` Randy Dunlap
  0 siblings, 1 reply; 2+ messages in thread
From: Thorsten Blum @ 2026-11-04 17:28 UTC (permalink / raw)
  To: Anatolij Gustschin, Madhavan Srinivasan, Michael Ellerman,
	Nicholas Piggin, Christophe Leroy (CS GROUP),
	Ritesh Harjani (IBM),
	Shrikanth Hegde, Randy Dunlap
  Cc: Thorsten Blum, linuxppc-dev, linux-kernel

The %pa format specifier already includes the 0x prefix. Remove the
redundant prefix to avoid printing 0x0x.

Fixes: 59510820fff7 ("powerpc/mpc52xx: Fix some pr_debug() issues")
Signed-off-by: Thorsten Blum <blum@kernel.org>
---
 arch/powerpc/platforms/52xx/mpc52xx_pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/52xx/mpc52xx_pci.c b/arch/powerpc/platforms/52xx/mpc52xx_pci.c
index 0ca4401ba781..a2e53cacd15b 100644
--- a/arch/powerpc/platforms/52xx/mpc52xx_pci.c
+++ b/arch/powerpc/platforms/52xx/mpc52xx_pci.c
@@ -288,7 +288,7 @@ mpc52xx_pci_setup(struct pci_controller *hose,
 		printk(KERN_ERR "%s: Didn't find IO resources\n", __FILE__);
 		return;
 	}
-	pr_debug(".io_resource = %pr .io_base_phys=0x%pa\n",
+	pr_debug(".io_resource = %pr .io_base_phys=%pa\n",
 		 res, &hose->io_base_phys);
 	out_be32(&pci_regs->iw2btar,
 	         MPC52xx_PCI_IWBTAR_TRANSLATION(hose->io_base_phys,

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] powerpc/mpc52xx: Remove redundant 0x prefix in mpc52xx_pci_setup()
  2026-11-04 17:28 [PATCH] powerpc/mpc52xx: Remove redundant 0x prefix in mpc52xx_pci_setup() Thorsten Blum
@ 2026-11-04 17:42 ` Randy Dunlap
  0 siblings, 0 replies; 2+ messages in thread
From: Randy Dunlap @ 2026-11-04 17:42 UTC (permalink / raw)
  To: Thorsten Blum, Anatolij Gustschin, Madhavan Srinivasan,
	Michael Ellerman, Nicholas Piggin, Christophe Leroy (CS GROUP),
	Ritesh Harjani (IBM),
	Shrikanth Hegde
  Cc: linuxppc-dev, linux-kernel



On 11/4/26 9:28 AM, Thorsten Blum wrote:
> The %pa format specifier already includes the 0x prefix. Remove the
> redundant prefix to avoid printing 0x0x.
> 
> Fixes: 59510820fff7 ("powerpc/mpc52xx: Fix some pr_debug() issues")
> Signed-off-by: Thorsten Blum <blum@kernel.org>

Acked-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.

> ---
>  arch/powerpc/platforms/52xx/mpc52xx_pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/powerpc/platforms/52xx/mpc52xx_pci.c b/arch/powerpc/platforms/52xx/mpc52xx_pci.c
> index 0ca4401ba781..a2e53cacd15b 100644
> --- a/arch/powerpc/platforms/52xx/mpc52xx_pci.c
> +++ b/arch/powerpc/platforms/52xx/mpc52xx_pci.c
> @@ -288,7 +288,7 @@ mpc52xx_pci_setup(struct pci_controller *hose,
>  		printk(KERN_ERR "%s: Didn't find IO resources\n", __FILE__);
>  		return;
>  	}
> -	pr_debug(".io_resource = %pr .io_base_phys=0x%pa\n",
> +	pr_debug(".io_resource = %pr .io_base_phys=%pa\n",
>  		 res, &hose->io_base_phys);
>  	out_be32(&pci_regs->iw2btar,
>  	         MPC52xx_PCI_IWBTAR_TRANSLATION(hose->io_base_phys,

-- 
~Randy

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-11-04 17:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-04 17:28 [PATCH] powerpc/mpc52xx: Remove redundant 0x prefix in mpc52xx_pci_setup() Thorsten Blum
2026-11-04 17:42 ` Randy Dunlap

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®