Change in ...coreboot[master]: lenovo/x230: introduce FHD variant

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28950 )

Change subject: lenovo/x230: introduce FHD variant
......................................................................


Patch Set 6: Code-Review-1
> Patch Set 6:
> > Patch Set 6: Code-Review+1
> >
> > looks good to me, but should be tested again before i give a +2
>
> I have build current master (edbf5d91) with patch set 6 applied for both a normal x230 and x230_fhd.
> I have tested the following setup: coreboot -> seabios -> grub on both devices.
>
> The normal x230 works as expected.
>
> On the device with FHD-mod booting works as well, but graphical output of coreboot, seabios and grub is only visible if an additional external monitor is attached via VGA or DP. The internal FHD panel becomes active as soon as the linux kernel takes over. Right now I am unsure if this is due to misconfiguration on my part or if coreboot/libgfxinit needs to be told to use eDP1 for output on modded devices. Hints for further tests appreciated.
>
> Conclusion: Patch seems stable to me on default and modded x230 Thinkpads but has room for improvement on modded devices

What you described for the FHD mod is "libgfxinit is not initializing the internal panel at all", which is a regression. Could you please investigate on this matter, or at least provide a coreboot log? Thanks!


-- 
To view, visit https://review.coreboot.org/c/coreboot/+/28950
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0355d39a61956792e69bccd5274cfc2749d72bf0
Gerrit-Change-Number: 28950
Gerrit-PatchSet: 6
Gerrit-Owner: Alexander Couzens <[e-mail]>
Gerrit-Reviewer: Alexander Couzens <[e-mail]>
Gerrit-Reviewer: Angel Pons <[e-mail]>
Gerrit-Reviewer: Arthur Heymans <[e-mail]>
Gerrit-Reviewer: Felix Held <[e-mail]>
Gerrit-Reviewer: Martin Roth <[e-mail]>
Gerrit-Reviewer: Nico Huber <[e-mail]>
Gerrit-Reviewer: Patrick Georgi <[e-mail]>
Gerrit-Reviewer: Patrick Rudolph <[e-mail]>
Gerrit-Reviewer: Paul Menzel <[e-mail]>
Gerrit-Reviewer: Richard Slindee
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-CC: Kilian Neuner <[e-mail]>
Gerrit-CC: Sergey Alirzaev <[e-mail]>
Gerrit-CC: Tom Hiller <[e-mail]>
Gerrit-Comment-Date: Wed, 23 Jan 2019 13:15:05 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment