Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/44790 )
Change subject: Add Bay Trail options ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/libgfxinit/+/44790/3/common/hw-gfx-gma-config.... File common/hw-gfx-gma-config.ads.template:
https://review.coreboot.org/c/libgfxinit/+/44790/3/common/hw-gfx-gma-config.... PS3, Line 166: -- VLV: False -- c.f. EDS or similar
Time to strip these comments?
Oops, true.
https://review.coreboot.org/c/libgfxinit/+/44790/3/configs/baytrail File configs/baytrail:
https://review.coreboot.org/c/libgfxinit/+/44790/3/configs/baytrail@1 PS3, Line 1: CONFIG_GFX_GMA_DYN_CPU = y
Only useful when a generation supports more than one CPU/SoC (yet).
So, should I remove it? Broxton has this set to nothing
I would also need to specify the CPU here then