commit 081b035fbcdf9c08f783a662c695f4412cdbec5a
author Patrick Venture <venture@google.com> Mon Jul 15 11:05:17 2019 -0700
committer Patrick Venture <venture@google.com> Tue Jul 16 10:47:51 2019 -0700
tree 14fbb1be36fb2b06223552e1a17a5296866240b9
parent 5ec202704d9952674ff91e5741ac00581733f73e

bugfix: writePath is the dbus object path for the sensor

For the sensor, if writePath is used for dbus active, it is always
FanPwm (presently) and needs to be the object path.

Tested: Not tested, only code inspected.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I4eafaff4316aac9dab9a2c9e1bd61093a670b023

util.cpp

diff --git a/util.cpp b/util.cpp
index f66f20b..ec60d7a 100644
--- a/util.cpp
+++ b/util.cpp
@@ -21,7 +21,6 @@
 static constexpr auto external_sensor =
     "/xyz/openbmc_project/extsensors/";                         // type/
 static constexpr auto openbmc_sensor = "/xyz/openbmc_project/"; // type/
-static constexpr auto dbus_pwm = "/xyz/openbmc_project/control/fanpwm/";
 static constexpr auto sysfs = "/sys/";
 
 IOInterfaceType getWriteInterfaceType(const std::string& path)
@@ -37,7 +36,7 @@
         return IOInterfaceType::SYSFS;
     }
 
-    if (path.find(dbus_pwm) != std::string::npos)
+    if (path.find(openbmc_sensor) != std::string::npos)
     {
         return IOInterfaceType::DBUSACTIVE;
     }