commit 364275c7df5ceb85cea89380f4beace5ae5f9f3e
author Alexander Hansen <alexander.hansen@9elements.com> Tue Nov 11 12:31:42 2025 +0100
committer Alexander Hansen <alexander.hansen@9elements.com> Wed Nov 12 10:41:03 2025 +0100
tree 8d57649c708aac80193cbb1e21d18a3fbc531d25
parent 70015ec570dcc174e95546db28e45ee468f51b88

BIOSConfig.Manager: use PDI constants

Pull property names, interface name, ... from PDI and drop a bunch of
local definitions.

The instance path is still remaining as a local definition, patch [1]
has been raised.

Tested: Inspection only.

References:
[1] https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/85118

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

common/utils.hpp

diff --git a/common/utils.hpp b/common/utils.hpp
index 867e064..7bd57b2 100644
--- a/common/utils.hpp
+++ b/common/utils.hpp
@@ -705,6 +705,8 @@
  */
 long int generateSwId();
 
+constexpr auto biosConfigPath = "/xyz/openbmc_project/bios_config/manager";
+
 /** @brief Method to get the value from a bios attribute
  *
  *  @param[in] dbusAttrName - the bios attribute name from
@@ -715,19 +717,19 @@
 template <typename T>
 std::optional<T> getBiosAttrValue(const std::string& dbusAttrName)
 {
-    constexpr auto biosConfigPath = "/xyz/openbmc_project/bios_config/manager";
-    constexpr auto biosConfigIntf = sdbusplus::common::xyz::openbmc_project::
-        bios_config::Manager::interface;
+    using BIOSConfigManager =
+        sdbusplus::common::xyz::openbmc_project::bios_config::Manager;
 
     std::string var1;
     std::variant<std::string, int64_t> var2, var3;
     auto& bus = DBusHandler::getBus();
     try
     {
-        auto service = pldm::utils::DBusHandler().getService(biosConfigPath,
-                                                             biosConfigIntf);
-        auto method = bus.new_method_call(service.c_str(), biosConfigPath,
-                                          biosConfigIntf, "GetAttribute");
+        auto service = pldm::utils::DBusHandler().getService(
+            biosConfigPath, BIOSConfigManager::interface);
+        auto method = bus.new_method_call(
+            service.c_str(), biosConfigPath, BIOSConfigManager::interface,
+            BIOSConfigManager::method_names::get_attribute);
         method.append(dbusAttrName);
         auto reply = bus.call(method, dbusTimeout);
         reply.read(var1, var2, var3);