chromiumos/platform/touch_updater.git
17 months agotouch_updater : Remove set -e from scripts to prevent silent failure 78/46478/4 factory-4128.B factory-4290.B factory-4455.B factory-pit-4280.B factory-pit-4390.B factory-pit-4471.B factory-spring-4131.B factory-spring-4262.B firmware-falco_peppy-4389.B firmware-leon-4389.26.B firmware-pit-4482.B firmware-wolf-4389.24.B master release-R27-3912.B release-R28-4100.B release-R29-4319.B release-R30-4537.B stabilize-3912.79.B stabilize-4008.0.B stabilize-4035.0.B stabilize-4068.0.B stabilize-4100.38.B stabilize-4255.B stabilize-4287.B stabilize-4443.B stabilize-4512.B stabilize-spring-4100.53.B toolchainA toolchainB
Benson Leung [Tue, 26 Mar 2013 01:23:23 +0000 (18:23 -0700)]
touch_updater : Remove set -e from scripts to prevent silent failure

In chromeos-touch-config-update.sh and chromeos-touch-firmware-update.sh,
we have set -e, which causes the scripts to exit immediately on error.

This is incorrect behavior as we want to be able to output a message to
the system log that the update failed, or in the case of the firmware update,
retry the firmware update.

Also, die early on an error condition.
Check the return value after issuing the printf to update_config.
Die early instead of checking the config csum in a loop 5 times.

Signed-off-by: Benson Leung <bleung@chromium.org>
BUG=chromium:223805,chromium:223814
TEST=Corrupt the 224sl.raw file on a test image.
edit this file on spring : /opt/google/touch/config/224sl.raw
Modify the 3rd line of the file, which should be a 6 digit
hexadecimal number. This is the info block CRC.
Replace it with 000000
Do the same with the 4th line, which is the config crc.
Run : /opt/google/touch/scripts/chromeos-touch-config-update.sh -d atmel_mxt_tp
This should log the error as follows.
Device config checksum : a84914
New config checksum : 000000
error: Config update failed. ret=1

Change-Id: Icb66a0bfd93a950f6c85a2190a06dc47e248be71
Reviewed-on: https://gerrit.chromium.org/gerrit/46478
Reviewed-by: Yufeng Shen <miletus@chromium.org>
Commit-Queue: Benson Leung <bleung@chromium.org>
Tested-by: Benson Leung <bleung@chromium.org>
17 months agotouch_updater : Fix so chromeos-touch-update job runs just at boot 94/45794/3
Benson Leung [Mon, 18 Mar 2013 22:09:43 +0000 (15:09 -0700)]
touch_updater : Fix so chromeos-touch-update job runs just at boot

Previously, the job consisted of a main script that would run to completion
each time the UI restarted. This means the firmware/config check would
happen on logout too. jrbarnette suggested this change to run the check
as a pre-start so it only blocks the UI on the first boot, not subsequent
UI restarts.

Signed-off-by: Benson Leung <bleung@chromium.org>
BUG=chromium:196823
TEST=Cold boot a system (say, Link).
Check /var/log/messages:
Verify that the update check happens properly.
Here is a normal no-update check :
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Product ID : 130.1
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Current Firmware: 1.0.170
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Updater Firmware: 1.0.170
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Firmware up to date.
chromeos-touch-config-update[6802]-atmel_mxt_tp: Device config checksum : ae968a
chromeos-touch-config-update[6802]-atmel_mxt_tp: New config checksum : ae968a
chromeos-touch-config-update[6802]-atmel_mxt_tp: Config is up-to-date, no need to update
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Product ID : 162.0
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Current Firmware: 1.1.170
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Updater Firmware: 1.1.170
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Firmware up to date.
chromeos-touch-config-update[6802]-atmel_mxt_ts: Device config checksum : 7d7cf1
chromeos-touch-config-update[6802]-atmel_mxt_ts: New config checksum : 7d7cf1
chromeos-touch-config-update[6802]-atmel_mxt_ts: Config is up-to-date, no need to update

Log in as Guest. Log out.
Check /var/log/messages again. Check that the above check does not appear around
the time of logout.

Change-Id: I3bb876180073783ec471959f155375a7101d05ca
Reviewed-on: https://gerrit.chromium.org/gerrit/45794
Reviewed-by: Benson Leung <bleung@chromium.org>
Tested-by: Benson Leung <bleung@chromium.org>
Commit-Queue: Benson Leung <bleung@chromium.org>

18 months agotouch-updater : Add initial touch updater scripts 73/45373/11 stabilize-3881.0.B
Benson Leung [Wed, 13 Mar 2013 23:03:10 +0000 (16:03 -0700)]
touch-updater : Add initial touch updater scripts

Adds the following files :
scripts/chromeos-touch-config-update.sh
scripts/chromeos-touch-firmware-update.conf
scripts/chromeos-touch-firmware-update.sh
scripts/chromeos-touch-common.sh

The config update and firmware update scripts are based on
the latest version found in the internal link overlay.

The config updater has been significantly cleaned up, with
a die function added, proper ${} around variables, etc.

Signed-off-by: Benson Leung <bleung@chromium.org>
BUG=chromium-os:39392
TEST=Install the four scripts here :
/opt/google/touch/scripts/chromeos-touch-config-update.sh
/opt/google/touch/scripts/chromeos-touch-firmware-update.sh
/opt/google/touch/scripts/chromeos-touch-common.sh
/etc/init/chromeos-touch-update.conf

Then run :
start chromeos-touch-update.conf on Pixel.
Check that the update check happens properly.

Here is a normal no-update check :
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Product ID : 130.1
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Current Firmware: 1.0.170
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Updater Firmware: 1.0.170
chromeos-touch-firmware-update[6802]-atmel_mxt_tp: Firmware up to date.
chromeos-touch-config-update[6802]-atmel_mxt_tp: Device config checksum : ae968a
chromeos-touch-config-update[6802]-atmel_mxt_tp: New config checksum : ae968a
chromeos-touch-config-update[6802]-atmel_mxt_tp: Config is up-to-date, no need to update
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Product ID : 162.0
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Current Firmware: 1.1.170
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Updater Firmware: 1.1.170
chromeos-touch-firmware-update[6802]-atmel_mxt_ts: Firmware up to date.
chromeos-touch-config-update[6802]-atmel_mxt_ts: Device config checksum : 7d7cf1
chromeos-touch-config-update[6802]-atmel_mxt_ts: New config checksum : 7d7cf1
chromeos-touch-config-update[6802]-atmel_mxt_ts: Config is up-to-date, no need to update

Change-Id: I1e8fc8d65eb733e02c5bc31727b6599db6575ad1
Reviewed-on: https://gerrit.chromium.org/gerrit/45373
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
Commit-Queue: Benson Leung <bleung@chromium.org>
Tested-by: Benson Leung <bleung@chromium.org>
18 months agotouch-updater : Add initial OWNERS, README, and LICENSE 53/45353/2
Benson Leung [Wed, 13 Mar 2013 22:01:39 +0000 (15:01 -0700)]
touch-updater : Add initial OWNERS, README, and LICENSE

Signed-off-by: Benson Leung <bleung@chromium.org>
BUG=chromium-os:39392
TEST=None.

Change-Id: Ib3d6e9874220c4f7dcdd91aa92525c9bca96fa28
Reviewed-on: https://gerrit.chromium.org/gerrit/45353
Reviewed-by: David James <davidjames@chromium.org>
Commit-Queue: Benson Leung <bleung@chromium.org>
Tested-by: Benson Leung <bleung@chromium.org>
18 months agoinit touch_updater repo
Benson Leung [Wed, 13 Mar 2013 21:24:28 +0000 (14:24 -0700)]
init touch_updater repo