commit 6fde5971ff53f43ece1aa3977b0689390934b06e
author Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Thu Jun 15 07:56:58 2023 +0000
committer Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Wed Jun 21 16:42:31 2023 +0000
tree c10d10fd1f4adabdbd16bfe08716b3ad17abbc57
parent 768a143dac3165cf2869fc92cccb1036f66b424f

Populate Memory attributes

Change MemoryMedia, Slot and Socket memory attributes to be byte, in
line with Phosphor-dbus-interfaces.

Tested:
1. Redfish validator - passed for this new change
2. Populated MiemoryMedia, Slot and Socket values as expected.
GET https://<BMC_IP>/redfish/v1/Systems/system/Memory/dimm0

Signed-off-by: poram srinivasa rao <poramx.srinivasa.rao@intel.com>
Signed-off-by: Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ib4894aca135af79d01ff0fda0ae8f3f1d03ea278

redfish-core/lib/memory.hpp

diff --git a/redfish-core/lib/memory.hpp b/redfish-core/lib/memory.hpp
index 328cc09..c85ff26 100644
--- a/redfish-core/lib/memory.hpp
+++ b/redfish-core/lib/memory.hpp
@@ -417,10 +417,10 @@
     const uint8_t* memoryAttributes = nullptr;
     const uint16_t* memoryConfiguredSpeedInMhz = nullptr;
     const std::string* memoryType = nullptr;
-    const std::string* channel = nullptr;
-    const std::string* memoryController = nullptr;
-    const std::string* slot = nullptr;
-    const std::string* socket = nullptr;
+    const std::uint8_t* channel = nullptr;
+    const std::uint8_t* memoryController = nullptr;
+    const std::uint8_t* slot = nullptr;
+    const std::uint8_t* socket = nullptr;
     const std::string* sparePartNumber = nullptr;
     const std::string* model = nullptr;
     const std::string* locationCode = nullptr;