linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: OMAP3: Fix SoC detection of OMAP36/37 Family
@ 2017-01-20 16:44 aford173 at gmail.com
  2017-01-20 18:14 ` Tony Lindgren
  0 siblings, 1 reply; 2+ messages in thread
From: aford173 at gmail.com @ 2017-01-20 16:44 UTC (permalink / raw)
  To: linux-arm-kernel

From: Adam Ford <aford173@gmail.com>

The OMAP36/37 families are similar, but there are a few features sections
that can help identify some of them.  Let's add checks for 3630/3730,
OMAP3621, DM3725, OMAP3615/DM3715, OMAP3611, and AM3703 all based on
features similar to what was done for the OMAP34xx/35xx series

The checkpatch flags some warnings for braces, but I kept the coding style
to match the adjacent code for consistency.

I don't have an OMAP36xx to test, but this was tested on both a DM3730 and
AM3703.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c
index cc6d9fa..e2274a1 100644
--- a/arch/arm/mach-omap2/id.c
+++ b/arch/arm/mach-omap2/id.c
@@ -223,7 +223,15 @@ static void __init omap3_cpuinfo(void)
 	 * and CPU class bits.
 	 */
 	if (soc_is_omap3630()) {
-		cpu_name = "OMAP3630";
+		if (omap3_has_iva() && omap3_has_sgx()) {
+			cpu_name = (omap3_has_isp()) ? "OMAP3630/DM3730" : "OMAP3621";
+		} else if (omap3_has_iva()) {
+			cpu_name = "DM3725";
+		} else if (omap3_has_sgx()) {
+			cpu_name = "OMAP3615/AM3715";
+		} else {
+			cpu_name = (omap3_has_isp()) ? "AM3703" : "OMAP3611";
+		}
 	} else if (soc_is_am35xx()) {
 		cpu_name = (omap3_has_sgx()) ? "AM3517" : "AM3505";
 	} else if (soc_is_ti816x()) {
-- 
2.7.4

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

* [PATCH] ARM: OMAP3: Fix SoC detection of OMAP36/37 Family
  2017-01-20 16:44 [PATCH] ARM: OMAP3: Fix SoC detection of OMAP36/37 Family aford173 at gmail.com
@ 2017-01-20 18:14 ` Tony Lindgren
  0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2017-01-20 18:14 UTC (permalink / raw)
  To: linux-arm-kernel

* aford173 at gmail.com <aford173@gmail.com> [170120 08:45]:
> From: Adam Ford <aford173@gmail.com>
> 
> The OMAP36/37 families are similar, but there are a few features sections
> that can help identify some of them.  Let's add checks for 3630/3730,
> OMAP3621, DM3725, OMAP3615/DM3715, OMAP3611, and AM3703 all based on
> features similar to what was done for the OMAP34xx/35xx series
> 
> The checkpatch flags some warnings for braces, but I kept the coding style
> to match the adjacent code for consistency.
> 
> I don't have an OMAP36xx to test, but this was tested on both a DM3730 and
> AM3703.

AFAIK 3630 is exactly the same as dm3730. Applying into
omap-for-v4.11/fixes-not-urgent thanks.

Regards,

Tony

> Signed-off-by: Adam Ford <aford173@gmail.com>
> 
> diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c
> index cc6d9fa..e2274a1 100644
> --- a/arch/arm/mach-omap2/id.c
> +++ b/arch/arm/mach-omap2/id.c
> @@ -223,7 +223,15 @@ static void __init omap3_cpuinfo(void)
>  	 * and CPU class bits.
>  	 */
>  	if (soc_is_omap3630()) {
> -		cpu_name = "OMAP3630";
> +		if (omap3_has_iva() && omap3_has_sgx()) {
> +			cpu_name = (omap3_has_isp()) ? "OMAP3630/DM3730" : "OMAP3621";
> +		} else if (omap3_has_iva()) {
> +			cpu_name = "DM3725";
> +		} else if (omap3_has_sgx()) {
> +			cpu_name = "OMAP3615/AM3715";
> +		} else {
> +			cpu_name = (omap3_has_isp()) ? "AM3703" : "OMAP3611";
> +		}
>  	} else if (soc_is_am35xx()) {
>  		cpu_name = (omap3_has_sgx()) ? "AM3517" : "AM3505";
>  	} else if (soc_is_ti816x()) {
> -- 
> 2.7.4
> 

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

end of thread, other threads:[~2017-01-20 18:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-20 16:44 [PATCH] ARM: OMAP3: Fix SoC detection of OMAP36/37 Family aford173 at gmail.com
2017-01-20 18:14 ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).