On Tue, Aug 24, 2010 at 05:10, <[email protected]> wrote:
> Log Message
>
> [#6193] change min/max channel from 2 to 1 else ad73311 which has
> only 1 cap/play channel can't work.
this changelog should look something like:
[#6193] asoc: bf5xx-i2s: change min/max channel from 2 to 1
ad73311 which has only 1 cap/play channel can't work otherwise
> Modified: trunk/sound/soc/blackfin/bf5xx-i2s.c (9095 => 9096)
>
> @@ -227,13 +227,13 @@
> .suspend = bf5xx_i2s_suspend,
> .resume = bf5xx_i2s_resume,
> .playback = {
> - .channels_min = 2,
> - .channels_max = 2,
> + .channels_min = 1,
> + .channels_max = 1,
> .rates = BF5XX_I2S_RATES,
> .formats = BF5XX_I2S_FORMATS,},
> .capture = {
> - .channels_min = 2,
> - .channels_max = 2,
> + .channels_min = 1,
> + .channels_max = 1,
> .rates = BF5XX_I2S_RATES,
> .formats = BF5XX_I2S_FORMATS,},
although, i kind of wonder about the correctness of this patch. the
bf5xx-i2s interface is used by any codec that has an i2s interface.
why are we disabling functionality just to make one codec (which isnt
even in the tree) work ?
-mike
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
