diff options
| author |   Markus Elfring <elfring@users.sourceforge.net> | 2015-01-03 22:55:54 +0100 |
|---|---|---|
| committer |   Takashi Iwai <tiwai@suse.de> | 2015-01-04 15:13:45 +0100 |
| commit | ff6defa6a8fae12205d64f55db395b1fcf35af8e (patch) | |
| tree | a08b5c63a0c7e21f955e254178dc53ea0d7b6e3f /sound/pci/aw2 | |
| parent | ALSA: sb: Delete an unnecessary check before the function call "snd_emux_free" (diff) | |
| download | linux-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.tar.xz linux-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.zip | |
ALSA: Deletion of checks before the function call "iounmap"
The iounmap() function performs also input parameter validation.
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/aw2')
| -rw-r--r-- | sound/pci/aw2/aw2-alsa.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sound/pci/aw2/aw2-alsa.c b/sound/pci/aw2/aw2-alsa.c index e1cf01949fda..8d2fee7b33bd 100644 --- a/sound/pci/aw2/aw2-alsa.c +++ b/sound/pci/aw2/aw2-alsa.c @@ -229,9 +229,7 @@ static int snd_aw2_dev_free(struct snd_device *device) if (chip->irq >= 0) free_irq(chip->irq, (void *)chip); /* release the i/o ports & memory */ - if (chip->iobase_virt) - iounmap(chip->iobase_virt); - + iounmap(chip->iobase_virt); pci_release_regions(chip->pci); /* disable the PCI entry */ pci_disable_device(chip->pci); |