Unverified Commit ec061186 authored by Troy Mitchell's avatar Troy Mitchell Committed by Mark Brown
Browse files

ASoC: spacemit: fix RX DMA params not set when TX is running 



When TX is already running (SSCR_SSE is set), the hw_params callback
returns early before setting up DMA parameters for the RX stream. This
prevents the capture path from configuring its DMA data properly.

Move the SSCR_SSE check after DMA parameter setup and format
constraints, so both TX and RX streams get their DMA configuration
regardless of whether the hardware is already enabled. The early return
now only skips the register writes that would disrupt an active stream.

Fixes: fce21744 ("ASoC: spacemit: add i2s support for K1 SoC")
Signed-off-by: default avatarTroy Mitchell <troy.mitchell@linux.spacemit.com>
Link: https://patch.msgid.link/20260429-k1-i2s-fix-v2-1-8d67835aaddc@linux.spacemit.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent e366ce8b

sound/soc/spacemit/k1_i2s.c

+3 −4
Original line number Diff line number Diff line
@@ -148,10 +148,6 @@ static int spacemit_i2s_hw_params(struct snd_pcm_substream *substream, 
	u32 val;

	int ret;



	val = readl(i2s->base + SSCR);

	if (val & SSCR_SSE)

		return 0;



	dma_data = &i2s->playback_dma_data;



	if (substream->stream == SNDRV_PCM_STREAM_CAPTURE)
@@ -199,6 +195,9 @@ static int spacemit_i2s_hw_params(struct snd_pcm_substream *substream, 
	}



	val = readl(i2s->base + SSCR);

	if (val & SSCR_SSE)

		return 0;



	val &= ~SSCR_DW_32BYTE;

	val |= data_width;

	writel(val, i2s->base + SSCR);