Commit 4f6136fa authored Feb 03, 2014 by Lars-Peter Clausen Committed by Mark Brown Feb 03, 2014

ASoC: rx1950_uda1380: Don't modify runtime->hw

Individual components are not supposed to modify the runtime->hw struct itself
as it is manged by the ASoC core. If a component wants to limit the list of
supported rates it should set a rate constraint. The rx1950_uda1380 already does
this, so it is safe to just remove the code that modifies runtime->hw.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Acked-by: Vasily Khoruzhick <anarsoul@gmail.com>
Signed-off-by: Mark Brown <broonie@linaro.org>

parent 1659b843

sound/soc/samsung/rx1950_uda1380.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -131,10 +131,6 @@ static int rx1950_startup(struct snd_pcm_substream *substream)
		{
		struct snd_pcm_runtime *runtime = substream->runtime;

		runtime->hw.rate_min = hw_rates.list[0];
		runtime->hw.rate_max = hw_rates.list[hw_rates.count - 1];
		runtime->hw.rates = SNDRV_PCM_RATE_KNOT;

		return snd_pcm_hw_constraint_list(runtime, 0,
		SNDRV_PCM_HW_PARAM_RATE,
		&hw_rates);

Admin message