commit e960184d378a4f735315a65a01d560158b555542
author Potin Lai <potin.lai@quantatw.com> Fri Mar 03 17:23:28 2023 +0800
committer Andrew Geissler <andrew@geissonator.com> Fri Mar 03 15:32:19 2023 +0000
tree 1fe16b882c3bc6ce60baa01f680c6550b3369a43
parent a0d7bb84fd516111c262f55fd2db8a298cb54a2b

discover_system_state: fix ifdef ONLY_RUN_APR_ON_POWER_LOSS always true

ONLY_RUN_APR_ON_POWER_LOSS is defined as 0 or 1 in config.h, it should
use `#if ONLY_RUN_APR_ON_POWER_LOSS` instead of
`#ifdef ONLY_RUN_APR_ON_POWER_LOSS`, otherwise it will be true
statement all the time.

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I36d1d0071c0c2c6b3d34d2d9f58d4ea0b4334cb7

discover_system_state.cpp

diff --git a/discover_system_state.cpp b/discover_system_state.cpp
index 6227ddf..8dfe004 100644
--- a/discover_system_state.cpp
+++ b/discover_system_state.cpp
@@ -178,7 +178,7 @@
         // Always execute power on if AlwaysOn is set, otherwise check config
         // option (and AC loss status) on whether to execute other policy
         // settings
-#ifdef ONLY_RUN_APR_ON_POWER_LOSS
+#if ONLY_RUN_APR_ON_POWER_LOSS
         else if (!phosphor::state::manager::utils::checkACLoss(hostId))
         {
             info(