diff options
| author | Lars-Peter Clausen <lars@metafoo.de> | 2015-07-26 19:04:58 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-07-29 14:01:37 +0100 |
| commit | 787126ebdb9821f1a19b1dfd1ab1bbb74b8c80b8 (patch) | |
| tree | 52452f745fd199aeb4efc3fb983c1caed92324d1 | |
| parent | 480689617510381391b3d906549477b948d9c4bc (diff) | |
| download | linux-787126ebdb9821f1a19b1dfd1ab1bbb74b8c80b8.tar.bz2 | |
ASoC: dapm: Drop always true checks
list_first_entry() always returns non NULL and since the code previously
checked that list is not empty it will also be a valid pointer. Furthermore
a path has always a sink or a source widget. So both checks are redundant
and can be removed.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/soc-dapm.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index cb4bc1cd5049..f80b7ded0721 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c @@ -3482,11 +3482,6 @@ static int snd_soc_dai_link_event(struct snd_soc_dapm_widget *w, sink_p = list_first_entry(&w->sinks, struct snd_soc_dapm_path, list_source); - if (WARN_ON(!source_p || !sink_p) || - WARN_ON(!sink_p->source || !source_p->sink) || - WARN_ON(!source_p->source || !sink_p->sink)) - return -EINVAL; - source = source_p->source->priv; sink = sink_p->sink->priv; |