diff options
| author | Robert Zieba <robertzieba@google.com> | 2022-10-05 10:43:00 -0600 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-05 18:25:53 +0000 |
| commit | b99bd46324736aabc4adae9211e5b7ac2b06e30c (patch) | |
| tree | 13a00f54592a4b382ea82d94cf2d967f3ef8b8f1 | |
| parent | 79559b39876a81548766f16655dcda6879061bf1 (diff) | |
| download | chrome-ec-b99bd46324736aabc4adae9211e5b7ac2b06e30c.tar.gz | |
skyrim: Remove Cr50 reset workaround config
This commit removes `CONFIG_PLATFORM_EC_BOARD_RESET_AFTER_POWER_ON` from
Skyrim. This config enables a brief delay as a workaround for a Cr50
issue that triggers a second reset of the EC shortly after it is reset.
Ti50 does not have this issue and this delay prevents GPIOs from being
configured before the jump to RW. RW does not attempt to configure
GPIOs again as it assumes that they were correctly configured by RO. The
misconfigured GPIOs cause the TCPC to be kept in reset which ultimately
causes a boot loop.
BRANCH=none
BUG=b:247543478
TEST=Ran on skyrim, verified that EC no longer boot loops.
Change-Id: I45562eba0a6bca2866ac173355765dd97e87d383
Signed-off-by: Robert Zieba <robertzieba@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3933588
Reviewed-by: Diana Z <dzigterman@chromium.org>
Commit-Queue: Diana Z <dzigterman@chromium.org>
| -rw-r--r-- | zephyr/projects/skyrim/prj.conf | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/zephyr/projects/skyrim/prj.conf b/zephyr/projects/skyrim/prj.conf index 65f0943605..2c8bc999fb 100644 --- a/zephyr/projects/skyrim/prj.conf +++ b/zephyr/projects/skyrim/prj.conf @@ -131,9 +131,6 @@ CONFIG_TASK_PD_STACK_SIZE=1280 # IOEX CONFIG_GPIO_NCT38XX=y -# Hibernate and wake -CONFIG_PLATFORM_EC_BOARD_RESET_AFTER_POWER_ON=y - # Motion sense CONFIG_PLATFORM_EC_MOTIONSENSE=y CONFIG_PLATFORM_EC_ACCEL_FIFO=y |