diff options
| author | Ștefan Ghețu <stefanghetu9@gmail.com> | 2026-08-30 23:51:06 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-09-01 13:48:14 +0100 |
| commit | d3dbccfe6afa7b9a6a7ed65cfaa76a47fa050a56 (patch) | |
| tree | 0135e7b7fa31b8162086bb2037b4905891e471fd /scripts/Makefile.thinlto | |
| parent | 7a4ce92d150b9e7ecf1a710a34d8cdeb590d3751 (diff) | |
| download | linux-next-d3dbccfe6afa7b9a6a7ed65cfaa76a47fa050a56.tar.gz linux-next-d3dbccfe6afa7b9a6a7ed65cfaa76a47fa050a56.zip | |
ASoC: fsl_micfil: balance mclk enable/disable
hw_params() enables mclk unconditionally and hw_free() disables it
unconditionally, but the PCM core does not guarantee 1:1 pairing:
hw_free() can run without hw_params(), and hw_params() can be called
multiple times from the SETUP state. This triggers an "already
disabled" WARN() in the first case and leaks an enable reference in
the second, leaving the clock ungateable.
Guard both sides with the existing mclk_flag, as fsl_sai.c does with
mclk_streams.
Fixes: b47024dc624b ("ASoC: fsl_micfil: Add mclk enable flag")
Signed-off-by: Ștefan Ghețu <stefanghetu9@gmail.com>
Reviewed-by: Chancel Liu <chancel.liu@nxp.com>
Link: https://patch.msgid.link/20260830205106.11267-1-stefanghetu9@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'scripts/Makefile.thinlto')
0 files changed, 0 insertions, 0 deletions