[coreboot-gerrit] Patch set updated for coreboot: lib: Don't require ULZMA compression for postcar

Lee Leahy (leroy.p.leahy@intel.com) gerrit at coreboot.org
Tue Jul 26 01:56:16 CEST 2016


Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15841

-gerrit

commit 947128c6b16b10cd9c285e9487b782dfa2d8c197
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Mon Jul 25 09:53:35 2016 -0700

    lib: Don't require ULZMA compression for postcar
    
    The build fails during postcar when ULZMA compression is not selected.
    Fix cbfs.c to support LZ compression for ramstage as well.
    
    The build error is:
    build/postcar/lib/cbfs.o: In function `cbfs_load_and_decompress':
    /home/lee/coreboot/public/src/lib/cbfs.c:116: undefined reference to
    `ulzman'
    make: *** [build/cbfs/fallback/postcar.debug] Error 1
    
    TEST=Build and run on Galileo Gen2
    
    Change-Id: I7fa8ff33c0d32e0c5ff5de7918e13e6efb1df38e
    Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
---
 src/lib/cbfs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c
index aa652c2..b8575f3 100644
--- a/src/lib/cbfs.c
+++ b/src/lib/cbfs.c
@@ -104,9 +104,9 @@ size_t cbfs_load_and_decompress(const struct region_device *rdev, size_t offset,
 	case CBFS_COMPRESS_LZMA:
 		if (ENV_BOOTBLOCK || ENV_VERSTAGE)
 			return 0;
-		if (ENV_ROMSTAGE && !IS_ENABLED(CONFIG_COMPRESS_RAMSTAGE))
+		if ((ENV_ROMSTAGE || ENV_POSTCAR)
+			&& !IS_ENABLED(CONFIG_COMPRESS_RAMSTAGE))
 			return 0;
-
 		void *map = rdev_mmap(rdev, offset, in_size);
 		if (map == NULL)
 			return 0;



More information about the coreboot-gerrit mailing list