Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35242 )
Change subject: mb/siemens/mc_apl5: Disable IGD if no EDID data available ......................................................................
Patch Set 1:
Patch Set 1:
Maybe you would want to transform the PTN3460 code into a driver. The ptn3460.c file for mc_apl1, mc_apl4 and mc_apl5 only differs in a few comments and some config values. And if mc_apl5 is having issues without this patch, then mc_apl1 and mc_apl4 boards might have them as well.
Transferring the PTN code into a driver is planned for a long time already. Unfortunately fsp_baytrail does not have its i2c driver ready to be used as coreboots i2c bus (work is already ongoing) so that PTN code needs to be there at least for mc_tcu3. But we are on the way to sort it out. mc_apl1 and mc_apl4 does not have the same issue as on these boards the LCD panel is fixed to the board and therefore the right EDID data is available at any time.