summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: d1e730a)
raw | patch | inline | side by side (parent: d1e730a)
author | Dan Carpenter <dan.carpenter@oracle.com> | |
Sat, 25 Jul 2015 00:03:38 +0000 (03:03 +0300) | ||
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
Mon, 17 Aug 2015 03:51:50 +0000 (20:51 -0700) |
commit 44008f0896ae205b02b0882dbf807f0de149efc4 upstream.
Smatch complains that we have nested checks for "spdif_present". It
turns out the current behavior isn't correct, we should remove the first
check and keep the second.
Fixes: 1077a024812d ('ALSA: hda - Use generic parser for Cirrus codec driver')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Smatch complains that we have nested checks for "spdif_present". It
turns out the current behavior isn't correct, we should remove the first
check and keep the second.
Fixes: 1077a024812d ('ALSA: hda - Use generic parser for Cirrus codec driver')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/pci/hda/patch_cirrus.c | patch | blob | history |
index 51e208022cc81e2885282ec8b43412a2d7b903b3..7b0aac9d27ca9be329e583e2739c53d1927a5b92 100644 (file)
spec->spdif_present = spdif_present;
/* SPDIF TX on/off */
- if (spdif_present)
- snd_hda_set_pin_ctl(codec, spdif_pin,
- spdif_present ? PIN_OUT : 0);
+ snd_hda_set_pin_ctl(codec, spdif_pin, spdif_present ? PIN_OUT : 0);
cs_automute(codec);
}