]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
ASoC: fsl_sai: fix bit order for DSD format
authorShengjiu Wang <shengjiu.wang@nxp.com>
Thu, 23 Oct 2025 06:45:37 +0000 (14:45 +0800)
committerMark Brown <broonie@kernel.org>
Thu, 23 Oct 2025 12:42:33 +0000 (13:42 +0100)
The DSD little endian format requires the msb first, because oldest bit
is in msb.
found this issue by testing with pipewire.

Fixes: c111c2ddb3fd ("ASoC: fsl_sai: Add PDM daifmt support")
Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com>
Link: https://patch.msgid.link/20251023064538.368850-2-shengjiu.wang@nxp.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_sai.c

index 757e7868e322a4f2fa1df1c0731a2c2a2427ee39..65093325a6b6f4c7588176466004123a349518dc 100644 (file)
@@ -353,7 +353,6 @@ static int fsl_sai_set_dai_fmt_tr(struct snd_soc_dai *cpu_dai,
                break;
        case SND_SOC_DAIFMT_PDM:
                val_cr2 |= FSL_SAI_CR2_BCP;
-               val_cr4 &= ~FSL_SAI_CR4_MF;
                sai->is_pdm_mode = true;
                break;
        case SND_SOC_DAIFMT_RIGHT_J:
@@ -638,7 +637,7 @@ static int fsl_sai_hw_params(struct snd_pcm_substream *substream,
        val_cr5 |= FSL_SAI_CR5_WNW(slot_width);
        val_cr5 |= FSL_SAI_CR5_W0W(slot_width);
 
-       if (sai->is_lsb_first || sai->is_pdm_mode)
+       if (sai->is_lsb_first)
                val_cr5 |= FSL_SAI_CR5_FBT(0);
        else
                val_cr5 |= FSL_SAI_CR5_FBT(word_width - 1);