ASoC: Intel: skl_nau88l25_ssm4567: rename shadowed variable (ddce36af) · Commits · 戴 / test

Fix cppcheck warning [sound/soc/intel/boards/skl_nau88l25_ssm4567.c:277] -> [sound/soc/intel/boards/skl_nau88l25_ssm4567.c:320]: (style) Local variable 'channels' shadows outer variable [sound/soc/intel/boards/skl_nau88l25_ssm4567.c:277] -> [sound/soc/intel/boards/skl_nau88l25_ssm4567.c:337]: (style) Local variable 'channels' shadows outer variable Signed-off-by:

Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Link: https://lore.kernel.org/r/20200113210428.27457-15-pierre-louis.bossart@linux.intel.com Signed-off-by:

Mark Brown <broonie@kernel.org>

sound/soc/intel/boards/skl_nau88l25_ssm4567.c

+5 −5

Original line number	Diff line number	Diff line
		@@ -317,13 +317,13 @@ static int skylake_ssp_fixup(struct snd_soc_pcm_runtime *rtd,
		{
		struct snd_interval *rate = hw_param_interval(params,
		SNDRV_PCM_HW_PARAM_RATE);
		struct snd_interval *channels = hw_param_interval(params,
		struct snd_interval *chan = hw_param_interval(params,
		SNDRV_PCM_HW_PARAM_CHANNELS);
		struct snd_mask *fmt = hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMAT);

		/* The ADSP will covert the FE rate to 48k, stereo */
		rate->min = rate->max = 48000;
		channels->min = channels->max = 2;
		chan->min = chan->max = 2;

		/* set SSP0 to 24 bit */
		snd_mask_none(fmt);
		@@ -334,12 +334,12 @@ static int skylake_ssp_fixup(struct snd_soc_pcm_runtime *rtd,
		static int skylake_dmic_fixup(struct snd_soc_pcm_runtime *rtd,
		struct snd_pcm_hw_params *params)
		{
		struct snd_interval *channels = hw_param_interval(params,
		struct snd_interval *chan = hw_param_interval(params,
		SNDRV_PCM_HW_PARAM_CHANNELS);
		if (params_channels(params) == 2 \|\| DMIC_CH(dmic_constraints) == 2)
		channels->min = channels->max = 2;
		chan->min = chan->max = 2;
		else
		channels->min = channels->max = 4;
		chan->min = chan->max = 4;

		return 0;
		}

Admin message