ASoC: SOF: imx8ulp: Constify snd_sof_dsp_ops

krzk · broonie · commit 232e0da9fa77 · 2024-04-30T00:16:54.000+09:00
'struct snd_sof_dsp_ops' is not modified by core code, so it can be made const for increased code safety. Tested-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Link: https://lore.kernel.org/r/20240426-n-const-ops-var-v2-10-e553fe67ae82@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/sound/soc/sof/imx/imx8ulp.c b/sound/soc/sof/imx/imx8ulp.c
@@ -408,7 +408,7 @@ static int imx8ulp_dsp_set_power_state(struct snd_sof_dev *sdev,
 }
 
 /* i.MX8 ops */
-static struct snd_sof_dsp_ops sof_imx8ulp_ops = {
+static const struct snd_sof_dsp_ops sof_imx8ulp_ops = {
 	/* probe and remove */
 	.probe		= imx8ulp_probe,
 	.remove		= imx8ulp_remove,

Original file line number	Diff line number	Diff line change
`@@ -408,7 +408,7 @@ static int imx8ulp_dsp_set_power_state(struct snd_sof_dev *sdev,`
`408`	`408`	`}`
`409`	`409`
`410`	`410`	`/* i.MX8 ops */`
`411`		`-static struct snd_sof_dsp_ops sof_imx8ulp_ops = {`
	`411`	`+static const struct snd_sof_dsp_ops sof_imx8ulp_ops = {`
`412`	`412`	`/* probe and remove */`
`413`	`413`	`.probe = imx8ulp_probe,`
`414`	`414`	`.remove = imx8ulp_remove,`