* [U-Boot] [PATCH] powerpc: Use lower case for the core names
@ 2013-04-21 16:11 Fabio Estevam
2013-06-21 20:43 ` [U-Boot] " Andy Fleming
0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2013-04-21 16:11 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
Freescale documentation presents the PowerPC core names in lower case, such as
"e300", "e500", "e600", etc.
Change the upper case occurrences into lower case so that the core names
reported in U-boot can match the ones from the documentation.
While at it also fix a checkpatch error:
ERROR: space prohibited before that close parenthesis ')'
#53: FILE: arch/powerpc/cpu/mpc86xx/cpu.c:81:
+ printf("e600 Core %d", (msscr0 & 0x20) ? 1 : 0 );
Reported-by: Heinz Wrobel <heinz.wrobel@freescale.com>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
arch/powerpc/cpu/mpc85xx/cpu.c | 8 ++++----
arch/powerpc/cpu/mpc86xx/cpu.c | 2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
index df2ab6d..e88ee26 100644
--- a/arch/powerpc/cpu/mpc85xx/cpu.c
+++ b/arch/powerpc/cpu/mpc85xx/cpu.c
@@ -121,16 +121,16 @@ int checkcpu (void)
switch(ver) {
case PVR_VER_E500_V1:
case PVR_VER_E500_V2:
- puts("E500");
+ puts("e500");
break;
case PVR_VER_E500MC:
- puts("E500MC");
+ puts("e500mc");
break;
case PVR_VER_E5500:
- puts("E5500");
+ puts("e5500");
break;
case PVR_VER_E6500:
- puts("E6500");
+ puts("e6500");
break;
default:
puts("Unknown");
diff --git a/arch/powerpc/cpu/mpc86xx/cpu.c b/arch/powerpc/cpu/mpc86xx/cpu.c
index c553415..b8b1d8f 100644
--- a/arch/powerpc/cpu/mpc86xx/cpu.c
+++ b/arch/powerpc/cpu/mpc86xx/cpu.c
@@ -78,7 +78,7 @@ checkcpu(void)
major = PVR_E600_MAJ(pvr);
minor = PVR_E600_MIN(pvr);
- printf("E600 Core %d", (msscr0 & 0x20) ? 1 : 0 );
+ printf("e600 Core %d", (msscr0 & 0x20) ? 1 : 0);
if (gur->pordevsr & MPC86xx_PORDEVSR_CORE1TE)
puts("\n Core1Translation Enabled");
debug(" (MSSCR0=%x, PORDEVSR=%x)", msscr0, gur->pordevsr);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [U-Boot] powerpc: Use lower case for the core names
2013-04-21 16:11 [U-Boot] [PATCH] powerpc: Use lower case for the core names Fabio Estevam
@ 2013-06-21 20:43 ` Andy Fleming
0 siblings, 0 replies; 2+ messages in thread
From: Andy Fleming @ 2013-06-21 20:43 UTC (permalink / raw)
To: u-boot
On Sun, Apr 21, 2013 at 01:11:02PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Freescale documentation presents the PowerPC core names in lower case, such as
> "e300", "e500", "e600", etc.
>
> Change the upper case occurrences into lower case so that the core names
> reported in U-boot can match the ones from the documentation.
>
> While at it also fix a checkpatch error:
>
> ERROR: space prohibited before that close parenthesis ')'
> #53: FILE: arch/powerpc/cpu/mpc86xx/cpu.c:81:
> + printf("e600 Core %d", (msscr0 & 0x20) ? 1 : 0 );
>
> Reported-by: Heinz Wrobel <heinz.wrobel@freescale.com>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Applied, thanks!
Andy
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-21 20:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-21 16:11 [U-Boot] [PATCH] powerpc: Use lower case for the core names Fabio Estevam
2013-06-21 20:43 ` [U-Boot] " Andy Fleming
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.