ASoC: cs530x: Correct log message with expected variable

simontrimmer · broonie · commit 248adfe32bfd · 2025-10-20T02:38:06.000+01:00
The function used one parameter for the switch statement, but logged a different parameter when it defaulted. Signed-off-by: Simon Trimmer <simont@opensource.cirrus.com> Signed-off-by: Vitaly Rodionov <vitalyr@opensource.cirrus.com> Link: https://patch.msgid.link/20251017161543.214235-2-vitalyr@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/sound/soc/codecs/cs530x.c b/sound/soc/codecs/cs530x.c
@@ -793,7 +793,7 @@ static int cs530x_set_sysclk(struct snd_soc_component *component, int clk_id,
 	case CS530X_SYSCLK_SRC_PLL:
 		break;
 	default:
-		dev_err(component->dev, "Invalid clock id %d\n", clk_id);
+		dev_err(component->dev, "Invalid sysclk source: %d\n", source);
 		return -EINVAL;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -793,7 +793,7 @@ static int cs530x_set_sysclk(struct snd_soc_component *component, int clk_id,`
`793`	`793`	`case CS530X_SYSCLK_SRC_PLL:`
`794`	`794`	`break;`
`795`	`795`	`default:`
`796`		`- dev_err(component->dev, "Invalid clock id %d\n", clk_id);`
	`796`	`+ dev_err(component->dev, "Invalid sysclk source: %d\n", source);`
`797`	`797`	`return -EINVAL;`
`798`	`798`	`}`
`799`	`799`