commit 653aec092d86597aa7d7d1259eee58e12310f202
author Alexander Hansen <alexander.hansen@9elements.com> Fri Nov 14 13:38:08 2025 +0100
committer Alexander Hansen <alexander.hansen@9elements.com> Fri Nov 14 14:07:38 2025 +0100
tree 8e8151a988126b5ea581e1dd37fca5cee72ead5e
parent f365c7ffccfaae696694865da3782cf52e62cbef

Sensor.Threshold.*: use PDI constants

Use PDI constants instead of local definitions.

Tested: Inspection only.

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

dbus-sdr/sdrutils.cpp

diff --git a/dbus-sdr/sdrutils.cpp b/dbus-sdr/sdrutils.cpp
index e7f6566..6065edb 100644
--- a/dbus-sdr/sdrutils.cpp
+++ b/dbus-sdr/sdrutils.cpp
@@ -20,6 +20,8 @@
 #include <nlohmann/json.hpp>
 #include <phosphor-logging/lg2.hpp>
 #include <xyz/openbmc_project/ObjectMapper/common.hpp>
+#include <xyz/openbmc_project/Sensor/Threshold/Critical/common.hpp>
+#include <xyz/openbmc_project/Sensor/Threshold/Warning/common.hpp>
 #include <xyz/openbmc_project/Sensor/Value/common.hpp>
 
 #include <fstream>
@@ -28,6 +30,10 @@
 
 using ObjectMapper = sdbusplus::common::xyz::openbmc_project::ObjectMapper;
 using SensorValue = sdbusplus::common::xyz::openbmc_project::sensor::Value;
+using SensorThresholdWarning =
+    sdbusplus::common::xyz::openbmc_project::sensor::threshold::Warning;
+using SensorThresholdCritical =
+    sdbusplus::common::xyz::openbmc_project::sensor::threshold::Critical;
 
 #ifdef FEATURE_HYBRID_SENSORS
 
@@ -173,8 +179,7 @@
     // Add sensors to SensorTree
     static constexpr const std::array sensorInterfaces = {
         SensorValue::interface, "xyz.openbmc_project.Sensor.ValueMutability",
-        "xyz.openbmc_project.Sensor.Threshold.Warning",
-        "xyz.openbmc_project.Sensor.Threshold.Critical"};
+        SensorThresholdWarning::interface, SensorThresholdCritical::interface};
     static constexpr const std::array vrInterfaces = {
         "xyz.openbmc_project.Control.VoltageRegulatorMode"};