Re: [PATCH] OMAP4: I2C: correct the SYSC register offset for OMAP4

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday 20 June 2011 09:21 PM, Shubhrajyoti wrote:
On Monday 20 June 2011 09:09 PM, Kevin Hilman wrote:
shubhrajyoti@xxxxxx writes:

From: Shubhrajyoti D<shubhrajyoti@xxxxxx>

The sysc offset is erroneous correct the offset.

Signed-off-by: Shubhrajyoti D<shubhrajyoti@xxxxxx>

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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux