[alsa-devel] [PATCH 1/3] ASoC: DaVinci: i2s, reduce underruns by combining into 1 element
Troy Kisky
troy.kisky at boundarydevices.com
Tue Sep 1 20:23:27 CEST 2009
Mark Brown wrote:
> On Mon, Aug 31, 2009 at 04:31:43PM -0700, Troy Kisky wrote:
>
>> + /*
>> + * Allowing this is more efficient and eliminates left and right swaps
>> + * caused by underruns, but will swap the left and right channels
>> + * when compared to previous behavior.
>> + */
>> + unsigned disable_channel_combine:1;
>
> Last time you submitted this I suggested changing this to make the
> default the other way round so that there's no breakage on existing
> boards which aren't designed for this channel swap behaviour. Is there
> some reason not to do that?
>
I think that it is better to make sure that they know the possible problems
disabling this may cause. Channels always swapped seems a lot better than
channels randomly swapped.
Troy
More information about the Alsa-devel
mailing list