commit 25a18f559b4a18e5bf7fecb51e844eef7ae17e1a
author Cheng-Han Yang <chenghan@google.com> Mon Nov 26 09:15:37 2018
committer chrome-bot <chrome-bot@chromium.org> Sat Dec 15 06:40:36 2018
tree befd3597660b3f7073f73f85ea873827f3f4dbf0
parent 3efebf6270c828f0f0e1dc4fbaf8f14d5e853bdb

factory_install: Use gsctool to update cr50.

usb_updater is deprecated. Use gsctool instead to update cr50 firmware.

BUG=b:118821734
TEST=Manually test on DUT

Change-Id: I5a860c2cb63e1f81c28467265f35bae62b7c0b94
Reviewed-on: https://chromium-review.googlesource.com/1350074
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Cheng-Han Yang <chenghan@chromium.org>
Reviewed-by: Ting Shen <phoenixshen@chromium.org>

factory_install.sh

diff --git a/factory_install.sh b/factory_install.sh
index 2ce9f30..65ea285 100644
--- a/factory_install.sh
+++ b/factory_install.sh
@@ -1307,21 +1307,31 @@
 action_u() {
   local temp_firmware="$(copy_prod_cr50_firmware)"
 
-  stop trunksd || true
   local result=0
-  usb_updater -s -u "${temp_firmware}" || result="$?"
+  "${GSCTOOL}" -a -u "${temp_firmware}" || result="$?"
 
   rm "${temp_firmware}"
 
   # Allow 0(no-op), 1(all_updated), 2(rw_updated), other return values are
   # considered fail.
-  # See trunk/src/platform/ec/extra/usb_updater/usb_updater.c for more detail.
+  # See trunk/src/platform/ec/extra/usb_updater/gsctool.h for more detail.
   case "${result}" in
-    "0" | "1" | "2" )
+    "0" )
+      log "Cr50 not updated. Returning to shim menu."
+      # sleep for a while to show the messages
+      sleep 3
+      return 0
+      ;;
+    "1" | "2" )
+      log "Cr50 updated. System will reboot shortly."
+      # sleep for a while to show the messages
+      sleep 3
+      reboot
+      sleep 1d
       return 0
       ;;
     *)
-      die "usb_updater execution failed as ${result}."
+      die "gsctool execution failed as ${result}."
       ;;
   esac
 }