On Mon, Jul 9, 2012 at 2:20 PM, Vaibhav Hiremath [off-list ref] wrote:

On 7/6/2012 2:51 PM, Santosh Shilimkar wrote:

quoted

From: R Sricharan <redacted>

OMAP5430 is Texas Instrument's SOC based on ARM Cortex-A15 SMP
architecture. It's a dual core SOC with GIC used for interrupt
handling and with an integrated L2 cache controller.

OMAP5432 is another variant of OMAP5430, with a
memory controller supporting DDR3 and SATA.

Patch includes:
 - The machine specific headers and sources updates.
 - Platform header updates.
 - Minimum initialisation support for serial.
 - IO table init

Signed-off-by: R Sricharan <redacted>
Signed-off-by: Santosh Shilimkar <redacted>
---

[..]

quoted

+#if defined(CONFIG_SOC_OMAP5)
+static struct omap_globals omap5_globals = {
+     .class  = OMAP54XX_CLASS,
+     .tap    = OMAP2_L4_IO_ADDRESS(OMAP54XX_SCM_BASE),
+     .ctrl   = OMAP2_L4_IO_ADDRESS(OMAP54XX_SCM_BASE),
+     .ctrl_pad       = OMAP2_L4_IO_ADDRESS(OMAP54XX_CTRL_BASE),
+     .prm    = OMAP2_L4_IO_ADDRESS(OMAP54XX_PRM_BASE),
+     .cm     = OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_AON_BASE),
+     .cm2    = OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_BASE),
+     .prcm_mpu = OMAP2_L4_IO_ADDRESS(OMAP54XX_PRCM_MPU_BASE),

I am not sure whether we had discussed on this before, couldn't find it.

Why don't we reuse OMAP4 data here and elsewhere??

Because data is not same between OMAP4 and OMAP5.
Wherever it is same, it is taken care.

[..]

quoted

diff --git a/arch/arm/plat-omap/sram.c b/arch/arm/plat-omap/sram.c
index 70cf825..766181c 100644
--- a/arch/arm/plat-omap/sram.c
+++ b/arch/arm/plat-omap/sram.c

@@ -6,8 +6,8 @@
  * Copyright (C) 2005 Nokia Corporation
  * Written by Tony Lindgren <tony@atomide.com>
  *
- * Copyright (C) 2009 Texas Instruments
- * Added OMAP4 support - Santosh Shilimkar <santosh.shilimkar@ti.com>
+ * Copyright (C) 2009-2012 Texas Instruments
+ * Added OMAP4/5 support - Santosh Shilimkar <santosh.shilimkar@ti.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 as

@@ -44,6 +44,7 @@
 #else
 #define OMAP4_SRAM_PUB_PA    (OMAP4_SRAM_PA + 0x4000)
 #endif
+#define OMAP5_SRAM_PA                0x40300000

We have mix of such definitions here, for example,

"arch/arm/plat-omap/include/plat/sram.h"
and now in arch/arm/plat-omap/sram.c here itself.


May be right time to clean it up now.

Thats because of an interconnect BUG which needed it exported
at plat level in case of OMAP4.

Regards
Santosh