[M] Change in coreboot[master]: mb/prodrive/hermes: Allow using the Intel iGPU as primary
Attention is currently required from: Arthur Heymans, Patrick Rudolph, Christian Walter.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62649 )
Change subject: mb/prodrive/hermes: Allow using the Intel iGPU as primary ......................................................................
Patch Set 7:
(3 comments)
File src/mainboard/prodrive/hermes/mainboard.c:
https://review.coreboot.org/c/coreboot/+/62649/comment/cabc7cb7_7eed5969 PS1, Line 262: internale_gfx
There's a typo, and enum type is unused anyway. […]
Done
https://review.coreboot.org/c/coreboot/+/62649/comment/365601d3_b4767b0e PS1, Line 280: mainboard_configure_internal_gfx
Why not call this from `mainboard_early()`?
To answer myself: the right place to call this from is when entering BS_DEV_RESOURCES.
https://review.coreboot.org/c/coreboot/+/62649/comment/60de5580_19077bbe PS1, Line 283: board_cfg
This can be NULL.
Done
-
Angel Pons (Code Review)