]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-client.git/commitdiff
ASoC: renesas: rz-ssi: Fix playback and capture
authorClaudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Fri, 13 Feb 2026 17:24:34 +0000 (19:24 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 17 Feb 2026 13:17:03 +0000 (13:17 +0000)
In the current codebase the rz_ssi_stream_get() is called at the beginning
of rz_ssi_dai_trigger() before rz_ssi_stream_init() is called.
Since rz_ssi_stream_init() sets the ssi->{playback, capture}->substream,
relying on it in rz_ssi_stream_get() is incorrect. Fix this by checking
substream->stream instead as the original code did.

Fixes: 53dfb2ad6fcf ("ASoC: renesas: rz-ssi: Simplify the logic in rz_ssi_stream_is_play()")
Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Link: https://patch.msgid.link/20260213172434.3801015-1-claudiu.beznea.uj@bp.renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/renesas/rz-ssi.c

index 39aa865bdca3dbe8893d738b389ca57eddfebd8d..74e078c041502f013776ce1540f591b5a03c6fdf 100644 (file)
@@ -180,7 +180,7 @@ static inline bool rz_ssi_stream_is_play(struct snd_pcm_substream *substream)
 static inline struct rz_ssi_stream *
 rz_ssi_stream_get(struct rz_ssi_priv *ssi, struct snd_pcm_substream *substream)
 {
-       return (ssi->playback.substream == substream) ? &ssi->playback : &ssi->capture;
+       return (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) ? &ssi->playback : &ssi->capture;
 }
 
 static inline bool rz_ssi_is_dma_enabled(struct rz_ssi_priv *ssi)