tree 2630b6576500c878ee247218e3597ae9ea200535
parent e8bbab68abad0a749e13909f3d304b71024e9b3c
author Matthew Blecker <matthewb@chromium.org> 1598747769 -0700
committer Commit Bot <commit-bot@chromium.org> 1599153614 +0000

chromeos-nvt-tcon-updater: inhibit suspend during TCON FW updates

BUG=b:167075994
TEST=Performed multiple no-update reboots.
The TCON FW updater correctly logged the TCON FW version and exited,
confirmed from /var/log/messages.

Triggered TCON FW update at boot by pointing TCON FW symlink at
different version.
Closed lid 5 times during the update, including for 3+ seconds at a time.
The DUT did not suspend, confirmed from /var/log/messages.
The TCON FW update succeeded, confirmed from /var/log/messages.

Triggered TCON FW update at boot by pointing TCON FW symlink at
different version.
Closed lid during the update, and left it closed.
Watched DUT power state from CR50 and EC consoles.
Per CR50, AP remained on for expected duration of the update,
then turned off at expected end of update.
Per EC, lid closed was registered but did not trigger immediate suspend.
After expected update completion, DUT bounced between power states until
settling on fake G3 (AP off).
Opened lid many minutes later.  The TCON FW update had succeeded,
confirmed from /var/log/messages.

Change-Id: Ia77bffcd5493501db5cb6271eb72ff72b5aca573
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2383935
Commit-Queue: Matthew Blecker <matthewb@chromium.org>
Tested-by: Matthew Blecker <matthewb@chromium.org>
Reviewed-by: Grant Grundler <grundler@chromium.org>
(cherry picked from commit f031e2d6a1c23615e945435faf4861f3da233ea8)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2384389
Reviewed-by: Matthew Blecker <matthewb@chromium.org>