[coreboot] New patch to review for coreboot: df4917d Remove unused free() function

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Mon Jul 23 23:20:41 CEST 2012


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1274

-gerrit

commit df4917d5d02ea8636c6c20ee1a312301712e7406
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Wed Jun 6 13:42:09 2012 -0700

    Remove unused free() function
    
    Since coreboot is running very short, we don't free memory.
    Hence, drop (dummy) free()
    
    Change-Id: I6e2737f07c6b9f73ebfad7d124b97a57cb7454a3
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 src/include/stdlib.h |    1 -
 src/lib/malloc.c     |    6 ------
 2 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/src/include/stdlib.h b/src/include/stdlib.h
index bd0ee30..bb57ab3 100644
--- a/src/include/stdlib.h
+++ b/src/include/stdlib.h
@@ -14,7 +14,6 @@
 #if !defined(__PRE_RAM__)
 void *memalign(size_t boundary, size_t size);
 void *malloc(size_t size);
-void free(void *ptr);
 #endif
 
 #endif /* STDLIB_H */
diff --git a/src/lib/malloc.c b/src/lib/malloc.c
index 4115738..bf8a15a 100644
--- a/src/lib/malloc.c
+++ b/src/lib/malloc.c
@@ -49,9 +49,3 @@ void *malloc(size_t size)
 {
 	return memalign(sizeof(u64), size);
 }
-
-void free(void *where)
-{
-	/* Don't care */
-	MALLOCDBG("free %p\n", where);
-}




More information about the coreboot mailing list