[coreboot-gerrit] New patch to review for coreboot: Revert "soc/intel/apollolake: Do not use StackBase FSP-M parameter"

[Brandon Breitenstein (brandon.breitenstein@intel.com)]

Brandon Breitenstein (brandon.breitenstein at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15068

-gerrit

commit c5ca8e5c5925056932682742ed40adf1bbd9cbdb
Author: Brandon Breitenstein <brandon.breitenstein at intel.com>
Date:   Sat Jun 4 00:52:21 2016 +0200

    Revert "soc/intel/apollolake: Do not use StackBase FSP-M parameter"
    
    This reverts commit 5ede3d8ccebde6f26c6b24f6458e57d99d5f3957.
    
    No longer needed due to FSP being updated to accept StackBase field
    
    Change-Id: Ic832d8dc4ca87631f5fef80d4d41558d9a72630a
---
 src/soc/intel/apollolake/romstage.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/soc/intel/apollolake/romstage.c b/src/soc/intel/apollolake/romstage.c
index da53799..8ec94a7 100644
--- a/src/soc/intel/apollolake/romstage.c
+++ b/src/soc/intel/apollolake/romstage.c
@@ -238,13 +238,9 @@ void platform_fsp_memory_init_params_cb(struct FSPM_UPD *mupd)
 	 * requests.
 	 * TODO: add checks to avoid overlap/conflict of CAR usage.
 	 */
-
-	/* fsp does not work with StackBase modified, so use default */
-#if 0
-	/* FIXME: remove this once FSP is fixed */
 	mupd->FspmArchUpd.StackBase = _car_region_end -
 					mupd->FspmArchUpd.StackSize;
-#endif
+
 	arch_upd->Bootmode = FSP_BOOT_WITH_FULL_CONFIGURATION;
 
 	if (IS_ENABLED(CONFIG_CACHE_MRC_SETTINGS)) {