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 | ALSA: lola: Deletion of an unnecessary check before the function call "vfree" (diff) | |
| download | linux-dev-c283661018e347bc72633969411974df8ec2ac92.tar.xz linux-dev-c283661018e347bc72633969411974df8ec2ac92.zip | |
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 '')
| -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 2eb8baf7b828..cf5a6c8b9a63 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) |