29 Aug
2020
29 Aug
'20
5:59 p.m.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/44790 )
Change subject: Add Bay Trail options ......................................................................
Patch Set 3:
(1 comment)
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
So, should I remove it? Broxton has this set to nothing […]
Hmmm, I just realized that I don't care. With Broxton, we already have a test case for the non-dyn path. And these config files are mostly for testing.
--
To view, visit https://review.coreboot.org/c/libgfxinit/+/44790
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-Change-Id: Ib30ea668da9641ccf1f94f8f25cfb897faa54aaa
Gerrit-Change-Number: 44790
Gerrit-PatchSet: 3
Gerrit-Owner: Angel Pons th3fanbus@gmail.com
Gerrit-Reviewer: Martin Roth martinroth@google.com
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com
Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Sat, 29 Aug 2020 17:59:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nico Huber nico.h@gmx.de
Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com
Gerrit-MessageType: comment