[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Compress calibration blob with LZ4

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36054 )

Change subject: soc/mediatek/mt8183: Compress calibration blob with LZ4
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36054/2/src/soc/mediatek/mt8183/Mak... 
File src/soc/mediatek/mt8183/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/36054/2/src/soc/mediatek/mt8183/Mak... 
PS2, Line 87: CBFS_PRERAM_COMPRESS_FLAG
> Hmmm. When I try that last time it just failed...  […]

correction. lzma needs to fetch whole buffer first (in ARM we don't have mmap so it's read into reserved space for CBFS cache) while lz4 can be decoded using same prepared buffer.

I think we want to stick with PRERAM_COMPRESS at the moment.