From f51c0bb090584a958f60192da35cf0f93e5038fb Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Sun, 20 Apr 2025 22:37:12 -0500 Subject: [PATCH] soc/intel/skylake: Hook up IGD config to option API MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hook up the IGD config options IgdDvmt50PreAlloc and ApertureSize to the option API, so they can be configured at runtime without recompilation. The Aperture size falls back to the FSP default value, so no change if unconfigured. Change-Id: Ic900cb7bdc6dc532aef0c0b51b32bf8308a8ae36 Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/c/coreboot/+/87392 Reviewed-by: Jérémy Compostella Tested-by: build bot (Jenkins) --- src/soc/intel/skylake/romstage/fsp_params.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/soc/intel/skylake/romstage/fsp_params.c b/src/soc/intel/skylake/romstage/fsp_params.c index 99bb5f1b6c..906301dd1f 100644 --- a/src/soc/intel/skylake/romstage/fsp_params.c +++ b/src/soc/intel/skylake/romstage/fsp_params.c @@ -116,7 +116,8 @@ static void soc_primary_gfx_config_params(FSP_M_CONFIG *m_cfg, * * If disabled, don't reserve memory for it. */ - m_cfg->IgdDvmt50PreAlloc = m_cfg->InternalGfx ? 2 : 0; + m_cfg->IgdDvmt50PreAlloc = m_cfg->InternalGfx ? get_uint_option("igd_dvmt_prealloc", 2) : 0; + m_cfg->ApertureSize = get_uint_option("igd_aperture_size", 1); m_cfg->PrimaryDisplay = config->PrimaryDisplay; }