sh: Make iounmap_fixed() return success/failure for iounmap() path.
authorPaul Mundt <lethal@linux-sh.org>
Mon, 18 Jan 2010 12:30:29 +0000 (21:30 +0900)
committerPaul Mundt <lethal@linux-sh.org>
Mon, 18 Jan 2010 12:30:29 +0000 (21:30 +0900)
This converts iounmap_fixed() to return success/error if it handled the
unmap request or not. At the same time, drop the __init label, as this
can be called in to later.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>
arch/sh/include/asm/io.h
arch/sh/mm/ioremap_fixed.c

index 9aa9438c66c3c1dde7eb50ac418a16d214cd0131..6a0dd8c1e0a909155f781ca5b96ef8838526ceff 100644 (file)
@@ -22,6 +22,7 @@
  * for old compat code for I/O offseting to SuperIOs, all of which are
  * better handled through the machvec ioport mapping routines these days.
  */
+#include <linux/errno.h>
 #include <asm/cache.h>
 #include <asm/system.h>
 #include <asm/addrspace.h>
@@ -239,7 +240,7 @@ void __iounmap(void __iomem *addr);
 
 #ifdef CONFIG_IOREMAP_FIXED
 extern void __iomem *ioremap_fixed(resource_size_t, unsigned long, pgprot_t);
-extern void iounmap_fixed(void __iomem *);
+extern int iounmap_fixed(void __iomem *);
 extern void ioremap_fixed_init(void);
 #else
 static inline void __iomem *
@@ -249,7 +250,7 @@ ioremap_fixed(resource_size t phys_addr, unsigned long size, pgprot_t prot)
 }
 
 static inline void ioremap_fixed_init(void) { }
-static inline void iounmap_fixed(void __iomem *addr) { }
+static inline int iounmap_fixed(void __iomem *addr) { return -EINVAL; }
 #endif
 
 static inline void __iomem *
index 3a9d3d88fe8d0eb7ccf950e15bab29e9a5e3d09b..425f6c6bf2509e521486d310b0ae811d49e45208 100644 (file)
@@ -103,7 +103,7 @@ ioremap_fixed(resource_size_t phys_addr, unsigned long size, pgprot_t prot)
        return map->addr;
 }
 
-void __init iounmap_fixed(void __iomem *addr)
+int iounmap_fixed(void __iomem *addr)
 {
        enum fixed_addresses idx;
        unsigned long virt_addr;
@@ -122,8 +122,11 @@ void __init iounmap_fixed(void __iomem *addr)
                }
        }
 
+       /*
+        * If we don't match, it's not for us.
+        */
        if (slot < 0)
-               return;
+               return -EINVAL;
 
        virt_addr = (unsigned long)addr;
 
@@ -141,4 +144,6 @@ void __init iounmap_fixed(void __iomem *addr)
 
        map->size = 0;
        map->addr = NULL;
+
+       return 0;
 }