Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36613 )
Change subject: Kconfig: Allow x86 to compress pre-ram stages if not run XIP ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36613/1/src/Kconfig File src/Kconfig:
https://review.coreboot.org/c/coreboot/+/36613/1/src/Kconfig@1220 PS1, Line 1220: depends on NO_XIP_EARLY_STAGES
I'm not sure but doesn't this and the dependencies above turn into […]
I'm not sure it's worth the overhead in bootblock and all stages needing to carry compression code along. Check out the case statement in cbfs_load_and_decompress()(src/lib/cbfs.c) in handling this support. Have you run the size comparisons of what this does from a code and cbfs footprint? Also, there is some baked-in policy in that file for what compression algos are supported. Makefile/Kconfig need to match.