[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso/fsp_params.c: GOP: pass VBIOS pointer to FSP

[Martin Roth (Code Review)]

Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/51420 )

Change subject: soc/amd/picasso/fsp_params.c: GOP: pass VBIOS pointer to FSP
......................................................................

soc/amd/picasso/fsp_params.c: GOP: pass VBIOS pointer to FSP

Pass PCI_VGA_RAM_IMAGE_START as VBIOS image pointer for GOP driver.

BUG=b:171234996
BRANCH=Zork

Change-Id: I49adcacf2abb914e460fbc87b488a22dca8c8af2
Signed-off-by: Nikolai Vyssotski 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51420
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
Reviewed-by: Raul Rangel 
---
M src/soc/amd/picasso/fsp_params.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, but someone else must approve
  Raul Rangel: Looks good to me, approved

diff --git a/src/soc/amd/picasso/fsp_params.c b/src/soc/amd/picasso/fsp_params.c
index 067ce2f..57e51d5 100644
--- a/src/soc/amd/picasso/fsp_params.c
+++ b/src/soc/amd/picasso/fsp_params.c
@@ -9,6 +9,7 @@
 #include <soc/soc_util.h>
 #include <fsp/api.h>
 #include "chip.h"
+#include <device/pci.h>

static void fsps_update_emmc_config(FSP_S_CONFIG *scfg,
    			const struct soc_amd_picasso_config *cfg)
@@ -179,7 +180,7 @@

static void fsp_assign_vbios_upds(FSP_S_CONFIG *scfg)
 {
-	scfg->vbios_buffer_addr = 0;
+	scfg->vbios_buffer_addr = CONFIG(RUN_FSP_GOP) ? PCI_VGA_RAM_IMAGE_START : 0;
 }

void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd)