[MTD] Cleanup of 'ioremap balanced with iounmap for drivers/mtd subsystem'

Updated version of patch, in response to comments from Francois Romieu
<romieu@fr.zoreil.com>

Remove gratuitous casts from iounmap and initialisation of variables.

Signed-off-by: Amol Lad <amol@verismonetworks.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>

1 files changed, 3 insertions, 3 deletions

diff --git a/drivers/mtd/maps/beech-mtd.c b/drivers/mtd/maps/beech-mtd.c
index a64b1a5ab316..d76d5981b863 100644
--- a/drivers/mtd/maps/beech-mtd.c
+++ b/drivers/mtd/maps/beech-mtd.c
@@ -72,7 +72,7 @@ static struct mtd_partition beech_partitions[2] = {
 static int __init
 init_beech_mtd(void)
 {
-        int err = 0;
+        int err;
 
         printk("%s: 0x%08x at 0x%08x\n", NAME, SIZE, PADDR);
 
@@ -89,7 +89,7 @@ init_beech_mtd(void)
         beech_mtd = do_map_probe("cfi_probe", &beech_mtd_map);
 
         if (!beech_mtd) {
-                iounmap((void *) beech_mtd_map.virt);
+                iounmap(beech_mtd_map.virt);
                 return -ENXIO;
         }
 
@@ -98,7 +98,7 @@ init_beech_mtd(void)
         err = add_mtd_partitions(beech_mtd, beech_partitions, 2);
         if (err) {
                 printk("%s: add_mtd_partitions failed\n", NAME);
-                iounmap((void *) beech_mtd_map.virt);
+                iounmap(beech_mtd_map.virt);
         }
 
         return err;