commit d5ef1a285223881f84fe7fd5d227183b8326d502
author Will Drewry <wad@google.com> Mon May 15 12:43:45 2017 -0500
committer Will Drewry <wad@google.com> Mon May 15 12:43:45 2017 -0500
tree 215e728ec8d1268c92dc64f2072ad3ddb3e2232b
parent 3eca2f28263f3ea5e5d760949c135a648f2df308

libese-hw: restore explicit power off

As some parts do not properly enter deep sleep
and the kernel does not always power off on shut
down, this change adds explicit power off back
to ese_close() path.

Test: boots, toggles
Bug: 38203165
Change-Id: I58f26292154ba8cdf1aa55df3f2f57dc9eaac880

libese-hw/nxp/pn80t/common.c

diff --git a/libese-hw/nxp/pn80t/common.c b/libese-hw/nxp/pn80t/common.c
index 9164fae..b2b4abd 100644
--- a/libese-hw/nxp/pn80t/common.c
+++ b/libese-hw/nxp/pn80t/common.c
@@ -308,10 +308,10 @@
   const uint32_t wait_sec = nxp_pn80t_send_cooldown(ese, true);
 
   /* After the cooldown, the device should go to sleep.
-   * There is no need to pull the power explicitly unless
-   * we're in an error state.
+   * If not post-use time is required, power down to ensure
+   * that the device is powered down when the OS is not on.
    */
-  if (ese_error(ese)) {
+  if (ese_error(ese) || wait_sec == 0) {
     platform->toggle_reset(ns->handle, 0);
     if (platform->toggle_power_req) {
       platform->toggle_power_req(ns->handle, 0);