"Nayak, Rajendra" <rnayak@xxxxxx> writes: > <snip>.. > >> > >> > Below is an untested patch to provide some mechanism to deal with this -- >> > I'd appreciate everyone's comments on this, particularly the comments in >> > the patch code on how to deal with this problem. >> >> Hi Paul, >> >> I'll test this patch to see if it solves the issue seen with i2c on n800. > > Hi Paul, Benoit, Kevin, > > So looks like the issue with i2c on n800 isn't really because of additional > delay needed after a clock enable, but related to the omap_readl/writel's > done on the 16bit i2c registers as I was suspecting earlier. > > The below patch fixes the n800 issue for me. My mailer might mess up the alignment > hence attaching it too. > > From af9733f2e63a16b446635d52015e05d631f0788f Mon Sep 17 00:00:00 2001 > From: Rajendra Nayak <rnayak@xxxxxx> > Date: Tue, 5 Oct 2010 16:36:30 +0530 > Subject: [PATCH] OMAP: hwmod: Handle modules with 16bit registers > > Some modules which have 16bit registers can cause imprecise > aborts if a __raw_readl/writel is used to read/write 32 bits. > > Add an additional flag to identify modules which have such > hard requirement, and handle it in the hwmod framework. > > Signed-off-by: Rajendra Nayak <rnayak@xxxxxx> Can you also send a corresponding patch to the i2c hwmod data. I'll rebase the rest of the series against current l-o master, and maybe we can still get this in for .37. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html