tree 5222e9f5a03893bd28ebd3155e4b8414a96f5c45
parent c33a039b56fc5789ae71289adaca1e572a48d318
author Nikhil Namjoshi <nikhilnamjoshi@google.com> 1663102484 +0000
committer Nikhil Namjoshi <namjoshiniks@gmail.com> 1663877071 +0000

Memory: Fix the missing fields in Dimm Response

The Dimm response has some missing fields i.e. Manufacturer and
FirmwareRevision, due to some properties not being unpacked. This
commit fixes that issue.

Tested:

Redfish validator passed

Dimm Response before fix shows missing Manufacturer and
FirmwareRevision fields

```
wget -qO- http://localhost:80/redfish/v1/Systems/system/Memory/dimm0

"@odata.id": "/redfish/v1/Systems/system/Memory/dimm0",
  "@odata.type": "#Memory.v1_11_0.Memory",
  "AllowedSpeedsMHz": [],
  "BaseModuleType": "...",
  "BusWidthBits": ...,
  "CapacityMiB": .....,
  "DataWidthBits": ..,
  "ErrorCorrection": ".....",
     <<<======= Missing FirmwareRevision field
  "Id": "dimm0",
  "Links": {
    "Chassis": {
      "@odata.id": "...."
    }
  },
  "Location": {
    "PartLocation": {
      "LocationType": "Slot",
      "ServiceLabel": "DIMM6"
    }
  },
     <<<<==== Missing Manufacturer field
  "MemoryDeviceType": "...",
  "MemoryType": "....",
  "Model": "",
  "Name": "DIMM Slot",
  "OperatingSpeedMhz": ....,
  "PartNumber": "......",
  "RankCount": ....,
  "SerialNumber": "....",
  "SparePartNumber": "..",
  "Status": {
    "Health": "OK",
    "HealthRollup": "OK",
    "State": "Enabled"
  }
}
```

Dimm Response after fix shows Manufacturer and
FirmwareRevision fields present

```
wget -qO- http://localhost:80/redfish/v1/Systems/system/Memory/dimm0
{
  "@odata.id": "/redfish/v1/Systems/system/Memory/dimm0",
  "@odata.type": "#Memory.v1_11_0.Memory",
  "AllowedSpeedsMHz": [],
  "BaseModuleType": "....",
  "BusWidthBits": ...,
  "CapacityMiB": ....,
  "DataWidthBits": ...,
  "ErrorCorrection": "...",
  "FirmwareRevision": "...",
  "Id": "dimm0",
  "Links": {
    "Chassis": {
      "@odata.id": "...."
    }
  },
  "Location": {
    "PartLocation": {
      "LocationType": "Slot",
      "ServiceLabel": "DIMM6"
    }
  },
  "Manufacturer": "....",
  "MemoryDeviceType": "...",
  "MemoryType": "...",
  "Model": "",
  "Name": "DIMM Slot",
  "OperatingSpeedMhz": ...,
  "PartNumber": "...",
  "RankCount": ...,
  "SerialNumber": ".....",
  "SparePartNumber": "...",
  "Status": {
    "Health": "OK",
    "HealthRollup": "OK",
    "State": "Enabled"
  }
}
```

Signed-off-by: Nikhil Namjoshi <nikhilnamjoshi@google.com>
Change-Id: I4de83a5038fe77e19a1b8b16e479348677fb30c1