commit fe15903709fec676946e593b5091001dee73724d
author Jason M. Bills <jason.m.bills@intel.com> Thu Sep 01 16:03:37 2022 -0700
committer Jason Bills <jason.m.bills@linux.intel.com> Thu Sep 22 17:41:50 2022 +0000
tree 0729491898599be243978b41af19aeceb68846da
parent 8643c79740d21f4a21184de729c19f9aec0f7c48

Always start psPowerOKWatchdog before waiting

There are a couple conditions where we wait for psPowerOK without
starting the watchdog timer. This can result in getting stuck in the
waiting state, so we should always start the psPowerOKWatchdog timer
before waiting.

Tested:
Confirmed that power-control still starts normally.

Change-Id: I50a5c8a1a10e234e72c3bfe3f0c2070749d9ffde
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>

src/power_control.cpp

diff --git a/src/power_control.cpp b/src/power_control.cpp
index 563b008..7362c6c 100644
--- a/src/power_control.cpp
+++ b/src/power_control.cpp
@@ -1786,6 +1786,7 @@
             break;
         }
         case Event::sioS5DeAssert:
+            psPowerOKWatchdogTimerStart();
             setPowerState(PowerState::waitForPSPowerOK);
             break;
         case Event::sioPowerGoodAssert:
@@ -1876,6 +1877,7 @@
         }
         case Event::sioS5DeAssert:
             powerCycleTimer.cancel();
+            psPowerOKWatchdogTimerStart();
             setPowerState(PowerState::waitForPSPowerOK);
             break;
         case Event::powerButtonPressed: