From 12c473337e3bdcce6d180d266c9e9f8127448f33 Mon Sep 17 00:00:00 2001 From: Ofer Fried Date: Thu, 28 Apr 2022 14:20:16 -0700 Subject: zork/guybrush: Add missing FP_PWR_EN configuration FP_PWR_EN is not required for flashing the FPMCU. It is required for POR after the flash is complete. Add FP_PWR_EN configuration for Zork and Guybrush. BRANCH=None BUG=b:229784281 TEST=Toggle the FPMCU_PWR_EN GPIO and confirm an additional `power-on` log appears in /var/log/cros_fp.log Signed-off-by: Ofer Fried Change-Id: I3375936496b40d6361c198a12643414600b0d080 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3615482 Reviewed-by: Tom Hughes Commit-Queue: Ofer Fried Tested-by: Ofer Fried --- util/flash_fp_mcu | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'util/flash_fp_mcu') diff --git a/util/flash_fp_mcu b/util/flash_fp_mcu index 9d3eb65cc4..75a989e1f4 100644 --- a/util/flash_fp_mcu +++ b/util/flash_fp_mcu @@ -656,9 +656,8 @@ config_zork() { readonly GPIO_NRST=267 # FPMCU BOOT0 is on AGPIO 69 = 256 + 69 = 325 readonly GPIO_BOOT0=325 - # FPMCU PWR_EN is on AGPIO 32 = 256 + 32 = 288, but should not be - # necessary for flashing. Set invalid value. - readonly GPIO_PWREN=-1 + # FPMCU PWR_EN is on AGPIO 32 = 256 + 32 = 288 + readonly GPIO_PWREN=288 } config_guybrush() { @@ -671,9 +670,8 @@ config_guybrush() { readonly GPIO_NRST=267 # FPMCU BOOT0 is on AGPIO 144 = 256 + 144 = 400 readonly GPIO_BOOT0=400 - # FPMCU PWR_EN is on AGPIO 32 = 256 + 32 = 288, but should not be - # necessary for flashing. Set invalid value. - readonly GPIO_PWREN=-1 + # FPMCU PWR_EN is on AGPIO 3 = 256 + 3 = 259 + readonly GPIO_PWREN=259 } main() { -- cgit v1.2.1