Re: LMB bits

[Benjamin Herrenschmidt <benh@kernel.crashing.org>]

On Sat, 2010-07-10 at 13:53 -0700, Yinghai Lu wrote:

> Ben/David, 
> 
> can you fold in following patch in your patchset too?
> 
> also please use -v2 [PATCH] lmb: rename to memblock
> that is removing one wrong change with microblaze dts

Will do. I'm also fixing another bug David found. I'll have an updated
branch later today.

Cheers,
Ben.

> Thanks
> 
> Yinghai
> 
> 
> Subject: [PATCH] memblock: memblock_find_base() should return MEMBLOCK_ERROR on failing path
> 
> all callees assume it return MEMBLOCK_ERROR when it fail to find a range
> 
> Signed-off-by: Yinghai Lu <yinghai@kernel.org>
> 
> ---
>  mm/memblock.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: linux-2.6/mm/memblock.c
> ===================================================================
> --- linux-2.6.orig/mm/memblock.c
> +++ linux-2.6/mm/memblock.c
> @@ -154,7 +154,7 @@ static phys_addr_t __init memblock_find_
>  		if (found != MEMBLOCK_ERROR)
>  			return found;
>  	}
> -	return 0;
> +	return MEMBLOCK_ERROR;
>  }
>  
>  static void memblock_remove_region(struct memblock_type *type, unsigned long r)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-arch" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html