13 Dec
2013
13 Dec
'13
3:49 p.m.
On Fri, Dec 13, 2013 at 02:42:32PM +0100, Lars-Peter Clausen wrote:
Normally you'd expect the I2S core to only put the 24 bits of data onto the bus for both S24_3LE and S24_LE (it might add necessary trailing padding if the bit clocks per frame is > 24). A CODEC driver should really not have to care about the in memory layout of the data since all it will see is a serialized bit stream. I think ideally we wouldn't check for params_format() but rather for snd_pcm_format_width(params_format()).
Yes, that would be a much better approach. In fact, let me go attack that...