On Monday 20 June 2011 09:21 PM, Shubhrajyoti wrote:
On Monday 20 June 2011 09:09 PM, Kevin Hilman wrote:
[email protected] writes:

From: Shubhrajyoti D<[email protected]>

The sysc offset is erroneous correct the offset.

Signed-off-by: Shubhrajyoti D<[email protected]>

Actally, a better fix for this problem is to completely remove SYSC
register accesses for OMAP3+ platforms.  This is handled already at the
hwmod layer.

Also, please Cc linux-omap for OMAP changes.
Agree,
1. Currently the flag HWMOD_INIT_NO_RESET is there for OMAP4.
2. Currently the reset is there even in the error paths and hwmod takes care of it in init only. 3. I2C reset sequence is disable -> reset -> enable and poll. So it needs a .reset function in hwmod.
Another issue I am not able to solve currently is that after reset the restore of settings.
Doing get_sync and put_sync is one option however it will be a hack again.

Anyways fixing it is in my plans till then thought of having it corrected.
Kevin

---
  drivers/i2c/busses/i2c-omap.c |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index d53cd61..cccec28 100644
--- a/drivers/i2c/busses/i2c-omap.c
+++ b/drivers/i2c/busses/i2c-omap.c
@@ -236,7 +236,7 @@ const static u8 reg_map_ip_v2[] = {
      [OMAP_I2C_BUF_REG] = 0x94,
      [OMAP_I2C_CNT_REG] = 0x98,
      [OMAP_I2C_DATA_REG] = 0x9c,
-    [OMAP_I2C_SYSC_REG] = 0x20,
+    [OMAP_I2C_SYSC_REG] = 0x10,
      [OMAP_I2C_CON_REG] = 0xa4,
      [OMAP_I2C_OA_REG] = 0xa8,
      [OMAP_I2C_SA_REG] = 0xac,


--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to