Patch set 4:

nobody else seems to enable this for default config so this raises some eyebrows here.

Maybe that's because the default config is using SeaBIOS and not GRUB.
Personally I have not observed any differences while using this framebuffer 118h VESA option together with SeaBIOS. But this change could benefit those G505S owners who would like to use GRUB as the payload, so could be useful for some.


> Your arguments sound as if this should be more globally enabled on default builds.

I think it should be decided for each board individually (don't know if this option could be as good/not-harmful for them, needs to be tested by those who are interested)

View Change

To view, visit change 31325. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0b4eacf61d905f1160531a988e192c3b626dca68
Gerrit-Change-Number: 31325
Gerrit-PatchSet: 4
Gerrit-Owner: mikeb mikeb <mikebdp2@gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Mike Banon <mikebdp2@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: mikeb mikeb <mikebdp2@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Wed, 27 Feb 2019 16:04:25 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment