From: Simon Glass Date: Fri, 14 Aug 2015 19:26:43 +0000 (-0600) Subject: malloc_simple: Correct the alignment logic in memalign_simple() X-Git-Url: http://git.dujemihanovic.xyz/img/sics.gif?a=commitdiff_plain;h=972ea5339006e12a6c5d79f8c263ff5d1f463b30;p=u-boot.git malloc_simple: Correct the alignment logic in memalign_simple() This should use the align parameter, not bytes. Natural alignment is one use case but should not be the only one supported by this function. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- diff --git a/common/malloc_simple.c b/common/malloc_simple.c index 9811ab60f6..134e059706 100644 --- a/common/malloc_simple.c +++ b/common/malloc_simple.c @@ -31,7 +31,7 @@ void *memalign_simple(size_t align, size_t bytes) ulong addr, new_ptr; void *ptr; - addr = ALIGN(gd->malloc_base + gd->malloc_ptr, bytes); + addr = ALIGN(gd->malloc_base + gd->malloc_ptr, align); new_ptr = addr + bytes; if (new_ptr > gd->malloc_limit) return NULL;