Unverified Commit f6cd55a1 authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Mark Brown
Browse files

ASoC: Intel: atom: controls: remove useless initializations 



cppcheck complains about invalid NULL dereferences but there's indeed
no need to initialize loop variables or before allocating memory.

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: default avatarPéter Ujfalusi <peter.ujfalusi@linux.intel.com>
Link: https://lore.kernel.org/r/20220520211719.607543-9-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent a140785b

sound/soc/intel/atom/sst-atom-controls.c

+2 −2
Original line number Diff line number Diff line
@@ -1328,7 +1328,7 @@ int sst_send_pipe_gains(struct snd_soc_dai *dai, int stream, int mute) 
{

	struct sst_data *drv = snd_soc_dai_get_drvdata(dai);

	struct snd_soc_dapm_widget *w;

	struct snd_soc_dapm_path *p = NULL;

	struct snd_soc_dapm_path *p;



	dev_dbg(dai->dev, "enter, dai-name=%s dir=%d\n", dai->name, stream);
@@ -1392,7 +1392,7 @@ int sst_send_pipe_gains(struct snd_soc_dai *dai, int stream, int mute) 
static int sst_fill_module_list(struct snd_kcontrol *kctl,

	 struct snd_soc_dapm_widget *w, int type)

{

	struct sst_module *module = NULL;

	struct sst_module *module;

	struct snd_soc_component *c = snd_soc_dapm_to_component(w->dapm);

	struct sst_ids *ids = w->priv;

	int ret = 0;