commit b1f1f1163b2313c04ea5e96e7897d801ad3ff8f8
author Alexander Hansen <alexander.hansen@9elements.com> Fri Nov 07 16:36:43 2025 +0100
committer Alexander <alexander.hansen@9elements.com> Fri Nov 14 08:47:10 2025 +0000
tree 83e6ff2bda65990f2dd3147e48d3a4ebccffccd1
parent 74c0863a0805ebfd61f621b7d0c2c8fe602cbb13

use PDI constants for State.BMC

Tested: Inspection only.

Change-Id: I7ba48d7f5f856d5f284b68f8344b982da08c58df
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

globalhandler.cpp

diff --git a/globalhandler.cpp b/globalhandler.cpp
index 7b64d42..36cf0a4 100644
--- a/globalhandler.cpp
+++ b/globalhandler.cpp
@@ -5,12 +5,7 @@
 
 #include <string>
 
-static constexpr auto bmcStateRoot = "/xyz/openbmc_project/state";
-static constexpr auto bmcStateIntf = "xyz.openbmc_project.State.BMC";
-static constexpr auto reqTransition = "RequestedBMCTransition";
-static constexpr auto match = "bmc0";
-
-using BMC = sdbusplus::server::xyz::openbmc_project::state::BMC;
+using BMCState = sdbusplus::server::xyz::openbmc_project::state::BMC;
 
 void registerNetFnGlobalFunctions() __attribute__((constructor));
 
@@ -22,16 +17,19 @@
 {
     ipmi::DbusObjectInfo bmcStateObj;
     boost::system::error_code ec = ipmi::getDbusObject(
-        ctx, bmcStateIntf, bmcStateRoot, match, bmcStateObj);
+        ctx, BMCState::interface, BMCState::namespace_path::value,
+        BMCState::namespace_path::bmc, bmcStateObj);
     if (!ec)
     {
         std::string service;
-        ec = ipmi::getService(ctx, bmcStateIntf, bmcStateObj.first, service);
+        ec = ipmi::getService(ctx, BMCState::interface, bmcStateObj.first,
+                              service);
         if (!ec)
         {
             ec = ipmi::setDbusProperty(
-                ctx, service, bmcStateObj.first, bmcStateIntf, reqTransition,
-                convertForMessage(BMC::Transition::Reboot));
+                ctx, service, bmcStateObj.first, BMCState::interface,
+                BMCState::property_names::requested_bmc_transition,
+                convertForMessage(BMCState::Transition::Reboot));
         }
     }
     if (ec)