commit d603dd1b286096151fac0619c4eb16e6641deac3
author Zev Weiss <zev@bewilderbeest.net> Wed Jun 19 21:50:52 2024 +0000
committer Zev Weiss <zev@bewilderbeest.net> Thu Jun 20 22:46:02 2024 +0000
tree d6faaabd0cd7e249250c1d73eed58f7a64dae1cc
parent ca47855d3ee7a9397aa80f1ad43ee64768327cf2

Ensure all dbus inputs are treated as active-high

Previously the polarity member of the ConfigData struct wasn't getting
set in loadConfigValues(), so it retained its default value
(active-low).  Active-low inputs via dbus don't make much sense, so
let's normalize them all to be active-high.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I529fd6362db9a49b859f97a231c09d23a782c23d

src/power_control.cpp

diff --git a/src/power_control.cpp b/src/power_control.cpp
index edf95e8..41e4c9a 100644
--- a/src/power_control.cpp
+++ b/src/power_control.cpp
@@ -2438,6 +2438,9 @@
             tempGpioData->lineName =
                 gpioConfig[dbusParams[DbusConfigType::property]];
 
+            // dbus-based inputs must be active-high.
+            tempGpioData->polarity = true;
+
             // MatchRegex is optional
             auto item = gpioConfig.find("MatchRegex");
             if (item != gpioConfig.end())