The function os_free() returns nothing. Its return type should be "void" rather than "void *".
Signed-off-by: Masahiro Yamada <yamad...@jp.panasonic.com> --- Simon, This commit fixes a warning: u-boot/include/os.h: In function ‘os_free’: arch/sandbox/cpu/os.c:153:1: warning: control reaches end of non-void function [-Wreturn-type] But, this warning message is not displayed on the current u-boot/master because commit cbe5cdfcd3 changed arch/sandbox/cpu/Makefile badly. The build rule in arch/sandbox/cpu/Makefile $(obj)os.o: ALL_CFLAGS := $(BASE_CPPFLAGS) \ $(patsubst %, -idirafter %, $(BASE_INCLUDE_DIRS)) drops many flags including "-Wall". That's why you did not notice such a mistake, which the build system should have detected. This problem can be fixed in my patch http://patchwork.ozlabs.org/patch/294233/ (It is also a prerequisite for my Kbuild series.) arch/sandbox/cpu/os.c | 2 +- include/os.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/sandbox/cpu/os.c b/arch/sandbox/cpu/os.c index 725b505..2e2fc58 100644 --- a/arch/sandbox/cpu/os.c +++ b/arch/sandbox/cpu/os.c @@ -143,7 +143,7 @@ void *os_malloc(size_t length) return hdr + 1; } -void *os_free(void *ptr) +void os_free(void *ptr) { struct os_mem_hdr *hdr = ptr; diff --git a/include/os.h b/include/os.h index b65fba4..d6d6e57 100644 --- a/include/os.h +++ b/include/os.h @@ -113,7 +113,7 @@ void *os_malloc(size_t length); * * \param ptr Pointer to memory block to free */ -void *os_free(void *ptr); +void os_free(void *ptr); /** * Reallocate previously-allocated memory to increase/decrease space -- 1.8.3.2 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot