Patch Set 1:

(3 comments)

There is thorough functionality in libgfxinit. If it's
only about fetching the EDID, we'd just have to export
one more function. Let me know if you want to give it
a try.

Ahh I was looking in the gma directory and could not find anything useful (except some really old code in the link mainboard dir) but didn't dig in to the libgfxinit repo. There does seem to be some support there and I'm certainly open to using it.

The current implementation here does what is required
to talk to Intel hardware. Though, it seems that most
of the AUX transaction handling specified by Display-
Port is missing.

Is this tested? with external displays?

Yes it is very limited, tested only with eDP since that is the only power well that FSP turns on by default.

My goal here is just to get the panel EDID and replace the ugly hack in the Sarien board where I have incompatible touchscreen devices at the same I2C address and want to use a lookup table to enable the appropriate driver.

I'll put this commit on hold while I check out libgfxinit..

View Change

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4aa7e687aea397d6d70c0398e0f6958611db6410
Gerrit-Change-Number: 31439
Gerrit-PatchSet: 1
Gerrit-Owner: Duncan Laurie <dlaurie@chromium.org>
Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-Comment-Date: Sat, 16 Feb 2019 17:18:19 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment