Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-27 Thread Mark Brown
On Fri, Nov 27, 2009 at 01:37:54PM +0530, Aggarwal, Anuj wrote: > [Aggarwal, Anuj] We were able to fine tune NFS and use arecord to capture > large files. But some more problems cropped up when tried to suspend / > resume. Basic playback is working fine wrt suspend/resume. But capture, Incidental

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-27 Thread Mark Brown
On Fri, Nov 27, 2009 at 01:37:54PM +0530, Aggarwal, Anuj wrote: > [Aggarwal, Anuj] We were able to fine tune NFS and use arecord to capture > large files. But some more problems cropped up when tried to suspend / > resume. Basic playback is working fine wrt suspend/resume. But capture, > either tr

RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-27 Thread Aggarwal, Anuj
> -Original Message- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Thursday, November 26, 2009 9:00 PM > To: Aggarwal, Anuj > Cc: 'Troy Kisky'; alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; > Arun KS > Subject: Re: [alsa-d

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Philby John
alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; > > Arun KS > > Subject: Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume > > hang > > > > On Thu, Nov 26, 2009 at 08:26:44PM +0530, Aggarwal, Anuj wrote: > > > > > driver, although I am not

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Mark Brown
On Thu, Nov 26, 2009 at 08:52:08PM +0530, Aggarwal, Anuj wrote: > [Aggarwal, Anuj] I am still surprised how this could be a NFS writeout issue > as we are seeing a consistent read/write rate of 2Mbps over tftp. When dd > command is used for read/write to further check NFS performance, 2Mbps for >

RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Aggarwal, Anuj
27;Troy Kisky'; alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; > Arun KS > Subject: RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume > hang > > > -Original Message- > > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com]

RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Aggarwal, Anuj
> -Original Message- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Thursday, November 26, 2009 8:41 PM > To: Aggarwal, Anuj > Cc: 'Troy Kisky'; alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; > Arun KS > Subject: Re: [alsa-d

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Mark Brown
On Thu, Nov 26, 2009 at 08:26:44PM +0530, Aggarwal, Anuj wrote: > driver, although I am not sure if the problem is in the audio driver. > When tried to capture, using NFS as storage, it gives overrun error and comes > out with: > arecord: pcm_read:1617: read error: Input/output error > It happe

RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Aggarwal, Anuj
> -Original Message- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Thursday, November 26, 2009 3:53 PM > To: Aggarwal, Anuj > Cc: 'Troy Kisky'; alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; > Arun KS > Subject: Re: [alsa-d

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-26 Thread Mark Brown
On Thu, Nov 26, 2009 at 08:31:11AM +0530, Aggarwal, Anuj wrote: > I have already tried that but it doesn't fix the problem entirely. > Still, I was not able to playback properly because the bias settings > were not handled correctly and as you can see, it writes to some > invalid registers too (a

RE: [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Aggarwal, Anuj
> -Original Message- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Thursday, November 26, 2009 1:10 AM > To: Aggarwal, Anuj > Cc: alsa-de...@alsa-project.org; linux-omap@vger.kernel.org > Subject: Re: [PATCH] ASoC: AM3517: Fix AIC23 suspend/resum

RE: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Aggarwal, Anuj
> -Original Message- > From: Troy Kisky [mailto:troy.ki...@boundarydevices.com] > Sent: Thursday, November 26, 2009 1:53 AM > To: Aggarwal, Anuj > Cc: alsa-de...@alsa-project.org; linux-omap@vger.kernel.org; Mark Brown; > Arun KS > Subject: Re: [alsa-devel] [PATCH] AS

Re: [alsa-devel] [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Troy Kisky
Anuj Aggarwal wrote: > System hang is observed While trying to do suspend. It was because > of an infinite loop in the AIC23 resume path which was trying to > restore AIC23 register values from the register cache. > > This patch fixes the problem by correcting the resume path and > properly activa

Re: [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Mark Brown
On Thu, Nov 26, 2009 at 12:49:11AM +0530, Aggarwal, Anuj wrote: > > int i; > u16 reg; > > /* Sync reg_cache with the hardware */ > for (reg = 0; reg < ARRAY_SIZE(tlv320aic23_reg); i++) { > u16 val = tlv320aic23_read_reg_cache(codec, reg); >

RE: [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Aggarwal, Anuj
> -Original Message- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Wednesday, November 25, 2009 7:04 PM > To: Aggarwal, Anuj > Cc: alsa-de...@alsa-project.org; linux-omap@vger.kernel.org > Subject: Re: [PATCH] ASoC: AM3517: Fix AIC23 suspend/r

Re: [PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Mark Brown
On Wed, Nov 25, 2009 at 06:40:31PM +0530, Anuj Aggarwal wrote: > System hang is observed While trying to do suspend. It was because > of an infinite loop in the AIC23 resume path which was trying to > restore AIC23 register values from the register cache. This doesn't appear to tie in with the ac

[PATCH] ASoC: AM3517: Fix AIC23 suspend/resume hang

2009-11-25 Thread Anuj Aggarwal
System hang is observed While trying to do suspend. It was because of an infinite loop in the AIC23 resume path which was trying to restore AIC23 register values from the register cache. This patch fixes the problem by correcting the resume path and properly activating/deactivating the digital int