ASoC: Intel: sof_ssp_amp: remove 'set-but-not-used' warning
Clang warning: >> sound/soc/intel/boards/sof_ssp_amp.c:97:6: warning: variable 'i' set but not used [-Wunused-but-set-variable] int i = 0; ^ The device counter is not used when the quirk is not set, which static analysis cannot know. Move its initialization before the loop to remove this warning. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com> Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com> Link: https://lore.kernel.org/r/20220421163645.319686-1-pierre-louis.bossart@linux.intel.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
d09a7db431
commit
aa4c06e02b
|
@ -94,7 +94,7 @@ static int sof_card_late_probe(struct snd_soc_card *card)
|
||||||
char jack_name[NAME_SIZE];
|
char jack_name[NAME_SIZE];
|
||||||
struct sof_hdmi_pcm *pcm;
|
struct sof_hdmi_pcm *pcm;
|
||||||
int err;
|
int err;
|
||||||
int i = 0;
|
int i;
|
||||||
|
|
||||||
if (!(sof_ssp_amp_quirk & SOF_HDMI_PLAYBACK_PRESENT))
|
if (!(sof_ssp_amp_quirk & SOF_HDMI_PLAYBACK_PRESENT))
|
||||||
return 0;
|
return 0;
|
||||||
|
@ -113,6 +113,7 @@ static int sof_card_late_probe(struct snd_soc_card *card)
|
||||||
return hda_dsp_hdmi_build_controls(card, component);
|
return hda_dsp_hdmi_build_controls(card, component);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
i = 0;
|
||||||
list_for_each_entry(pcm, &ctx->hdmi_pcm_list, head) {
|
list_for_each_entry(pcm, &ctx->hdmi_pcm_list, head) {
|
||||||
component = pcm->codec_dai->component;
|
component = pcm->codec_dai->component;
|
||||||
snprintf(jack_name, sizeof(jack_name),
|
snprintf(jack_name, sizeof(jack_name),
|
||||||
|
|
Loading…
Reference in New Issue