[coreboot-gerrit] New patch to review for coreboot: intel/skylake: Remove fsp_reset handling in soc/
Furquan Shaikh (furquan@google.com)
gerrit at coreboot.org
Fri Jul 22 18:36:44 CEST 2016
Furquan Shaikh (furquan at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15805
-gerrit
commit c69e71b6e4abd3823a1ce8fa452948581403b5b3
Author: Furquan Shaikh <furquan at google.com>
Date: Fri Jul 22 09:23:26 2016 -0700
intel/skylake: Remove fsp_reset handling in soc/
With recovery module in vboot handling the saving of recovery reason
across reboots, there is no need to have special fsp reset handling
under soc.
BUG=chrome-os-partner:55431
Change-Id: Iefe51d3379f718b27e9d600cb61b2aa1f2473e00
Signed-off-by: Furquan Shaikh <furquan at google.com>
---
src/soc/intel/skylake/Makefile.inc | 1 -
src/soc/intel/skylake/fsp_reset.c | 62 --------------------------------------
2 files changed, 63 deletions(-)
diff --git a/src/soc/intel/skylake/Makefile.inc b/src/soc/intel/skylake/Makefile.inc
index ea1895d..3928b97 100644
--- a/src/soc/intel/skylake/Makefile.inc
+++ b/src/soc/intel/skylake/Makefile.inc
@@ -38,7 +38,6 @@ ramstage-y += dsp.c
ramstage-y += elog.c
ramstage-y += finalize.c
ramstage-y += flash_controller.c
-ramstage-$(CONFIG_VBOOT_VERIFY_FIRMWARE) += fsp_reset.c
ramstage-y += gpio.c
ramstage-y += i2c.c
ramstage-y += igd.c
diff --git a/src/soc/intel/skylake/fsp_reset.c b/src/soc/intel/skylake/fsp_reset.c
deleted file mode 100644
index ca93399..0000000
--- a/src/soc/intel/skylake/fsp_reset.c
+++ /dev/null
@@ -1,62 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2015 Intel Corporation.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- */
-#include <bootstate.h>
-#include <vendorcode/google/chromeos/vboot_common.h>
-
-static int is_recovery; /* flag to identify recovery mode */
-
-/*
- * coreboot used to clear recovery status towards romstage end after FSP
- * memory init. Later inside FSP silicon init due to HSIO CRC mismatch
- * or other silicon related programming may request for an additional
- * reset. Thus on the next boot the system resumed in normal mode rather than
- * recovery because it lost its original state due to FSP silicon init reset.
- * Hence it needs an addition reset to get into old state and continue
- * booting into recovery mode. This function will set recovery reason
- * during Silicon init, in case of recovery mode booting,
- * so, system will not lose its original context.
- */
-static void set_recovery_request(void *unused)
-{
- is_recovery = recovery_mode_enabled();
- /*
- * Set recovery flag during Recovery Mode Silicon Init
- * & store recovery request into VBNV
- */
- if (is_recovery)
- set_recovery_mode_into_vbnv(vboot_handoff_get_recovery_reason());
-
-}
-
-static void clear_recovery_request(void *unused)
-{
- /*
- * Done with Silicon Init, it's safe to clear
- * reset request now with assumption that no reset occurs hereafter
- * so we will not miss original data.
- */
- if (is_recovery)
- set_recovery_mode_into_vbnv(0);
-}
-/*
- * On Recovery Path Set Recovery Request during early RAMSTAGE
- * before initiated Silicon Init
- */
-BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_ENTRY, set_recovery_request, NULL);
-/*
- * On Recovery Path Clear Recovery Request during early RAMSTAGE
- * end of Silicon Init
- */
-BOOT_STATE_INIT_ENTRY(BS_DEV_INIT_CHIPS, BS_ON_EXIT, clear_recovery_request, NULL);
More information about the coreboot-gerrit
mailing list