From eff96e99588f9aad8325bcb5206c21b3075c7634 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Fri, 14 Aug 2015 13:26:43 -0600 Subject: [PATCH] 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 --- common/malloc_simple.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; -- 2.39.2