Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33189 )
Change subject: soc/intel/common: Skip SoC GT programming based on CONFIG_SKIP_GRAPHICS_ENABLING ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/33189/1/src/soc/intel/icelake/graphics.c File src/soc/intel/icelake/graphics.c:
https://review.coreboot.org/#/c/33189/1/src/soc/intel/icelake/graphics.c@48 PS1, Line 48: DDI_A_4_LANES
It's more of enabling the software to deal with the realities of bringing up new silicon. […]
i don't understand what for you are arguing? as i have mentioned in commit msg this CL is to skip GT programming for non-GT early soc sample. coreboot selected code which we are trying to skip is touching some GPU registers which will cause hang when GPU is not enable.
Its that simple, I don't think i can able to make you understand that we are trying to get display using framebuffer only mode that i call as sw rendering rather relying on GPU registers that i called as HW acceleration (i think ppl called that the same name)