[PATCH v4 0/4] OMAP2/3: DMA: FIFO drain errata fixes

2010-10-04 Thread Peter Ujfalusi
Hello,

Changes since v3:
- use flags for errata handling
 - OMAP2 buffering disable converted to use flags
 - OMAP3 i541 is using the dma_errata flag
- Spelling fix in OMAP3 drain timeout message
Changes since v2:
- remove multiple assignment
Changes since v1:
- Errata ID added to the comment (i541)

Intro mail from v1:

The following series fixes the sDMA FIFO drain issue present for OMAP2 and
OMAP3, and covered by an errata.
For OMAP2 the omap_start_dma had a comment about this errata, but the workaround
configured wrong bit (CCR_EN instead of the correct BUFFERING_DISABLE bit).
The first patch from Jarkko Nikula jhnik...@gmail.com fixes this.
For OMAP3 the suggestion is to set sDMA to NoStandby before disabling
the channel, and wait for the drain to finish, than configure sDMA to
SmartStandby again. The second patch implements this workaround for OMAP3.

The FIFO drain problem can be reproduced on both OMAP2 and OMAP3 while using
audio (capture case):
Either introduce load to the CPU:
nice -19 arecord -D hw:0 -M -B 1 -F 5000 -f dat  /dev/null  \
dd if=/dev/urandom of=/dev/null

or suspending the arecord, and resuming it:
arecord -D hw:0 -M -B 1 -F 5000 -f dat  /dev/null
CTRL+Z; fg; CTRL+Z; fg; ...

On OMAP2 we can not recover from this problem. The board has to be rebooted,
while OMAP3 can be recovered by reseting the sDMA channel, or introducing
unrelated sDMA activity (which takes sDMA out from Standby - but this is not
working all the time).

---
Peter Ujfalusi (3):
  OMAP: DMA: Introduce errata handling variable, and macro
  OMAP2: DMA: Use errata flag for disabling buffering
  OMAP3: DMA: Errata i541: sDMA FIFO draining does not finish

ext Jarkko Nikula (1):
  omap: dma: Fix buffering disable bit setting for omap24xx

 arch/arm/plat-omap/dma.c  |   60 ++--
 arch/arm/plat-omap/include/plat/dma.h |4 ++
 2 files changed, 60 insertions(+), 4 deletions(-)

--
1.7.3

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 0/4] OMAP2/3: DMA: FIFO drain errata fixes

2010-10-04 Thread Peter Ujfalusi
On Monday 04 October 2010 11:29:16 ext Peter Ujfalusi wrote:
 Hello,
 
 Will resend, since

I'll wait for comments on v4 series, and I'll send the v5 later today after the 
comments to reduce the noise.

-- 
Péter
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 0/4] OMAP2/3: DMA: FIFO drain errata fixes

2010-10-04 Thread Peter Ujfalusi
Hello,

Will resend, since

On Monday 04 October 2010 11:18:10 ext Peter Ujfalusi wrote:

...

 ext Jarkko Nikula (1):
   omap: dma: Fix buffering disable bit setting for omap24xx

Our mail server changed Jarkko's name.
Sorry.

-- 
Péter
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 0/4] OMAP2/3: DMA: FIFO drain errata fixes

2010-10-04 Thread Tony Lindgren
* Peter Ujfalusi peter.ujfal...@nokia.com [101004 01:26]:
 On Monday 04 October 2010 11:29:16 ext Peter Ujfalusi wrote:
  Hello,
  
  Will resend, since
 
 I'll wait for comments on v4 series, and I'll send the v5 later today after 
 the 
 comments to reduce the noise.

Looks like we should have the minimal two patches to queue as fixes
for v2.6.36, then queue the errata flag changes for the upcoming
merge window.

Regards,

Tony
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html