Patchwork [U-Boot] am33xx: Fix warning with CONFIG_DISPLAY_CPUINFO

login
register
mail settings
Submitter Tom Rini
Date April 25, 2013, 8:46 p.m.
Message ID <1366922816-22182-1-git-send-email-trini@ti.com>
Download mbox | patch
Permalink /patch/239596/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Tom Rini - April 25, 2013, 8:46 p.m.
The arm_freq and ddr_freq variables are unused, so remove.  Fixup
whitespace slightly while in here.

Signed-off-by: Tom Rini <trini@ti.com>
---
 arch/arm/cpu/armv7/am33xx/sys_info.c |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)
Peter Korsgaard - April 26, 2013, 9:42 p.m.
>>>>> "Tom" == Tom Rini <trini@ti.com> writes:

 Tom> The arm_freq and ddr_freq variables are unused, so remove.  Fixup
 Tom> whitespace slightly while in here.

 Tom> Signed-off-by: Tom Rini <trini@ti.com>

Reviewed-by: Peter Korsgaard <jacmet@sunsite.dk>
Tom Rini - April 29, 2013, 8:21 p.m.
On Fri, Apr 26, 2013 at 11:42:12PM +0200, Peter Korsgaard wrote:
> >>>>> "Tom" == Tom Rini <trini@ti.com> writes:
> 
>  Tom> The arm_freq and ddr_freq variables are unused, so remove.  Fixup
>  Tom> whitespace slightly while in here.
> 
>  Tom> Signed-off-by: Tom Rini <trini@ti.com>
> 
> Reviewed-by: Peter Korsgaard <jacmet@sunsite.dk>

Applied to u-boot-ti/master, thanks!

Patch

diff --git a/arch/arm/cpu/armv7/am33xx/sys_info.c b/arch/arm/cpu/armv7/am33xx/sys_info.c
index 5fd8b47..ac049ac 100644
--- a/arch/arm/cpu/armv7/am33xx/sys_info.c
+++ b/arch/arm/cpu/armv7/am33xx/sys_info.c
@@ -92,7 +92,6 @@  u32 get_sysboot_value(void)
 int print_cpuinfo(void)
 {
 	char *cpu_s, *sec_s;
-	int arm_freq, ddr_freq;
 
 	switch (get_cpu_type()) {
 	case AM335X:
@@ -123,10 +122,7 @@  int print_cpuinfo(void)
 		sec_s = "?";
 	}
 
-	printf("%s-%s rev %d\n",
-			cpu_s, sec_s, get_cpu_rev());
-
-	/* TODO: Print ARM and DDR frequencies  */
+	printf("%s-%s rev %d\n", cpu_s, sec_s, get_cpu_rev());
 
 	return 0;
 }