memblock: Define MEMBLOCK_ERROR internally instead of using ~(phys_addr_t)0
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 6 Jul 2010 22:39:09 +0000 (15:39 -0700)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 5 Aug 2010 02:56:16 +0000 (12:56 +1000)
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
mm/memblock.c

index c1d2060e213eface023727e6e71090a4d50d6028..fc7f97b2d9941937feccc94b9970d2846e5e75b6 100644 (file)
@@ -22,6 +22,8 @@ static int memblock_debug;
 static struct memblock_region memblock_memory_init_regions[INIT_MEMBLOCK_REGIONS + 1];
 static struct memblock_region memblock_reserved_init_regions[INIT_MEMBLOCK_REGIONS + 1];
 
+#define MEMBLOCK_ERROR (~(phys_addr_t)0)
+
 static int __init early_memblock(char *p)
 {
        if (p && strstr(p, "debug"))
@@ -326,7 +328,7 @@ static phys_addr_t __init memblock_find_region(phys_addr_t start, phys_addr_t en
                base = memblock_align_down(res_base - size, align);
        }
 
-       return ~(phys_addr_t)0;
+       return MEMBLOCK_ERROR;
 }
 
 phys_addr_t __weak __init memblock_nid_range(phys_addr_t start, phys_addr_t end, int *nid)
@@ -353,14 +355,14 @@ static phys_addr_t __init memblock_alloc_nid_region(struct memblock_region *mp,
                this_end = memblock_nid_range(start, end, &this_nid);
                if (this_nid == nid) {
                        phys_addr_t ret = memblock_find_region(start, this_end, size, align);
-                       if (ret != ~(phys_addr_t)0 &&
+                       if (ret != MEMBLOCK_ERROR &&
                            memblock_add_region(&memblock.reserved, ret, size) >= 0)
                                return ret;
                }
                start = this_end;
        }
 
-       return ~(phys_addr_t)0;
+       return MEMBLOCK_ERROR;
 }
 
 phys_addr_t __init memblock_alloc_nid(phys_addr_t size, phys_addr_t align, int nid)
@@ -379,7 +381,7 @@ phys_addr_t __init memblock_alloc_nid(phys_addr_t size, phys_addr_t align, int n
        for (i = 0; i < mem->cnt; i++) {
                phys_addr_t ret = memblock_alloc_nid_region(&mem->regions[i],
                                               size, align, nid);
-               if (ret != ~(phys_addr_t)0)
+               if (ret != MEMBLOCK_ERROR)
                        return ret;
        }
 
@@ -430,7 +432,7 @@ phys_addr_t __init __memblock_alloc_base(phys_addr_t size, phys_addr_t align, ph
                        continue;
                base = min(memblockbase + memblocksize, max_addr);
                res_base = memblock_find_region(memblockbase, base, size, align);
-               if (res_base != ~(phys_addr_t)0 &&
+               if (res_base != MEMBLOCK_ERROR &&
                    memblock_add_region(&memblock.reserved, res_base, size) >= 0)
                        return res_base;
        }