[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