commit d20f0ac4fb8643586af3969d2a9434c9f4c7e082
author Alexander Hansen <alexander.hansen@9elements.com> Wed Nov 26 14:16:01 2025 +0100
committer ManojKiran Eda <manojkiran.eda@gmail.com> Tue Dec 02 05:11:19 2025 +0000
tree 264562621077be5b0d54ab3f2d3bb6a42c65625a
parent 1976152a7c7b26bb0991f38da4d11cf143ccf18e

mctp_endpoint_discovery: use PDI definitions

Use PDI symbols instead of string literals for the
interface name of `xyz.openbmc_project.Common.UUID`.

Tested: Inspection only.

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

requester/mctp_endpoint_discovery.cpp

diff --git a/requester/mctp_endpoint_discovery.cpp b/requester/mctp_endpoint_discovery.cpp
index 41ad909..6f825f2 100644
--- a/requester/mctp_endpoint_discovery.cpp
+++ b/requester/mctp_endpoint_discovery.cpp
@@ -133,7 +133,7 @@
     try
     {
         auto properties = pldm::utils::DBusHandler().getDbusPropertiesVariant(
-            service.c_str(), path.c_str(), EndpointUUID);
+            service.c_str(), path.c_str(), CommonUUID::interface);
 
         if (properties.contains("UUID"))
         {
@@ -201,7 +201,7 @@
     try
     {
         auto service = pldm::utils::DBusHandler().getService(
-            objPath.str.c_str(), EndpointUUID);
+            objPath.str.c_str(), CommonUUID::interface);
         uuid = getEndpointUUIDProp(service, objPath.str);
     }
     catch (const sdbusplus::exception_t& e)

requester/mctp_endpoint_discovery.hpp

diff --git a/requester/mctp_endpoint_discovery.hpp b/requester/mctp_endpoint_discovery.hpp
index 1db6292..3c37e44 100644
--- a/requester/mctp_endpoint_discovery.hpp
+++ b/requester/mctp_endpoint_discovery.hpp
@@ -6,6 +6,7 @@
 #include <libpldm/pldm.h>
 
 #include <sdbusplus/bus/match.hpp>
+#include <xyz/openbmc_project/Common/UUID/common.hpp>
 #include <xyz/openbmc_project/MCTP/Endpoint/client.hpp>
 
 #include <filesystem>
@@ -13,6 +14,7 @@
 #include <vector>
 
 using MCTPEndpoint = sdbusplus::common::xyz::openbmc_project::mctp::Endpoint;
+using CommonUUID = sdbusplus::common::xyz::openbmc_project::common::UUID;
 
 class TestMctpDiscovery;
 
@@ -21,7 +23,6 @@
 
 const std::string emptyUUID = "00000000-0000-0000-0000-000000000000";
 constexpr const char* MCTPService = "au.com.codeconstruct.MCTP1";
-constexpr const char* EndpointUUID = "xyz.openbmc_project.Common.UUID";
 constexpr const char* MCTPPath = "/au/com/codeconstruct/mctp1";
 constexpr const char* MCTPInterfaceCC = "au.com.codeconstruct.MCTP.Endpoint1";
 constexpr const char* MCTPConnectivityProp = "Connectivity";