diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2017-09-21 17:52:36 -0700 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2017-10-04 20:55:12 +0000 |
commit | 47dffa59f35a46ada039dd86a20497caa8f518e8 (patch) | |
tree | 7a88e916aa974e06c53f2e99f4252e311818d977 /src/soc/intel/baytrail/romstage/raminit.c | |
parent | 94b64431f3de19c79e7494d9ff25f9ebd1ab7cbc (diff) | |
download | coreboot-47dffa59f35a46ada039dd86a20497caa8f518e8.tar.gz coreboot-47dffa59f35a46ada039dd86a20497caa8f518e8.tar.bz2 coreboot-47dffa59f35a46ada039dd86a20497caa8f518e8.zip |
chromeec: Remove checks for EC in RO
This patch removes checks that ensure EC to be in RO for recovery
boot. We do not need these checks because when recovery is requested
automatically (as opposed to manually), we show 'broken' screen where
users can only reboot the device or request recovery manually.
If recovery is requested, Depthcharge will check whether EC is in RO
or not and recovery switch was pressed or not. If it's a legitimate
manual recovery, EC should be in RO. Thus, we can trust the recovery
button state it reports.
This patch removes all calls to google_chromeec_check_ec_image,
which is called to avoid duplicate memory training when recovery
is requested but EC is in RW.
BUG=b:66516882
BRANCH=none
CQ-DEPEND=CL:693008
TEST=Boot Fizz.
Change-Id: I45a874b73c46ea88cb831485757d194faa9f4c99
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://review.coreboot.org/21711
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/soc/intel/baytrail/romstage/raminit.c')
-rw-r--r-- | src/soc/intel/baytrail/romstage/raminit.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/soc/intel/baytrail/romstage/raminit.c b/src/soc/intel/baytrail/romstage/raminit.c index f9a5fd162697..44e0923b9bf4 100644 --- a/src/soc/intel/baytrail/romstage/raminit.c +++ b/src/soc/intel/baytrail/romstage/raminit.c @@ -137,12 +137,6 @@ void raminit(struct mrc_params *mp, int prev_sleep_state) reset_system(); } else { printk(BIOS_DEBUG, "No MRC cache found.\n"); -#if IS_ENABLED(CONFIG_EC_GOOGLE_CHROMEEC) - if (prev_sleep_state == ACPI_S0) { - /* Ensure EC is running RO firmware. */ - google_chromeec_check_ec_image(EC_IMAGE_RO); - } -#endif } /* Determine if mrc.bin is in the cbfs. */ |