summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAseda Aboagye <aaboagye@google.com>2017-10-11 13:14:23 -0700
committerchrome-bot <chrome-bot@chromium.org>2017-10-12 13:37:15 -0700
commit051339e28cb948d919c0714f1a2d3c65183c2a58 (patch)
tree902b06a7bfea59e2ebd920a24421299f930a4caa
parentdfe7473ed8fefe6b6bb15324fa835aafde4b0ffb (diff)
downloadchrome-ec-051339e28cb948d919c0714f1a2d3c65183c2a58.tar.gz
flash_ec: Save more variables.
It was found on zoombini, when flashing via flash_ec failed, it would prevent the board from booting as certain controls were left in their "flashing" state. This was because these controls were missing from the variables list which is what was used to restore the controls. BUG=b:65694294 BRANCH=None TEST=Attempt to flash zoombini without the flex connected. Attach the servo flex, apply power, verify DUT boots up. Change-Id: Ic2bc74ef1a61d4f10da6d3ceac77fbd373697838 Signed-off-by: Aseda Aboagye <aaboagye@google.com> Reviewed-on: https://chromium-review.googlesource.com/714023 Commit-Ready: Aseda Aboagye <aaboagye@chromium.org> Tested-by: Aseda Aboagye <aaboagye@chromium.org> Reviewed-by: Shawn N <shawnn@chromium.org>
-rwxr-xr-xutil/flash_ec9
1 files changed, 6 insertions, 3 deletions
diff --git a/util/flash_ec b/util/flash_ec
index a7956f4c93..73d9a76ed9 100755
--- a/util/flash_ec
+++ b/util/flash_ec
@@ -433,9 +433,9 @@ esac
# Not every control is supported on every servo type. Therefore, define which
# controls are supported by each servo type.
-servo_v2_VARS="jtag_buf_on_flex_en jtag_buf_en"
-servo_micro_VARS=
-servo_v4_with_ccd_cr50_VARS=
+servo_v2_VARS="jtag_buf_on_flex_en jtag_buf_en cold_reset spi1_vref"
+servo_micro_VARS="cold_reset spi1_vref"
+servo_v4_with_ccd_cr50_VARS="cold_reset"
# Flashing an STM32 over the UART requires modifying the UART properties along
# with the boot mode pin.
if [ "${CHIP}" = "stm32" ] ; then
@@ -453,6 +453,9 @@ if $(in_array "${BOARDS_STM32_PROG_EN[@]}" "${BOARD}"); then
fi
toad_VARS="${MCU}_uart_parity ${MCU}_uart_baudrate boot_mode"
+if $(in_array "${BOARDS_NPCX_INT_SPI[@]}" "${BOARD}"); then
+ servo_v2_VARS+=" fw_up"
+fi
# Some servo boards use the same controls.
servo_v3_VARS="${servo_v2_VARS}"
servo_v4_with_servo_micro_VARS="${servo_micro_VARS}"