diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-17 07:04:14 -0500 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2014-11-17 07:46:46 -0500 |
| commit | c283661018e347bc72633969411974df8ec2ac92 (patch) | |
| tree | d1259880b6ede09b75a323be99431d47fcddbf0e /sound/pci | |
| parent | 6da95e1ea8e2492530eac9c51b293226e3f4ce94 (diff) | |
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')
| -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) | |||
| 5307 | 5307 | ||
| 5308 | snd_hdsp_free_buffers(hdsp); | 5308 | snd_hdsp_free_buffers(hdsp); |
| 5309 | 5309 | ||
| 5310 | if (hdsp->firmware) | 5310 | release_firmware(hdsp->firmware); |
| 5311 | release_firmware(hdsp->firmware); | ||
| 5312 | vfree(hdsp->fw_uploaded); | 5311 | vfree(hdsp->fw_uploaded); |
| 5313 | 5312 | ||
| 5314 | if (hdsp->iobase) | 5313 | if (hdsp->iobase) |