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/mixart | |
| parent | b76c0e95ab0f535e210df588b8ac42dd75b31c75 (diff) | |
| download | op-kernel-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.zip op-kernel-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.tar.gz | |
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/mixart')
| -rw-r--r-- | sound/pci/mixart/mixart.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sound/pci/mixart/mixart.c b/sound/pci/mixart/mixart.c index 1faf47e..c3a9f39 100644 --- a/sound/pci/mixart/mixart.c +++ b/sound/pci/mixart/mixart.c @@ -1114,10 +1114,9 @@ static int snd_mixart_free(struct mixart_mgr *mgr) } /* release the i/o ports */ - for (i = 0; i < 2; i++) { - if (mgr->mem[i].virt) - iounmap(mgr->mem[i].virt); - } + for (i = 0; i < 2; ++i) + iounmap(mgr->mem[i].virt); + pci_release_regions(mgr->pci); /* free flowarray */ |
