commit 21a719be49fb721b5d095eddcd708da75d23371a
author Karthick Sundarrajan <karthick.sundarrajan@intel.com> Fri Nov 15 15:59:01 2019 -0800
committer Kuiying Wang <kuiying.wang@intel.com> Mon Jan 13 02:47:30 2020 +0000
tree 95f3a92bf7e7b98dce9e7e1b2bd5a8b05b476504
parent 8d6602196d496a8611a789cd36036b2f4ea14f76

Stop overwrite of CapabilitiesFlags during boot

Chassis capabilities should not be set from power control.
This also overwrites the other capabilities than Front panel
and NMI. Removing the chassis capabilities updation from here.

Tested:
Before reboot:
root@intel-obmc:~# ipmitool raw 0x00 0x00
 01 22 44 66 88 aa
After reboot:
root@intel-obmc:~# ipmitool raw 0x00 0x00
 01 22 44 66 88 aa

Change-Id: I424aafa66ee9e421bd2184e6e4fc9dca00660d3c
Signed-off-by: Karthick Sundarrajan <karthick.sundarrajan@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 1d0950a..90e8789 100644
--- a/power-control-x86/src/power_control.cpp
+++ b/power-control-x86/src/power_control.cpp
@@ -2332,21 +2332,6 @@
         power_control::idButtonIface->initialize();
     }
 
-    // set Chassis CapabilitiesFlags Property with 0x06: front panel(0x2) and
-    // NMI out(0x4)
-    power_control::conn->async_method_call(
-        [](boost::system::error_code ec) {
-            if (ec)
-            {
-                std::cerr << "failed to set Chassis CapabilitiesFlags\n";
-            }
-        },
-        "xyz.openbmc_project.Settings",
-        "/xyz/openbmc_project/control/chassis_capabilities_config",
-        "org.freedesktop.DBus.Properties", "Set",
-        "xyz.openbmc_project.Control.ChassisCapabilities", "CapabilitiesFlags",
-        std::variant<uint8_t>(0x06));
-
     // OS State Service
     sdbusplus::asio::object_server osServer =
         sdbusplus::asio::object_server(power_control::conn);