commit 0e45ced1a8977f3ad29959efbf7bcf5415637645
author Matt Spinler <spinler@us.ibm.com> Mon Feb 12 14:36:07 2018 -0600
committer Matt Spinler <spinler@us.ibm.com> Mon Feb 26 14:50:12 2018 +0000
tree 24889f7aaf7399579243f4567a9f965b1de40500
parent 5e361c5cf7993c56d45e7e1c4d508a83be777018

Call out GPUs using their inventory path

The existing code was calling out GPUs in the metadata
with 'GPU=GPUn' because at the time the GPUs were not
in the inventory.

Now that the GPUs are in the inventory, they can be called
out using their inventory objects which is the correct
way to do it.

Resolves openbmc/openbmc#2869

Change-Id: Ic7b3ac465d482aade532f75ef10cc05df4928767
Signed-off-by: Matt Spinler <spinler@us.ibm.com>

elog-errors.hpp

diff --git a/elog-errors.hpp b/elog-errors.hpp
index 3582c49..150875a 100644
--- a/elog-errors.hpp
+++ b/elog-errors.hpp
@@ -1193,14 +1193,6 @@
     explicit constexpr RAW_STATUS(const char* a) : _entry(entry(str, a)) {};
     type _entry;
 };
-struct GPU
-{
-    static constexpr auto str = "GPU=%s";
-    static constexpr auto str_short = "GPU";
-    using type = std::tuple<std::decay_t<decltype(str)>,const char*>;
-    explicit constexpr GPU(const char* a) : _entry(entry(str, a)) {};
-    type _entry;
-};
 
 }  // namespace _GPUPowerFault
 
@@ -1208,8 +1200,8 @@
 {
     static constexpr auto L = level::ERR;
     using RAW_STATUS = _GPUPowerFault::RAW_STATUS;
-    using GPU = _GPUPowerFault::GPU;
-    using metadata_types = std::tuple<RAW_STATUS, GPU>;
+    using CALLOUT_INVENTORY_PATH = xyz::openbmc_project::Common::Callout::Inventory::CALLOUT_INVENTORY_PATH;
+    using metadata_types = std::tuple<RAW_STATUS, CALLOUT_INVENTORY_PATH>;
 
 };
 
@@ -1249,14 +1241,6 @@
     explicit constexpr RAW_STATUS(const char* a) : _entry(entry(str, a)) {};
     type _entry;
 };
-struct GPU
-{
-    static constexpr auto str = "GPU=%s";
-    static constexpr auto str_short = "GPU";
-    using type = std::tuple<std::decay_t<decltype(str)>,const char*>;
-    explicit constexpr GPU(const char* a) : _entry(entry(str, a)) {};
-    type _entry;
-};
 
 }  // namespace _GPUOverTemp
 
@@ -1264,8 +1248,8 @@
 {
     static constexpr auto L = level::ERR;
     using RAW_STATUS = _GPUOverTemp::RAW_STATUS;
-    using GPU = _GPUOverTemp::GPU;
-    using metadata_types = std::tuple<RAW_STATUS, GPU>;
+    using CALLOUT_INVENTORY_PATH = xyz::openbmc_project::Common::Callout::Inventory::CALLOUT_INVENTORY_PATH;
+    using metadata_types = std::tuple<RAW_STATUS, CALLOUT_INVENTORY_PATH>;
 
 };