On Mon, Jul 01, 2013 at 07:43:27PM +0200, Marek Vasut wrote:
Dear Fabio Estevam,
On Mon, Jul 1, 2013 at 12:33 PM, Shawn Guo shawn.guo@linaro.org wrote:
On Mon, Jul 01, 2013 at 12:11:58PM -0300, Fabio Estevam wrote:
On Mon, Jul 1, 2013 at 11:34 AM, Shawn Guo shawn.guo@linaro.org wrote:
And also why we do not see the impact of i2c issue on sgtl5000 before the offending commit af8ee11 (ASoC: sgtl5000: Fix driver probe after reset)?
Probably because the mxs i2c was working before this commit.
Do you have a commit id at which mxs i2c is known good? I would like to find out it's a mxs i2c issue or sgtl5000 problem.
I don't have it, but I am adding Alexandre in case he knows.
Alexandre,
Do you happen to know a commit id which does not show the mxs i2c timeouts you have been observing?
I think you can just disable the PIO mode altogether (around line 500 ... if (msg->len < 8) ... replace this with if (0) ) and then the problem should not be there (if it's a PIO problem).
Ok, Fabio is right. With the change, sgtl5000 write works even without that msleep(50).
So, Mark, please disregard the patch, and we should fix mxs i2c driver instead.
Shawn