[coreboot-gerrit] New patch to review for coreboot: soc/intel/skylake: Remove redundent BootLoaderTolumSize assignment

Subrata Banik (subrata.banik@intel.com) gerrit at coreboot.org
Mon Dec 5 14:54:15 CET 2016


Subrata Banik (subrata.banik at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17721

-gerrit

commit 18aac76a90861e2e0f21e55f8abdc4e3229e5d35
Author: Subrata Banik <subrata.banik at intel.com>
Date:   Mon Dec 5 19:15:17 2016 +0530

    soc/intel/skylake: Remove redundent BootLoaderTolumSize assignment
    
    BootLoaderTolumSize FSP-M UPD already initialized with cbmem_overhead_size()
    inside driver/fsp2_0/memory_init.c, hence removing duplicate assignment.
    
    Change-Id: I0b1d9769cd2a863bf0547ce5f44928cacc5a63b6
    Signed-off-by: Subrata Banik <subrata.banik at intel.com>
---
 src/soc/intel/skylake/romstage/romstage_fsp20.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/soc/intel/skylake/romstage/romstage_fsp20.c b/src/soc/intel/skylake/romstage/romstage_fsp20.c
index 1132577..73d726c 100644
--- a/src/soc/intel/skylake/romstage/romstage_fsp20.c
+++ b/src/soc/intel/skylake/romstage/romstage_fsp20.c
@@ -163,10 +163,6 @@ void platform_fsp_memory_init_params_cb(FSPM_UPD *mupd, uint32_t version)
 	m_t_cfg->DidInitStat = 0x01;
 
 	mainboard_memory_init_params(mupd);
-
-	/* Reserve enough memory under TOLUD to save CBMEM header */
-	mupd->FspmArchUpd.BootLoaderTolumSize = cbmem_overhead_size();
-
 }
 
 __attribute__((weak)) void mainboard_memory_init_params(FSPM_UPD *mupd)



More information about the coreboot-gerrit mailing list