Re: [U-Boot] [PATCH] arm: omap3: Update cpuinfo for DM3730, DM3725, AM3715, and AM3703

2017-01-20 Thread Adam Ford
On Fri, Jan 20, 2017 at 4:32 AM, Ladislav Michl  wrote:
> On Thu, Jan 19, 2017 at 02:04:11PM -0600, Adam Ford wrote:
>> The check for OMAP3630/3730 only checks for 800MHz 3630/3730, but
>> anything else is lumped into 36XX/37XX with an assumed 1GHz speed.
>>
>> Based on the DM3730 TRM bit 9 shows the MPU Frequency (800MHz/1GHZ).
>> This also adds the ability to distinguish between the DM3730, DM3725,
>> AM3715, and AM3703 and correctly display their maximum speed.
>
> Prints nicely:
> AM3703-GP ES1.2, CPU-OPP2, L3-200MHz, Max CPU Clock 1 GHz
>
> Tested-by: Ladislav Michl 
>
> Btw,
>
> I would be nice to use correct spelling of ISO units, so I propose what's
> attached (you patch rebased ot top of spelling change).
> Not sure if you wish to take extra effort to resend, though.
>

I was originally going to do a separate commit to fix the spelling of
MHz and GHz once the first patch was approved, because I didn't want
to combine a typo fix with a feature addition.  Since you've done the
work, why not just submit your patch and take credit for it. :-)

adam
> Best regards,
> ladis
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] arm: omap3: Update cpuinfo for DM3730, DM3725, AM3715, and AM3703

2017-01-19 Thread Adam Ford
The check for OMAP3630/3730 only checks for 800MHz 3630/3730, but
anything else is lumped into 36XX/37XX with an assumed 1GHz speed.

Based on the DM3730 TRM bit 9 shows the MPU Frequency (800MHz/1GHZ).
This also adds the ability to distinguish between the DM3730, DM3725,
AM3715, and AM3703 and correctly display their maximum speed.

Signed-off-by: Adam Ford 

diff --git a/arch/arm/include/asm/arch-omap3/omap.h 
b/arch/arm/include/asm/arch-omap3/omap.h
index 417ff89..a70a426 100644
--- a/arch/arm/include/asm/arch-omap3/omap.h
+++ b/arch/arm/include/asm/arch-omap3/omap.h
@@ -230,6 +230,14 @@ struct gpio {
 #define AM3517 0x1c00
 
 #define OMAP3730   0x0c00
+#define OMAP3725   0x4c00
+#define AM3715 0x1c00
+#define AM3703 0x5c00
+
+#define OMAP3730_1GHZ  0x0e00
+#define OMAP3725_1GHZ  0x4e00
+#define AM3715_1GHZ0x1e00
+#define AM3703_1GHZ0x5e00
 
 /*
  * ROM code API related flags
diff --git a/arch/arm/mach-omap2/omap3/sys_info.c 
b/arch/arm/mach-omap2/omap3/sys_info.c
index 6818fab..99c9120 100644
--- a/arch/arm/mach-omap2/omap3/sys_info.c
+++ b/arch/arm/mach-omap2/omap3/sys_info.c
@@ -295,16 +295,54 @@ int print_cpuinfo (void)
max_clk = "600 Mhz";
break;
case CPU_OMAP36XX:
-   cpu_family_s = "OMAP";
switch (get_cpu_type()) {
+   case AM3703:
+   cpu_family_s = "AM";
+   cpu_s = "3703";
+   max_clk = "800 MHz";
+   break;
+   case AM3703_1GHZ:
+   cpu_family_s = "AM";
+   cpu_s = "3703";
+   max_clk = "1 GHz";
+   break;
+   case AM3715:
+   cpu_family_s = "AM";
+   cpu_s = "3715";
+   max_clk = "800 MHz";
+   break;
+   case AM3715_1GHZ:
+   cpu_family_s = "AM";
+   cpu_s = "3715";
+   max_clk = "1 GHz";
+   break;
+   case OMAP3725:
+   cpu_family_s = "OMAP";
+   cpu_s = "3625/3725";
+   max_clk = "800 MHz";
+   break;
+   case OMAP3725_1GHZ:
+   cpu_family_s = "OMAP";
+   cpu_s = "3625/3725";
+   max_clk = "1 GHz";
+   break;
case OMAP3730:
+   cpu_family_s = "OMAP";
cpu_s = "3630/3730";
+   max_clk = "800 MHz";
+   break;
+   case OMAP3730_1GHZ:
+   cpu_family_s = "OMAP";
+   cpu_s = "3630/3730";
+   max_clk = "1 GHz";
break;
default:
+   cpu_family_s = "OMAP/AM";
cpu_s = "36XX/37XX";
+   max_clk = "1 GHz";
break;
}
-   max_clk = "1 Ghz";
+
break;
default:
cpu_family_s = "OMAP";
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot