Bug 12272 - Channel Swap (causes Test Tone to fail)
: Channel Swap (causes Test Tone to fail)
Status: RESOLVED FIXED
Product: SB Radio
Classification: Unclassified
Component: Audio
: Include FW version in comment
: PC Windows XP
: P1 critical (vote)
: MPQ
Assigned To: Richard Titmuss
:
Depends on:
Blocks: 12456 12666
  Show dependency treegraph
 
Reported: 2009-06-04 14:15 UTC by James Richardson
Modified: 2009-09-08 09:29 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description James Richardson 2009-06-04 14:15:26 UTC
Playing the built in test tone, bass and mid may fail to play or cut in and out
Comment 1 Caleb Crome 2009-06-08 10:44:28 UTC
This is caused by audio channels swapping left and right.  We updated the test file to make it so it doesn't matter if the channels swap.

So, I'm sending to Richard to fix the swap issues (not necessary for PB2).
Comment 2 James Richardson 2009-06-08 10:56:27 UTC
With the new test tone, moving the volume knob to 0 then back up, when the tone is playing will cause it to stop or act badly
Comment 3 Blackketter Dean 2009-06-11 09:40:14 UTC
And the tone stops for me too after a while.
Comment 4 Caleb Crome 2009-06-11 11:19:28 UTC
This could be due to imx25 errata ENGcm09835.  Implemented a change in svn r6064 that needs to be tested.
Comment 5 Richard Titmuss 2009-07-08 13:12:34 UTC
Fix from Freescale in r6435. From Sammy @ freescale:


SSI underrun reduced much since two FIFOs are used. In same condtion, 
it is reduced from 100 times to 5~6 times with same audio playback 
duration in my side. There is no audio swap since left channel is put 
to FIFO0, right channel is put to FIFO1 even with underrun.

Current SSI underrun was caused by the high overloading in EMI bus. The
SDMA core needs to access data in the context and descriptor almost all
the time, and the data is in EMI currently.

In order to speed up the SDMA data transaction, putting SDMA content and
description data in the IRAM will decrease the EMI loading because IRAM
access bypass the EMI. And also current audio data are located in the
IRAM, the audio playback is isolated from the heavy EMI loading that's
caused by other routines, such as display rotation, audio codec, LCD
update, etc.
Comment 6 Richard Titmuss 2009-07-09 06:03:03 UTC
The last patch distorted the left channel, a new version has been checked in r6454. This one sounds good, and fixes the SSI underruns.