Hello Adam,
Am 09.04.2017 um 04:29 schrieb Adam Ford:
On Sat, Apr 8, 2017 at 5:53 PM, Tom Rini trini@konsulko.com wrote:
On Sat, Apr 08, 2017 at 05:34:29PM -0500, Adam Ford wrote:
The register names and offset were not correct as per the TRM for OMAP3530 and OMAP3630. Correct the naing and offsets per the documentation
Signed-off-by: Adam Ford aford173@gmail.com
diff --git a/arch/arm/include/asm/arch-omap3/i2c.h b/arch/arm/include/asm/arch-omap3/i2c.h index b370290..6b3a3da 100644 --- a/arch/arm/include/asm/arch-omap3/i2c.h +++ b/arch/arm/include/asm/arch-omap3/i2c.h @@ -17,7 +17,7 @@ struct i2c { unsigned short res2; unsigned short stat; /* 0x08 */ unsigned short res3;
unsigned short iv; /* 0x0C */
unsigned short we; /* 0x0C */ unsigned short res4; unsigned short syss; /* 0x10 */ unsigned short res4a;@@ -43,6 +43,18 @@ struct i2c { unsigned short res14; unsigned short systest; /* 0x3c */ unsigned short res15;
unsigned short bufstat; /* 0x40 */unsigned short res16;unsigned short oa1; /* 0x44 */unsigned short res17;unsigned short oa2; /* 0x48 */unsigned short res18;unsigned short oa3; /* 0x4c */unsigned short res19;unsigned short actoa; /* 0x50 */unsigned short res20;unsigned short sblock; /* 0x54 */unsigned short res21;Are we making use of any of these new entries?
Not really. I am trying to get DM_I2C working on OMAP3 and I was looking through the history of what was done to make it work on AM33xx and OMAP4+. Commit 3465f807d4b90378d86b3 ("omap4: i2c: correct register offset for sync register") corrected some incorrect registers, so I thought I'd look through the corresponding file for omap3. I noticed that offset 0x0C was not named right per the TRM, so I read through to the rest. I can see the argument for not pulling in the new stuff, but I was trying to be consistent with what was done for omap4 and omap5.
I like to have this corrected version, so:
Reviewed-by: Heiko Schocher hs@denx.de
but last OK should come from Tom, thanks!
bye, Heiko
adam
-- Tom
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot