diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-17 13:04:14 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2014-11-17 13:46:46 +0100 |
commit | c283661018e347bc72633969411974df8ec2ac92 (patch) | |
tree | d1259880b6ede09b75a323be99431d47fcddbf0e /sound/pci/rme9652/hdsp.c | |
parent | 6da95e1ea8e2492530eac9c51b293226e3f4ce94 (diff) | |
download | op-kernel-dev-c283661018e347bc72633969411974df8ec2ac92.zip op-kernel-dev-c283661018e347bc72633969411974df8ec2ac92.tar.gz |
ALSA: hdsp: Deletion of an unnecessary check before the function call "release_firmware"
The release_firmware() function tests whether its argument is NULL and then
return 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>
Diffstat (limited to 'sound/pci/rme9652/hdsp.c')
-rw-r--r-- | sound/pci/rme9652/hdsp.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sound/pci/rme9652/hdsp.c b/sound/pci/rme9652/hdsp.c index 2eb8baf..cf5a6c8 100644 --- a/sound/pci/rme9652/hdsp.c +++ b/sound/pci/rme9652/hdsp.c @@ -5307,8 +5307,7 @@ static int snd_hdsp_free(struct hdsp *hdsp) snd_hdsp_free_buffers(hdsp); - if (hdsp->firmware) - release_firmware(hdsp->firmware); + release_firmware(hdsp->firmware); vfree(hdsp->fw_uploaded); if (hdsp->iobase) |