Patrick Rudolph has uploaded a new patch set (#10) to the change originally created by Christian Walter. ( https://review.coreboot.org/c/coreboot/+/39454 )
Change subject: soc/intel/cannonlake: Steal no memory for disabled IGD ......................................................................
soc/intel/cannonlake: Steal no memory for disabled IGD
Set IgdDvmt50PreAlloc to zero if InternalGfx is disabled. It is 'correct' to do it like this, otherwise the FSP would always allocate memory for the IGD even if it is disabled. In addition the FSP enables the graphics panel even if no IGD is present which leads to a crashing FSP. Thus, if no IGD is present we switch of the panel via UPDs.
Also apply a workaround to prevent a crash, see https://github.com/IntelFsp/FSP/issues/49 for details.
Tested on: * CFL platform with IGD * CFL platform without IGD
Change-Id: I6f9e0f9855224614471d8ed23bf2a9786386ddca Signed-off-by: Christian Walter christian.walter@9elements.com --- M Documentation/soc/intel/fsp/index.md M src/soc/intel/cannonlake/romstage/fsp_params.c 2 files changed, 27 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/39454/10