m68k/amiga: Chip RAM - Use lookup_resource()
authorGeert Uytterhoeven <geert@linux-m68k.org>
Sat, 7 May 2011 18:56:00 +0000 (20:56 +0200)
committerGeert Uytterhoeven <geert@linux-m68k.org>
Sat, 30 Jul 2011 19:21:39 +0000 (21:21 +0200)
Replace a custom implementation (which doesn't lock the resource tree) by a
call to lookup_resource()

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
arch/m68k/amiga/chipram.c

index 4790f77cbd486aedce9b0db34c43914cc5c0cbc0..99449fbf9a726834ec2b2c268956a13f0ab0b99d 100644 (file)
@@ -93,21 +93,21 @@ void *amiga_chip_alloc_res(unsigned long size, struct resource *res)
 void amiga_chip_free(void *ptr)
 {
        unsigned long start = ZTWO_PADDR(ptr);
-       struct resource **p, *res;
+       struct resource *res;
        unsigned long size;
 
-       for (p = &chipram_res.child; (res = *p); p = &res->sibling) {
-               if (res->start != start)
-                       continue;
-               *p = res->sibling;
-               size = resource_size(res);
-               pr_debug("amiga_chip_free: free %lu bytes at %p\n", size, ptr);
-               atomic_add(size, &chipavail);
-               kfree(res);
+       res = lookup_resource(&chipram_res, start);
+       if (!res) {
+               pr_err("amiga_chip_free: trying to free nonexistent region at "
+                      "%p\n", ptr);
                return;
        }
-       pr_err("amiga_chip_free: trying to free nonexistent region at %p\n",
-              ptr);
+
+       size = resource_size(res);
+       pr_debug("amiga_chip_free: free %lu bytes at %p\n", size, ptr);
+       atomic_add(size, &chipavail);
+       release_resource(res);
+       kfree(res);
 }
 EXPORT_SYMBOL(amiga_chip_free);