commit 2260ff34ee49dc4e1ee82eead6c87c8e54d4bbdc
author Alexander Hansen <alexander.hansen@9elements.com> Fri Nov 07 17:12:52 2025 +0100
committer Alexander Hansen <alexander.hansen@9elements.com> Fri Nov 14 09:47:46 2025 +0100
tree d46d5b1c56e9c031626e3772b1d931be4495f51e
parent 76bfd7563b83f7cf04e27465cbf2b9246c09e28a

use PDI constants for State.Chassis

Tested: Inspection only.

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

chassishandler.cpp

diff --git a/chassishandler.cpp b/chassishandler.cpp
index 2f8cbad..a6a8062 100644
--- a/chassishandler.cpp
+++ b/chassishandler.cpp
@@ -126,6 +126,7 @@
 using namespace sdbusplus::server::xyz::openbmc_project::control::boot;
 using Intrusion = sdbusplus::client::xyz::openbmc_project::chassis::Intrusion<>;
 using HostState = sdbusplus::common::xyz::openbmc_project::state::Host;
+using ChassisState = sdbusplus::common::xyz::openbmc_project::state::Chassis;
 
 namespace chassis
 {
@@ -861,11 +862,10 @@
 {
     // OpenBMC Chassis State Manager dbus framework
     constexpr auto chassisStatePath = "/xyz/openbmc_project/state/chassis0";
-    constexpr auto chassisStateIntf = "xyz.openbmc_project.State.Chassis";
 
     std::string service;
-    boost::system::error_code ec =
-        ipmi::getService(ctx, chassisStateIntf, chassisStatePath, service);
+    boost::system::error_code ec = ipmi::getService(
+        ctx, ChassisState::interface, chassisStatePath, service);
 
     // Convert to string equivalent of the passed in transition enum.
     auto request =
@@ -874,9 +874,9 @@
 
     if (!ec)
     {
-        ec = ipmi::setDbusProperty(ctx, service, chassisStatePath,
-                                   chassisStateIntf, "RequestedPowerTransition",
-                                   request);
+        ec = ipmi::setDbusProperty(
+            ctx, service, chassisStatePath, ChassisState::interface,
+            ChassisState::property_names::requested_power_transition, request);
     }
     if (ec)
     {
@@ -981,14 +981,12 @@
     {
         constexpr const char* chassisStatePath =
             "/xyz/openbmc_project/state/chassis0";
-        constexpr const char* chassisStateIntf =
-            "xyz.openbmc_project.State.Chassis";
         auto service =
-            ipmi::getService(*busp, chassisStateIntf, chassisStatePath);
+            ipmi::getService(*busp, ChassisState::interface, chassisStatePath);
 
-        ipmi::Value powerState =
-            ipmi::getDbusProperty(*busp, service, chassisStatePath,
-                                  chassisStateIntf, "CurrentPowerState");
+        ipmi::Value powerState = ipmi::getDbusProperty(
+            *busp, service, chassisStatePath, ChassisState::interface,
+            ChassisState::property_names::current_power_state);
         std::string powerStateStr = std::get<std::string>(powerState);
         if (powerStateStr.ends_with(".On") ||
             powerStateStr.ends_with(".TransitioningToOff"))