diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-20 07:50:43 -0500 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-11-26 01:44:10 -0500 |
commit | 05a221bb1f49e6eebc9a3858cb45506f403b3ab6 (patch) | |
tree | aa85b86eb0cc5d7aca9e6da39394157d8491f87c /drivers/mtd/chips | |
parent | 775a9134f4398ca98a10af8cc3cf9b664017267f (diff) |
mtd: delete unnecessary checks before two function calls
The functions kfree() and pci_dev_put() test whether their 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: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd/chips')
-rw-r--r-- | drivers/mtd/chips/cfi_cmdset_0001.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mtd/chips/cfi_cmdset_0001.c b/drivers/mtd/chips/cfi_cmdset_0001.c index 3096f3ded3ad..286b97a304cf 100644 --- a/drivers/mtd/chips/cfi_cmdset_0001.c +++ b/drivers/mtd/chips/cfi_cmdset_0001.c | |||
@@ -2654,8 +2654,7 @@ static void cfi_intelext_destroy(struct mtd_info *mtd) | |||
2654 | kfree(cfi); | 2654 | kfree(cfi); |
2655 | for (i = 0; i < mtd->numeraseregions; i++) { | 2655 | for (i = 0; i < mtd->numeraseregions; i++) { |
2656 | region = &mtd->eraseregions[i]; | 2656 | region = &mtd->eraseregions[i]; |
2657 | if (region->lockmap) | 2657 | kfree(region->lockmap); |
2658 | kfree(region->lockmap); | ||
2659 | } | 2658 | } |
2660 | kfree(mtd->eraseregions); | 2659 | kfree(mtd->eraseregions); |
2661 | } | 2660 | } |