commit 92fb0b556b525ca3f94d11e36d17cd8f5d26c017
author Manojkiran Eda <manojkiran.eda@gmail.com> Wed Apr 17 10:48:17 2024 +0530
committer ManojKiran Eda <manojkiran.eda@gmail.com> Wed Apr 17 05:26:14 2024 +0000
tree 6e7a93697cc847ff938d63414a8534d104b45e2f
parent 15b9411186c9a89c3996fcd5ead17359918cb7ef

Revert "Adding severity when reporting an error"

This reverts commit d28f08cf0238435ac92184fabaafd9c76bff62cf.

Reason for revert : Pel severity is completely openpower-specific , we should not polluting common code with IBM specific things, hence reverting this change for now.

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

host-bmc/dbus_to_host_effecters.cpp

diff --git a/host-bmc/dbus_to_host_effecters.cpp b/host-bmc/dbus_to_host_effecters.cpp
index 21e893d..d5aeb58 100644
--- a/host-bmc/dbus_to_host_effecters.cpp
+++ b/host-bmc/dbus_to_host_effecters.cpp
@@ -289,14 +289,14 @@
             error("Failed to decode setStateEffecterStates response, rc {RC}",
                   "RC", rc);
             pldm::utils::reportError(
-                "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed");
+                "xyz.openbmc_project.bmc.pldm.SetHostEffecterFailed");
         }
         if (completionCode)
         {
             error("Failed to set a Host effecter, cc = {CC}", "CC",
                   static_cast<unsigned>(completionCode));
             pldm::utils::reportError(
-                "xyz.openbmc_project.PLDM.Error.SetHostEffecterFailed");
+                "xyz.openbmc_project.bmc.pldm.SetHostEffecterFailed");
         }
     };
 

host-bmc/host_pdr_handler.cpp

diff --git a/host-bmc/host_pdr_handler.cpp b/host-bmc/host_pdr_handler.cpp
index 0d85d2b..72f2478 100644
--- a/host-bmc/host_pdr_handler.cpp
+++ b/host-bmc/host_pdr_handler.cpp
@@ -773,7 +773,7 @@
         {
             error("Failed to get State sensor PDR");
             pldm::utils::reportError(
-                "xyz.openbmc_project.PLDM.Error.SetHostSensorState.GetStateSensorPDRFail");
+                "xyz.openbmc_project.bmc.pldm.InternalFailure");
             return;
         }
 
@@ -807,7 +807,7 @@
                         "Failed to encode_get_state_sensor_readings_req, rc = {RC}",
                         "RC", rc);
                     pldm::utils::reportError(
-                        "xyz.openbmc_project.PLDM.Error.SetHostSensorState.EncodeStateSensorFail");
+                        "xyz.openbmc_project.bmc.pldm.InternalFailure");
                     return;
                 }
 
@@ -834,6 +834,8 @@
                             "Failed to decode_get_state_sensor_readings_resp, rc = {RC} cc = {CC}",
                             "RC", rc, "CC",
                             static_cast<unsigned>(completionCode));
+                        pldm::utils::reportError(
+                            "xyz.openbmc_project.bmc.pldm.InternalFailure");
                     }
 
                     uint8_t eventState;