ASoC: Intel: sof_pcm512x: remove unnecessary init

plbossart · plbossart · commit 2b7da56bef6b · 2022-05-20T09:30:25.000-05:00
cppcheck complains about an invalid NULL dereference but indeed there
is no need to initialize a loop variable.

Signed-off-by: Pierre-Louis Bossart &lt;pierre-louis.bossart@linux.intel.com&gt;
diff --git a/sound/soc/intel/boards/sof_pcm512x.c b/sound/soc/intel/boards/sof_pcm512x.c
@@ -419,7 +419,7 @@ static int sof_audio_probe(struct platform_device *pdev)
 static int sof_pcm512x_remove(struct platform_device *pdev)
 {
 	struct snd_soc_card *card = platform_get_drvdata(pdev);
-	struct snd_soc_component *component = NULL;
+	struct snd_soc_component *component;
 
 	for_each_card_components(card, component) {
 		if (!strcmp(component->name, pcm512x_component[0].name)) {

Original file line number	Diff line number	Diff line change
`@@ -419,7 +419,7 @@ static int sof_audio_probe(struct platform_device *pdev)`
`419`	`419`	`static int sof_pcm512x_remove(struct platform_device *pdev)`
`420`	`420`	`{`
`421`	`421`	`struct snd_soc_card *card = platform_get_drvdata(pdev);`
`422`		`- struct snd_soc_component *component = NULL;`
	`422`	`+ struct snd_soc_component *component;`
`423`	`423`
`424`	`424`	`for_each_card_components(card, component) {`
`425`	`425`	`if (!strcmp(component->name, pcm512x_component[0].name)) {`