ASoC: fsl_xcvr: clear the channel status control memory

TE-N-ShengjiuWang · broonie · commit 73b97d46dde6 · 2025-11-26T13:24:19.000Z
memset_io() writes memory byte by byte with __raw_writeb() on the arm platform if the size is word. but XCVR data RAM memory can't be accessed with byte address, so with memset_io() the channel status control memory is not really cleared, use writel_relaxed() instead. Fixes: 2856448 ("ASoC: fsl_xcvr: Add XCVR ASoC CPU DAI driver") Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> Link: https://patch.msgid.link/20251126064509.1900974-1-shengjiu.wang@nxp.com Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/sound/soc/fsl/fsl_xcvr.c b/sound/soc/fsl/fsl_xcvr.c
@@ -1421,7 +1421,7 @@ static irqreturn_t irq0_isr(int irq, void *devid)
 						bitrev32(val);
 				}
 				/* clear CS control register */
-				memset_io(reg_ctrl, 0, sizeof(val));
+				writel_relaxed(0, reg_ctrl);
 			}
 		} else {
 			regmap_read(xcvr->regmap, FSL_XCVR_RX_CS_DATA_0,

Original file line number	Diff line number	Diff line change
`@@ -1421,7 +1421,7 @@ static irqreturn_t irq0_isr(int irq, void *devid)`
`1421`	`1421`	`bitrev32(val);`
`1422`	`1422`	`}`
`1423`	`1423`	`/* clear CS control register */`
`1424`		`- memset_io(reg_ctrl, 0, sizeof(val));`
	`1424`	`+ writel_relaxed(0, reg_ctrl);`
`1425`	`1425`	`}`
`1426`	`1426`	`} else {`
`1427`	`1427`	`regmap_read(xcvr->regmap, FSL_XCVR_RX_CS_DATA_0,`