On Mon, Dec 22, 2008 at 4:45 PM, Mark Brown < broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On Mon, Dec 22, 2008 at 10:57:33AM +0200, Jarkko Nikula wrote: > > Thanks to Troy Kisky <troy.kisky@xxxxxxxxxxxxxxxxxxx> for noticing. > > > - DSP_A format has 1-bit data delay which corresponds to SSM6202 submode > 2 > > - DSP_B has 0-bit data delay which corresponds to submode 1 > > - Currently driver sets them opposite so swap the submode setting > > > Signed-off-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxx> > > Cc: Cliff Cai <cliff.cai@xxxxxxxxxx> > > Presumably this means that the Blackfin I2S driver also needs to be > updated (probably just replacing DSP_A with DSP_B by the looks of it)? > Though since the SSM6206 machine driver uses I2S it shouldn't matter too > much if they're out of sync temporarily. > That was my first impression as well, but then I noticed that bf5xx-ad73311.c machine driver is using DSP_A and AD73311 looks like it's hard coded for DSP_A. Hopefully bf5xx-i2s and ssm2602 are still compatible with each other if using DSP_A :-) Jarkko _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
