commit 8e715ae9c8be2993e1bbcf369e236b792dce5499
author Manojkiran Eda <manojkiran.eda@gmail.com> Sun Jul 04 21:46:43 2021 +0530
committer Tom Joseph <rushtotom@gmail.com> Mon Aug 21 02:12:55 2023 +0000
tree ef3751308bfd42385acc6924b1ac84809f9c9e37
parent cbdc272356e9b37dd060d041da8ca9654531999d

pldm: Enhance the find & add pdr utilities

Change the names of pldm_entity_association_tree_find() and
pldm_entity_association_tree_add() in accordance with
the changes in libpldm to
pldm_entity_association_tree_find_with_locality() and
pldm_entity_association_tree_add_entity. As per -
https://gerrit.openbmc.org/c/openbmc/libpldm/+/63585 and
https://gerrit.openbmc.org/c/openbmc/libpldm/+/65623. Also,
change by adding parameters according to the newly changed
signatures.

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Ib721c496314c963759a1d51eed481356cb296006

host-bmc/host_pdr_handler.cpp

diff --git a/host-bmc/host_pdr_handler.cpp b/host-bmc/host_pdr_handler.cpp
index 89c170e..bcd3928 100644
--- a/host-bmc/host_pdr_handler.cpp
+++ b/host-bmc/host_pdr_handler.cpp
@@ -231,12 +231,13 @@
         pldm_entity parent{};
         if (getParent(entities[i].entity_type, parent))
         {
-            auto node = pldm_entity_association_tree_find(entityTree, &parent);
+            auto node = pldm_entity_association_tree_find_with_locality(
+                entityTree, &parent, true);
             if (node)
             {
-                pldm_entity_association_tree_add(entityTree, &entities[i],
-                                                 0xFFFF, node,
-                                                 entityPdr->association_type);
+                pldm_entity_association_tree_add_entity(
+                    entityTree, &entities[i], 0xFFFF, node,
+                    entityPdr->association_type, false, true, 0xFFFF);
                 merged = true;
             }
         }

libpldmresponder/fru.cpp

diff --git a/libpldmresponder/fru.cpp b/libpldmresponder/fru.cpp
index b135093..a716f5d 100644
--- a/libpldmresponder/fru.cpp
+++ b/libpldmresponder/fru.cpp
@@ -81,7 +81,8 @@
             {
                 pldm_entity node =
                     pldm_entity_extract(objToEntityNode.at(currPath));
-                if (pldm_entity_association_tree_find(entityTree, &node))
+                if (pldm_entity_association_tree_find_with_locality(
+                        entityTree, &node, false))
                 {
                     break;
                 }
@@ -114,19 +115,20 @@
 
                 if (currPath == prePath)
                 {
-                    auto node = pldm_entity_association_tree_add(
+                    auto node = pldm_entity_association_tree_add_entity(
                         entityTree, &entity, 0xFFFF, nullptr,
-                        PLDM_ENTITY_ASSOCIAION_PHYSICAL);
+                        PLDM_ENTITY_ASSOCIAION_PHYSICAL, false, true, 0xFFFF);
                     objToEntityNode[currPath] = node;
                 }
                 else
                 {
                     if (objToEntityNode.contains(prePath))
                     {
-                        auto node = pldm_entity_association_tree_add(
+                        auto node = pldm_entity_association_tree_add_entity(
                             entityTree, &entity, 0xFFFF,
                             objToEntityNode[prePath],
-                            PLDM_ENTITY_ASSOCIAION_PHYSICAL);
+                            PLDM_ENTITY_ASSOCIAION_PHYSICAL, false, true,
+                            0xFFFF);
                         objToEntityNode[currPath] = node;
                     }
                 }