commit 303bd58c4b5d3da4a2d791f27e1a3efcfadfe523
author Chen Yugang <yugang.chen@linux.intel.com> Fri Nov 01 08:45:06 2019 +0800
committer Jason Bills <jason.m.bills@linux.intel.com> Mon Nov 25 20:03:05 2019 +0000
tree 18f42baa2955cedee5abdd4c2adc5d132bca35f0
parent 605e9c775e78a0e4e53b164e6b6f9a95be2c7434

Update NMISource behavior after its Property moved to phospher-dbus-interfaces

It's needed to update the NMI source bahavior since NMI source Property
is moved from intel-dbus-interface to phospher-dbus-interfaces by
changing from "com.intel" to "xyz.openbmc_project".

Tested
it's tested in system for IPMI OEM command - get/set NMI source

Change-Id: I0dcb9fab7083a0aee17c7a0eb31568f98ebdfb05
Signed-off-by: Chen Yugang <yugang.chen@linux.intel.com>

power-control-x86/src/power_control.cpp

diff --git a/power-control-x86/src/power_control.cpp b/power-control-x86/src/power_control.cpp
index 661f7e9..4126a75 100644
--- a/power-control-x86/src/power_control.cpp
+++ b/power-control-x86/src/power_control.cpp
@@ -1743,9 +1743,11 @@
                 std::cerr << "failed to set NMI source\n";
             }
         },
-        "xyz.openbmc_project.Settings", "/com/intel/control/NMISource",
-        "org.freedesktop.DBus.Properties", "Set", "com.intel.Control.NMISource",
-        "Enabled", std::variant<bool>{value});
+        "xyz.openbmc_project.Settings",
+        "/xyz/openbmc_project/Chassis/Control/NMISource",
+        "org.freedesktop.DBus.Properties", "Set",
+        "xyz.openbmc_project.Chassis.Control.NMISource", "Enabled",
+        std::variant<bool>{value});
 }
 
 static void nmiReset(void)
@@ -1787,7 +1789,8 @@
         std::make_unique<sdbusplus::bus::match::match>(
             *conn,
             "type='signal',interface='org.freedesktop.DBus.Properties',"
-            "member='PropertiesChanged',arg0namespace='com.intel.Control."
+            "member='PropertiesChanged',arg0namespace='xyz.openbmc_project."
+            "Chassis.Control."
             "NMISource'",
             [](sdbusplus::message::message& msg) {
                 std::string interfaceName;
@@ -1830,11 +1833,12 @@
                 std::cerr << "failed to set NMI source\n";
             }
         },
-        "xyz.openbmc_project.Settings", "/com/intel/control/NMISource",
-        "org.freedesktop.DBus.Properties", "Set", "com.intel.Control.NMISource",
-        "BMCSource",
-        std::variant<std::string>{
-            "com.intel.Control.NMISource.BMCSourceSignal.FpBtn"});
+        "xyz.openbmc_project.Settings",
+        "/xyz/openbmc_project/Chassis/Control/NMISource",
+        "org.freedesktop.DBus.Properties", "Set",
+        "xyz.openbmc_project.Chassis.Control.NMISource", "BMCSource",
+        std::variant<std::string>{"xyz.openbmc_project.Chassis.Control."
+                                  "NMISource.BMCSourceSignal.FpBtn"});
     // set Enable Property
     nmiSetEnablePorperty(true);
 }