Subrata Banik (subrata.banik@intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17721
-gerrit
commit 1a37955da389d75eee0ecee45d3ad5daa1d2c2ed Author: Subrata Banik subrata.banik@intel.com Date: Mon Dec 5 19:15:17 2016 +0530
soc/intel/skylake: Remove redundant BootLoaderTolumSize assignment
BootLoaderTolumSize FSP-M UPD is already initialized with cbmem_overhead_size() inside driver/fsp2_0/memory_init.c, hence remove the duplicate assignment.
Change-Id: I0b1d9769cd2a863bf0547ce5f44928cacc5a63b6 Signed-off-by: Subrata Banik subrata.banik@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)