Patchwork [U-Boot] ARM: AM33XX: Fix typo that causes an AM duplication in CPU name.

login
register
mail settings
Submitter Enric Balletbò i Serra
Date March 15, 2013, 11:35 a.m.
Message ID <1363347337-32384-1-git-send-email-eballetbo@gmail.com>
Download mbox | patch
Permalink /patch/227958/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Enric Balletbò i Serra - March 15, 2013, 11:35 a.m.
From: Enric Balletbo i Serra <eballetbo@iseebcn.com>

Just fix a typo displaying the CPU info. With CONFIG_DISPLAY_INFO we see
something like AMAM335X-GP rev 0 instead of AM335X-GP rev 0.

Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
---
 arch/arm/cpu/armv7/am33xx/sys_info.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Javier Martinez Canillas - March 15, 2013, 11:51 a.m.
On Fri, Mar 15, 2013 at 12:35 PM, Enric Balletbo i Serra
<eballetbo@gmail.com> wrote:
> From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
>
> Just fix a typo displaying the CPU info. With CONFIG_DISPLAY_INFO we see
> something like AMAM335X-GP rev 0 instead of AM335X-GP rev 0.
>
> Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> ---
>  arch/arm/cpu/armv7/am33xx/sys_info.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/cpu/armv7/am33xx/sys_info.c b/arch/arm/cpu/armv7/am33xx/sys_info.c
> index 507b618..db99e95 100644
> --- a/arch/arm/cpu/armv7/am33xx/sys_info.c
> +++ b/arch/arm/cpu/armv7/am33xx/sys_info.c
> @@ -120,7 +120,7 @@ int print_cpuinfo(void)
>                 sec_s = "?";
>         }
>
> -       printf("AM%s-%s rev %d\n",
> +       printf("%s-%s rev %d\n",
>                         cpu_s, sec_s, get_cpu_rev());
>
>         /* TODO: Print ARM and DDR frequencies  */
> --

Hi Enric,

Looks good to me

Reviewed-by: Javier Martinez Canillas <javier@dowhile0.org>
Tom Rini - March 26, 2013, 2:53 p.m.
On Fri, Mar 15, 2013 at 01:35:37AM -0000, Enric Balletbo i Serra wrote:

> From: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> 
> Just fix a typo displaying the CPU info. With CONFIG_DISPLAY_INFO we see
> something like AMAM335X-GP rev 0 instead of AM335X-GP rev 0.
> 
> Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
> Reviewed-by: Javier Martinez Canillas <javier@dowhile0.org>

Applied to u-boot-ti/master (and already pulled into u-boot-arm),
thanks!

Patch

diff --git a/arch/arm/cpu/armv7/am33xx/sys_info.c b/arch/arm/cpu/armv7/am33xx/sys_info.c
index 507b618..db99e95 100644
--- a/arch/arm/cpu/armv7/am33xx/sys_info.c
+++ b/arch/arm/cpu/armv7/am33xx/sys_info.c
@@ -120,7 +120,7 @@  int print_cpuinfo(void)
 		sec_s = "?";
 	}
 
-	printf("AM%s-%s rev %d\n",
+	printf("%s-%s rev %d\n",
 			cpu_s, sec_s, get_cpu_rev());
 
 	/* TODO: Print ARM and DDR frequencies  */