Attention is currently required from: Arthur Heymans, Kapil Porwal, Lean Sheng Tan, Nick Vaccaro, Subrata Banik, Werner Zeh.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78080?usp=email )
Change subject: soc/intel/cmn/gfx: Add API to report presence of external display ......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/common/block/graphics/graphics.c:
https://review.coreboot.org/c/coreboot/+/78080/comment/7fd2a0f2_82427b07 : PS1, Line 59: #if CONFIG(REPORT_EXTERNAL_DISPLAY_AT_PREBOOT)
Are there really platforms using this file for which this code doesn't work? Otherwise you should […]
So you're saying that those other platforms would return 0 from report_external_display() (because they would always initialize the eDP display), which is the same as the stub function you implemented for when the Kconfig is disabled? Sounds like that would work fine to me. The point is to just reduce the number of Kconfig options and special cases to keep track of, to make the code easier to follow. Especially if this only affects older boards and we're planning to have the external display initialization in that case for all future devices, I don't think there's a problem with making some older platforms read a few pointless (but harmless) registers, if it helps simplify things a bit.