20 Nov
2007
20 Nov
'07
1:02 p.m.
Takashi Iwai wrote:
Mark Brown wrote:
+#define WM9713_PCM_FORMATS \
- (SNDRV_PCM_FORMAT_S16_LE | SNDRV_PCM_FORMAT_S20_3LE | \
SNDRV_PCM_FORMAT_S24_LE)Is it S24_LE or S24_3LE? I see S24_LE in other codec drivers, but am not pretty sure whether I checked it...
Apparently, these are the formats that an AC'97 controller connected with this codec can support.
Samples on the AC'97 bus are always MSB-aligned with at most 20 bits. I don't think it makes much sense to let the codec driver decide what sample formats the controller should support, because the controller will have to convert the samples anyway (from/to 16, 24 or 32 bits). Furthermore, I'm sure there are AC'97 controllers that can handle big- endian samples on the memory bus.
Regards, Clemens