ASoC: mediatek: mt8188: add missing initialization
authorTrevor Wu <trevor.wu@mediatek.com>
Wed, 1 Mar 2023 11:01:59 +0000 (19:01 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 1 Mar 2023 14:49:16 +0000 (14:49 +0000)
In etdm dai driver, dai_etdm_parse_of() function is used to parse dts
properties to get parameters. There are two for-loops which are
sepearately for all etdm and etdm input only cases. In etdm in only
loop, dai_id is not initialized, so it keeps the value intiliazed in
another loop.

In the patch, add the missing initialization to fix the unexpected
parsing problem.

Fixes: 2babb4777489 ("ASoC: mediatek: mt8188: support etdm in platform driver")
Signed-off-by: Trevor Wu <trevor.wu@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20230301110200.26177-2-trevor.wu@mediatek.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/mediatek/mt8188/mt8188-dai-etdm.c

index 1c53d4c..7a37752 100644 (file)
@@ -2498,6 +2498,9 @@ static void mt8188_dai_etdm_parse_of(struct mtk_base_afe *afe)
 
        /* etdm in only */
        for (i = 0; i < 2; i++) {
+               dai_id = ETDM_TO_DAI_ID(i);
+               etdm_data = afe_priv->dai_priv[dai_id];
+
                snprintf(prop, sizeof(prop), "mediatek,%s-chn-disabled",
                         of_afe_etdms[i].name);