diff options
author | Tom Hughes <tomhughes@chromium.org> | 2022-08-26 11:18:32 -0700 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-08-30 18:21:34 +0000 |
commit | 749fe7ccdd4a8a69cd56e0245bfba85d2ba3c803 (patch) | |
tree | f7a92ec19e45312d4bd971b59f0daedb4f727484 | |
parent | 2e84b48242fed4cc005c9c81245ce31f656cbb19 (diff) | |
download | chrome-ec-749fe7ccdd4a8a69cd56e0245bfba85d2ba3c803.tar.gz |
util/flash_ec: Fix shellcheck warnings
dev_cnt=$(lsusb -d ${DFU_DEVICE} | wc -l)
^-----------^ SC2248: Prefer double quoting even when
variables don't contain special characters.
BRANCH=none
BUG=b:242127759
TEST=./util/flash_ec --board=dartmonkey
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ie299348fc4aa94db2c6060c38c27fadb52525389
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3860075
Reviewed-by: Bobby Casey <bobbycasey@google.com>
-rwxr-xr-x | util/flash_ec | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/util/flash_ec b/util/flash_ec index 26907d0112..46cfecc53b 100755 --- a/util/flash_ec +++ b/util/flash_ec @@ -854,7 +854,7 @@ function claim_pty() { # interfere with flashing. servo_save_add "${MCU}_ec3po_interp_connect" - dut_control ${MCU}_ec3po_interp_connect:off || \ + dut_control "${MCU}_ec3po_interp_connect:off" || \ warn "hdctools cannot disconnect the EC-3PO interpreter from" \ "the UART." @@ -1017,7 +1017,7 @@ function flash_flashrom() { servo_save_add "spi1_buf_en" "off" # Turn on SPI1 interface on servo for SPI Flash Chip - dut_control spi1_vref:${SPI_VOLTAGE} spi1_buf_en:on + dut_control "spi1_vref:${SPI_VOLTAGE}" spi1_buf_en:on if [[ ! "${SERVO_TYPE}" =~ "servo_micro" ]]; then # Servo micro doesn't support this control. servo_save_add "spi1_buf_on_flex_en" "off" @@ -1072,18 +1072,18 @@ function flash_flashrom() { { # Patch temp image up to SPI_SIZE cat "${IMG}" if [[ ${IMG_SIZE} -lt ${SPI_SIZE} ]] ; then - dd if=/dev/zero bs=${PATCH_SIZE} count=1 | \ + dd if=/dev/zero bs="${PATCH_SIZE}" count=1 | \ tr '\0' '\377' fi - } > ${T} + } > "${T}" else { # Patch temp image up to SPI_SIZE if [[ ${IMG_SIZE} -lt ${SPI_SIZE} ]] ; then - dd if=/dev/zero bs=${PATCH_SIZE} count=1 | \ + dd if=/dev/zero bs="${PATCH_SIZE}" count=1 | \ tr '\0' '\377' fi cat "${IMG}" - } > ${T} + } > "${T}" fi info "Programming EC firmware image." @@ -1238,12 +1238,12 @@ function flash_stm32_dfu() { DFU_DEVICE=0483:df11 ADDR=0x08000000 DFU_UTIL='dfu-util' - which ${DFU_UTIL} &> /dev/null || die \ + which "${DFU_UTIL}" &> /dev/null || die \ "no dfu-util util found. Did you 'sudo emerge dfu-util'" info "Using dfu flasher : ${DFU_UTIL}" - dev_cnt=$(lsusb -d ${DFU_DEVICE} | wc -l) + dev_cnt=$(lsusb -d "${DFU_DEVICE}" | wc -l) if [ "${dev_cnt}" -eq 0 ] ; then die "unable to locate dfu device at ${DFU_DEVICE}" elif [ "${dev_cnt}" -ne 1 ] ; then @@ -1252,12 +1252,12 @@ function flash_stm32_dfu() { SIZE=$(wc -c "${IMG}" | cut -d' ' -f1) # Remove read protection - print_or_run sudo timeout -k 10 -s 9 "${FLAGS_timeout}" ${DFU_UTIL} -a 0 \ + print_or_run sudo timeout -k 10 -s 9 "${FLAGS_timeout}" "${DFU_UTIL}" -a 0 \ -d "${DFU_DEVICE}" -s "${ADDR}:${SIZE}:force:unprotect" -D "${IMG}" # Wait for mass-erase and reboot after unprotection print_or_run sleep 1 # Actual image flashing - print_or_run sudo timeout -k 10 -s 9 "${FLAGS_timeout}" ${DFU_UTIL} -a 0 \ + print_or_run sudo timeout -k 10 -s 9 "${FLAGS_timeout}" "${DFU_UTIL}" -a 0 \ -d "${DFU_DEVICE}" -s "${ADDR}:${SIZE}" -D "${IMG}" } |