Publish Location under PCIeDevice schema (#1083)

The commit publishes Location code under PCIeDevice schema.

Validator was executed and no new error was found.

Test:
curl -k -H "X-Auth-Token: $bmc_token" -X GET "https://${BMC_IP}/redfish/
v1/Systems/system/PCIeDevices/chassis_motherboard_pcieslot0_pcie_card0"
{
  "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/
   chassis_motherboard_pcieslot0_pcie_card0",
  "@odata.type": "#PCIeDevice.v1_9_0.PCIeDevice",
  "Id": "chassis_motherboard_pcieslot0_pcie_card0",
  "Oem": {
    "IBM": {
      "@odata.type": "#IBMPCIeDevice.v1_0_0.IBM",
      "LinkReset": false
    }
  },
  "PCIeFunctions": {
    "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/
    chassis_motherboard_pcieslot0_pcie_card0/PCIeFunctions"
  },
  "PCIeInterface": {
    "LanesInUse": null
  },
  "Slot": {
    "Location": {
      "PartLocation": {
        "ServiceLabel": "U78DA.ND0.WZS003T-P0-C0"
      }
    },
    "SlotType": "FullLength"
  },
  "Status": {
    "Health": "OK",
    "State": "Enabled"
  }
}

Change-Id: Iba1b98734fdfee43c2995f6c9f1f2660e196814a

Signed-off-by: Sunny Srivastava <[email protected]>
Co-authored-by: Sunny Srivastava <[email protected]>
Signed-off-by: Souvik Roy <[email protected]>

Author: 2 people

redfish-core/lib/pcie.hpp

@@ -523,6 +523,46 @@ inline void getPCIeDeviceAsset(
         });
 }
 
+inline void getPCIeDeviceLocation(
+    const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+    const std::string& pcieDevicePath, const std::string& service)
+{
+    sdbusplus::asio::getAllProperties(
+        *crow::connections::systemBus, service, pcieDevicePath,
+        "xyz.openbmc_project.Inventory.Decorator.LocationCode",
+        [pcieDevicePath,
+         asyncResp](const boost::system::error_code& ec,
+                    const dbus::utility::DBusPropertiesMap& locationCodeList) {
+            if (ec)
+            {
+                if (ec.value() != EBADR)
+                {
+                    BMCWEB_LOG_ERROR("DBUS response error for Properties{}",
+                                     ec.value());
+                    messages::internalError(asyncResp->res);
+                }
+                return;
+            }
+
+            const std::string* locationCode = nullptr;
+            const bool success = sdbusplus::unpackPropertiesNoThrow(
+                dbus_utils::UnpackErrorPrinter(), locationCodeList,
+                "LocationCode", locationCode);
+
+            if (!success)
+            {
+                messages::internalError(asyncResp->res);
+                return;
+            }
+
+            if (locationCode != nullptr)
+            {
+                asyncResp->res.jsonValue["Slot"]["Location"]["PartLocation"]
+                                        ["ServiceLabel"] = *locationCode;
+            }
+        });
+}
+
 inline void addPCIeDeviceProperties(
     const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
     const std::string& pcieDeviceId,
@@ -676,6 +716,7 @@ inline void afterGetValidPcieDevicePath(
     getPCIeDeviceSlotPath(
         pcieDevicePath, asyncResp,
         std::bind_front(afterGetPCIeDeviceSlotPath, asyncResp));
+    getPCIeDeviceLocation(asyncResp, pcieDevicePath, service);
 }
 
 inline void handlePCIeDeviceGet(