summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-06-28 11:15:28 +0200
committerTakashi Iwai <tiwai@suse.de>2015-06-29 11:02:36 +0200
commit1947a114bff140b35bbad1c84a5af216f8416281 (patch)
treece26e13b16daa05fa5040c8f1437d030ebc95a16
parent4275554dccdf0afac07b2b638ba7456095629558 (diff)
downloadlinux-1947a114bff140b35bbad1c84a5af216f8416281.tar.gz
linux-1947a114bff140b35bbad1c84a5af216f8416281.tar.bz2
linux-1947a114bff140b35bbad1c84a5af216f8416281.zip
ALSA: hda: Delete an unnecessary check before the function call "snd_info_free_entry"
The snd_info_free_entry() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r--sound/pci/hda/patch_hdmi.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
index f8527342a150..2f2433845d04 100644
--- a/sound/pci/hda/patch_hdmi.c
+++ b/sound/pci/hda/patch_hdmi.c
@@ -591,7 +591,7 @@ static int eld_proc_new(struct hdmi_spec_per_pin *per_pin, int index)
static void eld_proc_free(struct hdmi_spec_per_pin *per_pin)
{
- if (!per_pin->codec->bus->shutdown && per_pin->proc_entry) {
+ if (!per_pin->codec->bus->shutdown) {
snd_info_free_entry(per_pin->proc_entry);
per_pin->proc_entry = NULL;
}