Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31439 )
Change subject: drivers/intel/gma: Add DP AUX interface and I2C bus ......................................................................
Patch Set 1:
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..