ASoC: max98088: Check for clk_prepare_enable() error

[ Upstream commit 1a70579723 ]

clk_prepare_enable() may fail, so we should better check its return
value and propagate it in the case of error.

Fixes: 62a7fc32a6 ("ASoC: max98088: Add master clock handling")
Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
Link: https://patch.msgid.link/20240628080534.843815-1-nichen@iscas.ac.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Chen Ni 2024-06-28 16:05:34 +08:00 committed by Greg Kroah-Hartman
parent cc3c0d2784
commit 57a6e7619c

View File

@ -1318,6 +1318,7 @@ static int max98088_set_bias_level(struct snd_soc_component *component,
enum snd_soc_bias_level level)
{
struct max98088_priv *max98088 = snd_soc_component_get_drvdata(component);
int ret;
switch (level) {
case SND_SOC_BIAS_ON:
@ -1333,10 +1334,13 @@ static int max98088_set_bias_level(struct snd_soc_component *component,
*/
if (!IS_ERR(max98088->mclk)) {
if (snd_soc_component_get_bias_level(component) ==
SND_SOC_BIAS_ON)
SND_SOC_BIAS_ON) {
clk_disable_unprepare(max98088->mclk);
else
clk_prepare_enable(max98088->mclk);
} else {
ret = clk_prepare_enable(max98088->mclk);
if (ret)
return ret;
}
}
break;