Commit 972ea533 authored by Simon Glass's avatar Simon Glass Committed by Tom Rini

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: default avatarSimon Glass <sjg@chromium.org>
Reviewed-by: default avatarTom Rini <trini@konsulko.com>
parent 90c36d8a
...@@ -31,7 +31,7 @@ void *memalign_simple(size_t align, size_t bytes) ...@@ -31,7 +31,7 @@ void *memalign_simple(size_t align, size_t bytes)
ulong addr, new_ptr; ulong addr, new_ptr;
void *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; new_ptr = addr + bytes;
if (new_ptr > gd->malloc_limit) if (new_ptr > gd->malloc_limit)
return NULL; return NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment