[coreboot-gerrit] Patch set updated for coreboot: soc/intel/fsp_broadwell_de: Remove the enforced fsp1.0 APIs call sequence
York Yang (york.yang@intel.com)
gerrit at coreboot.org
Fri Oct 7 22:27:51 CEST 2016
York Yang (york.yang at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16892
-gerrit
commit 32c3a604155e23f6289065610fb740cc7be500f7
Author: York Yang <york.yang at intel.com>
Date: Wed Oct 5 09:32:46 2016 -0700
soc/intel/fsp_broadwell_de: Remove the enforced fsp1.0 APIs call sequence
The enforced FSP 1.0 APIs call was used to work around an fsp1.0 driver
issue. As the issue has been addressed in fsp1.0 driver (Change 9780),
remove the enforced workaround. Otherwise will see error message
'FSP API NotifyPhase failed' in serial log.
Verified on Intel Camelback Mountain CRB and confirmed that the serial
log error message regarding the 'FSP API NotifyPhase failed' is gone.
Change-Id: Iafa1d22e2476769fd841a3ebaa1ab4f9713c6c39
Signed-off-by: York Yang <york.yang at intel.com>
---
src/soc/intel/fsp_broadwell_de/chip.c | 12 ------------
1 file changed, 12 deletions(-)
diff --git a/src/soc/intel/fsp_broadwell_de/chip.c b/src/soc/intel/fsp_broadwell_de/chip.c
index 78649fc..b8db395 100644
--- a/src/soc/intel/fsp_broadwell_de/chip.c
+++ b/src/soc/intel/fsp_broadwell_de/chip.c
@@ -66,21 +66,9 @@ static void enable_dev(device_t dev)
}
}
-static void fsp_notify(void *arg)
-{
- FspNotify (*(uint32_t *)arg);
-}
-
-static uint32_t gFspNotifyAfterPciEnumeration = EnumInitPhaseAfterPciEnumeration;
-static uint32_t gFspNotifyReadtToBoot = EnumInitPhaseReadyToBoot;
-static BOOT_STATE_CALLBACK(bscb_fspnotify1, fsp_notify, &gFspNotifyAfterPciEnumeration);
-static BOOT_STATE_CALLBACK(bscb_fspnotify2, fsp_notify, &gFspNotifyReadtToBoot);
-
/* Called at BS_DEV_INIT_CHIPS time -- very early. Just after BS_PRE_DEVICE. */
static void soc_init(void *chip_info)
{
- boot_state_sched_on_exit(&bscb_fspnotify1, BS_DEV_RESOURCES);
- boot_state_sched_on_exit(&bscb_fspnotify2, BS_PAYLOAD_LOAD);
broadwell_de_init_pre_device();
}
More information about the coreboot-gerrit
mailing list