staging: unisys: memregion: Use proper errno for mapit()
authorJes Sorensen <Jes.Sorensen@redhat.com>
Tue, 5 May 2015 22:36:23 +0000 (18:36 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 8 May 2015 13:26:02 +0000 (15:26 +0200)
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorutil/memregion_direct.c

index 93c10b46dc86e3df4398af814be5a20782aac11e..7ba68bcef3718114465ef22b910fcf77da3df880 100644 (file)
@@ -32,7 +32,7 @@ struct memregion {
        BOOL requested;
 };
 
-static BOOL mapit(struct memregion *memregion);
+static int mapit(struct memregion *memregion);
 static void unmapit(struct memregion *memregion);
 
 struct memregion *
@@ -47,7 +47,7 @@ visor_memregion_create(HOSTADDRESS physaddr, ulong nbytes)
 
        memregion->physaddr = physaddr;
        memregion->nbytes = nbytes;
-       if (!mapit(memregion)) {
+       if (mapit(memregion)) {
                rc = NULL;
                goto cleanup;
        }
@@ -61,19 +61,24 @@ cleanup:
 }
 EXPORT_SYMBOL_GPL(visor_memregion_create);
 
-static BOOL
+static int
 mapit(struct memregion *memregion)
 {
        ulong physaddr = (ulong)(memregion->physaddr);
        ulong nbytes = memregion->nbytes;
 
        memregion->requested = FALSE;
-       if (request_mem_region(physaddr, nbytes, MYDRVNAME))
-               memregion->requested = TRUE;
+       if (!request_mem_region(physaddr, nbytes, MYDRVNAME))
+               return -EBUSY;
+
+       memregion->requested = TRUE;
        memregion->mapped = ioremap_cache(physaddr, nbytes);
-       if (!memregion->mapped)
-               return FALSE;
-       return TRUE;
+       if (!memregion->mapped) {
+               memregion->requested = TRUE;
+               return -EFAULT;
+       }
+
+       return 0;
 }
 
 static void
@@ -114,15 +119,16 @@ EXPORT_SYMBOL_GPL(visor_memregion_get_pointer);
 int
 visor_memregion_resize(struct memregion *memregion, ulong newsize)
 {
+       int rc;
+
        if (newsize == memregion->nbytes)
                return 0;
 
        unmapit(memregion);
        memregion->nbytes = newsize;
-       if (!mapit(memregion))
-               return -EIO;
+       rc = mapit(memregion);
 
-       return 0;
+       return rc;
 }
 EXPORT_SYMBOL_GPL(visor_memregion_resize);