[coreboot-gerrit] [XS] Change in coreboot[master]: drivers/intel/gma/Kconfig: Add VBT compression configuration entry

[Arthur Heymans (Code Review)]

Attention is currently required from: Anil Kumar K, Jérémy Compostella, Subrata Banik.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76816?usp=email )

Change subject: drivers/intel/gma/Kconfig: Add VBT compression configuration entry
......................................................................

Patch Set 7:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/76816/comment/71cc246d_c00f9d27 :
PS7, Line 14: For instance, it can be set to `none' to allow the VBT binary to be
            : used in pre-memory stages such as romstage.
Why is that necessary? Compression does work pre-memory. Is that VBT binary so large it does not fit in CAR?

File src/drivers/intel/gma/Kconfig:

https://review.coreboot.org/c/coreboot/+/76816/comment/ef76696f_05a45593 :
PS7, Line 69: 
No input validation on the string? coreboot only has 2 supported compression mechanisms. Can a bool be used instead?