From eabfd3a7c162d5eb96f65085ad8b05238e53a437 Mon Sep 17 00:00:00 2001 From: Andrew Wu Date: Mon, 19 Aug 2013 11:43:36 +0800 Subject: [PATCH] Don't include LZMA in romstage if ramstage is not compressed. If ramstage is not compressed, the CBFS module in romstage doesn't need to support LZMA. Removing the LZMA module in this case can save about 3000 bytes in romstage. Change-Id: Id6f7869e32979080e2985c07029edcb39eee9106 Signed-off-by: Andrew Wu Reviewed-on: http://review.coreboot.org/3878 Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi --- src/lib/Makefile.inc | 2 +- src/lib/cbfs.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/lib/Makefile.inc b/src/lib/Makefile.inc index c42103810e..90efe6c79b 100644 --- a/src/lib/Makefile.inc +++ b/src/lib/Makefile.inc @@ -42,7 +42,7 @@ endif romstage-y += memcmp.c rmodules-y += memcmp.c romstage-y += cbfs.c -romstage-y += lzma.c +romstage-$(CONFIG_COMPRESS_RAMSTAGE) += lzma.c #romstage-y += lzmadecode.c romstage-$(CONFIG_CACHE_AS_RAM) += ramtest.c romstage-$(CONFIG_CONSOLE_SERIAL8250) += uart8250.c diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c index f48d887d87..4ccc9e6c30 100644 --- a/src/lib/cbfs.c +++ b/src/lib/cbfs.c @@ -29,6 +29,8 @@ # define CBFS_MINI_BUILD #elif defined(__BOOT_BLOCK__) /* No LZMA in boot block. */ +#elif defined(__PRE_RAM__) && !CONFIG_COMPRESS_RAMSTAGE + /* No LZMA in romstage if ramstage is not compressed. */ #else # define CBFS_CORE_WITH_LZMA # include