RE: [PATCH 3/4] ASoC: wm8994: fix MCLK index used for setting MCLK rate

["Sylwester Nawrocki" <s.nawrocki@samsung.com>]

> -----Original Message-----
> From: Marek Szyprowski <m.szyprowski@samsung.com>
> Sent: środa, 21 października 2026 12:18
> Cc: Marek Szyprowski <m.szyprowski@samsung.com>; Liam Girdwood
> <lgirdwood@gmail.com>; Mark Brown <broonie@kernel.org>; Sylwester
> Nawrocki <s.nawrocki@samsung.com>; Charles Keepax
> <ckeepax@opensource.cirrus.com>; Krzysztof Kozlowski <krzk@kernel.org>;
> Simon Shields <simon@lineageos.org>; Artur Weber
> <aweber.kernel@gmail.com>; patches@opensource.cirrus.com; linux-
> sound@vger.kernel.org
> Subject: [PATCH 3/4] ASoC: wm8994: fix MCLK index used for setting MCLK
> rate
> 
> wm8994_set_dai_sysclk() passes the AIF index (dai->id - 1) to
> wm8994_set_mclk_rate(), which uses it to index the MCLK clocks and
> their cached rates. As a result, selecting MCLK2 as the AIF1 clock
> source sets and reads back the rate of MCLK1 instead, and the rate
> of MCLK1 is then stored as the MCLK2 rate.
> 
> On the Exynos4412 Trats2 board, where MCLK1 is a fixed 24MHz clock and
> MCLK2 a 32768Hz clock used when the codec is idle, this gives:
> 
>  wm8994-codec wm8994-codec: AIF1 using MCLK2 at 24000000Hz
> 
> so the driver assumes a 12MHz SYSCLK while it actually runs at 32kHz.
> This skips the 128fs AIF clock ratio setup for low SYSCLK rates and
> makes the mic detection rate configuration use the wrong clock rate.
> 
> Pass the MCLK index matching the selected clock source instead.
> 
> Fixes: 419e2f508336 ("ASoC: wm8994: Add support for setting MCLK clock
> rate")
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

Reviewed-by: Sylwester Nawrocki <s.nawrocki@samsung.com>