[PATCH 2/6] omap4: prcm: Fix the CPUx clockdomain offsets

Santosh Shilimkar [off-list ref] writes:

CPU0 and CPU1 clockdomain is at the offset of 0x18 from the LPRM base.
The header file has set it wrongly to 0x0. Offset 0x0 is for CPUx power
domain control register

Fix the same.

Has this also been updated in the autogen scripts?  

Benoit?

Kevin

On 2/2/2011 2:20 AM, Hilman, Kevin wrote:

Santosh Shilimkar[off-list ref]  writes:

quoted

CPU0 and CPU1 clockdomain is at the offset of 0x18 from the LPRM base.
The header file has set it wrongly to 0x0. Offset 0x0 is for CPUx power
domain control register

Fix the same.

Has this also been updated in the autogen scripts?

Benoit?

No, I didn't see any patch to update that yet.

Santosh or Rajendra,
Did you already fix it?

Benoit

Kevin

quoted

Signed-off-by: Santosh Shilimkar<redacted>
Cc: Paul Walmsley<paul@pwsan.com>
---
  arch/arm/mach-omap2/prcm_mpu44xx.h |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-omap2/prcm_mpu44xx.h b/arch/arm/mach-omap2/prcm_mpu44xx.h
index 729a644..3300ff6 100644
--- a/arch/arm/mach-omap2/prcm_mpu44xx.h
+++ b/arch/arm/mach-omap2/prcm_mpu44xx.h

@@ -38,8 +38,8 @@
  #define OMAP4430_PRCM_MPU_CPU1_INST		0x0800

  /* PRCM_MPU clockdomain register offsets (from instance start) */
-#define OMAP4430_PRCM_MPU_CPU0_MPU_CDOFFS	0x0000
-#define OMAP4430_PRCM_MPU_CPU1_MPU_CDOFFS	0x0000
+#define OMAP4430_PRCM_MPU_CPU0_MPU_CDOFFS	0x0018
+#define OMAP4430_PRCM_MPU_CPU1_MPU_CDOFFS	0x0018


  /*

Hi Kevin,

On 2/2/2011 10:24 AM, Cousson, Benoit wrote:

On 2/2/2011 2:20 AM, Hilman, Kevin wrote:

quoted

Santosh Shilimkar[off-list ref]   writes:

quoted

CPU0 and CPU1 clockdomain is at the offset of 0x18 from the LPRM base.
The header file has set it wrongly to 0x0. Offset 0x0 is for CPUx power
domain control register

Fix the same.

Has this also been updated in the autogen scripts?

Benoit?

No, I didn't see any patch to update that yet.

Santosh or Rajendra,
Did you already fix it?

I updated the scripts with Santosh fixes and found a register name issue in this file.

The fix is inlined at the end. 
Santosh will include it in a new revision of the series.

Regards,
Benoit

---