From mboxrd@z Thu Jan 1 00:00:00 1970 From: marex@denx.de (Marek Vasut) Date: Mon, 1 Jul 2013 19:43:27 +0200 Subject: [alsa-devel] [PATCH 1/4] ASoC: sgtl5000: give it a ramping time before writting In-Reply-To: References: <1372666571-17276-1-git-send-email-shawn.guo@linaro.org> <20130701153327.GX13206@S2101-09.ap.freescale.net> Message-ID: <201307011943.27489.marex@denx.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dear Fabio Estevam, > On Mon, Jul 1, 2013 at 12:33 PM, Shawn Guo 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 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). Best regards, Marek Vasut