commit 3e8c7dfce4869711e22ccc76c90cebb55984d36c
author Spencer Ku <Spencer.Ku@quantatw.com> Thu Sep 02 16:17:52 2021 +0800
committer Ed Tanous <ed@tanous.net> Sat Sep 25 15:25:29 2021 +0000
tree 14dae9516b0bc384b5225bd17c8c56705cc7bf85
parent 81856681e1feabe267c0d1998182c9226106aa6d

PCIe: Implement "PcieType" PCIe device property

This commit publishes PCIe device property "PcieType" which
defined in the Redfish PCIeDevice schema.

New property:
PCIeType   : The PCIe interface generation in use by the device.

Dbus interfaces dependency PR:
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/46437

Peci-pcie dependency PR:
https://gerrit.openbmc-project.xyz/c/openbmc/peci-pcie/+/46438

Sample output:

/redfish/v1/Systems/systemPCIeDevices/S0B1D0/
{
  "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/S0B1D0",
  "@odata.type": "#PCIeDevice.v1_4_0.PCIeDevice",
  "DeviceType": "SingleFunction",
  "Id": "S0B1D0",
  "Manufacturer": "PLDA",
  "Name": "PCIe Device",
  "PCIeFunctions": {
    "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/S0B1D0/PCIeFunctions"
  },
  "PCIeInterface": {
    "PcieType": "Gen2"
  }
}

Signed-off-by: Spencer Ku <Spencer.Ku@quantatw.com>
Change-Id: Ic6bc19455c648a3d256856938a79a4265ce8eb59

redfish-core/lib/pcie.hpp

diff --git a/redfish-core/lib/pcie.hpp b/redfish-core/lib/pcie.hpp
index e534434..79d9c1e 100644
--- a/redfish-core/lib/pcie.hpp
+++ b/redfish-core/lib/pcie.hpp
@@ -94,6 +94,38 @@
             });
 }
 
+inline std::string analysisGeneration(const std::string& pcieType)
+{
+    if (pcieType ==
+        "xyz.openbmc_project.Inventory.Item.PCIeSlot.Generations.Gen1")
+    {
+        return "Gen1";
+    }
+    if (pcieType ==
+        "xyz.openbmc_project.Inventory.Item.PCIeSlot.Generations.Gen2")
+    {
+        return "Gen2";
+    }
+    if (pcieType ==
+        "xyz.openbmc_project.Inventory.Item.PCIeSlot.Generations.Gen3")
+    {
+        return "Gen3";
+    }
+    if (pcieType ==
+        "xyz.openbmc_project.Inventory.Item.PCIeSlot.Generations.Gen4")
+    {
+        return "Gen4";
+    }
+    if (pcieType ==
+        "xyz.openbmc_project.Inventory.Item.PCIeSlot.Generations.Gen5")
+    {
+        return "Gen5";
+    }
+
+    // Unknown or others
+    return "";
+}
+
 inline void requestRoutesSystemPCIeDevice(App& app)
 {
     BMCWEB_ROUTE(app, "/redfish/v1/Systems/system/PCIeDevices/<str>/")
@@ -150,6 +182,19 @@
                         asyncResp->res.jsonValue["DeviceType"] = *property;
                     }
 
+                    if (std::string* property = std::get_if<std::string>(
+                            &pcieDevProperties["PcieType"]);
+                        property)
+                    {
+                        std::string pcieType = analysisGeneration(*property);
+                        if (!pcieType.empty())
+                        {
+                            asyncResp->res
+                                .jsonValue["PCIeInterface"]["PcieType"] =
+                                pcieType;
+                        }
+                    }
+
                     asyncResp->res.jsonValue["PCIeFunctions"] = {
                         {"@odata.id",
                          "/redfish/v1/Systems/system/PCIeDevices/" + device +