diff options
author | Peter Rosin <[email protected]> | 2021-09-20 16:49:39 +0200 |
---|---|---|
committer | Mark Brown <[email protected]> | 2021-09-21 13:23:34 +0100 |
commit | 3f4b57ad07d9237acf1b8cff3f8bf530cacef87a (patch) | |
tree | 0012241b4a6837940ee2506b94e07df3d725b483 /scripts/gcc-plugins/cyc_complexity_plugin.c | |
parent | cfacfefd382af3b42905108b54f02820dca225c4 (diff) |
ASoC: pcm512x: Mend accesses to the I2S_1 and I2S_2 registers
Commit 25d27c4f68d2 ("ASoC: pcm512x: Add support for more data formats")
breaks the TSE-850 device, which is using a pcm5142 in I2S and
CBM_CFS mode (maybe not relevant). Without this fix, the result
is:
pcm512x 0-004c: Failed to set data format: -16
And after that, no sound.
This fix is not 100% correct. The datasheet of at least the pcm5142
states that four bits (0xcc) in the I2S_1 register are "RSV"
("Reserved. Do not access.") and no hint is given as to what the
initial values are supposed to be. So, specifying defaults for
these bits is wrong. But perhaps better than a broken driver?
Fixes: 25d27c4f68d2 ("ASoC: pcm512x: Add support for more data formats")
Cc: Liam Girdwood <[email protected]>
Cc: Mark Brown <[email protected]>
Cc: Jaroslav Kysela <[email protected]>
Cc: Takashi Iwai <[email protected]>
Cc: Kirill Marinushkin <[email protected]>
Cc: Peter Ujfalusi <[email protected]>
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Peter Rosin <[email protected]>
Signed-off-by: Peter Ujfalusi <[email protected]>
Reviewed-by: Peter Ujfalusi <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>
Diffstat (limited to 'scripts/gcc-plugins/cyc_complexity_plugin.c')
0 files changed, 0 insertions, 0 deletions