Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/44790 )
Change subject: Add Bay Trail options ......................................................................
Patch Set 3:
(5 comments)
https://review.coreboot.org/c/libgfxinit/+/44790/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/libgfxinit/+/44790/3//COMMIT_MSG@7 PS3, Line 7: Add Bay Trail options gma:
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?
https://review.coreboot.org/c/libgfxinit/+/44790/3/common/hw-gfx-gma-config.... PS3, Line 186: -- wise icon checks bay trail and says: s/.../TODO/ ?
https://review.coreboot.org/c/libgfxinit/+/44790/3/common/hw-gfx-gma-config.... PS3, Line 322: looked looked *up*?
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).