From 65bcb57eea2c1c0a482f2678185920e24513b303 Mon Sep 17 00:00:00 2001 From: Kapil Porwal Date: Mon, 28 Nov 2022 18:53:40 +0530 Subject: soc/intel/cmn/block/{pcie/rtd3,usb4}: Use helper functions for _DSD BUG=b:259716145 TEST=Verified SSDT on google/rex. Signed-off-by: Kapil Porwal Change-Id: Ib57dea9b16e4590ca2d75ac1512fdaf773ec50f6 Reviewed-on: https://review.coreboot.org/c/coreboot/+/70065 Reviewed-by: Tarun Tuli Reviewed-by: Subrata Banik Reviewed-by: Eric Lai Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/pcie/rtd3/rtd3.c | 38 ++++------------------------- 1 file changed, 5 insertions(+), 33 deletions(-) (limited to 'src/soc/intel/common/block/pcie') diff --git a/src/soc/intel/common/block/pcie/rtd3/rtd3.c b/src/soc/intel/common/block/pcie/rtd3/rtd3.c index 2c60eff069..f9b587df3a 100644 --- a/src/soc/intel/common/block/pcie/rtd3/rtd3.c +++ b/src/soc/intel/common/block/pcie/rtd3/rtd3.c @@ -13,25 +13,6 @@ #include #include "chip.h" -/* - * The "ExternalFacingPort" and "HotPlugSupportInD3" properties are defined at - * https://docs.microsoft.com/en-us/windows-hardware/drivers/pci/dsd-for-pcie-root-ports - */ -#define PCIE_EXTERNAL_PORT_UUID "EFCC06CC-73AC-4BC3-BFF0-76143807C389" -#define PCIE_EXTERNAL_PORT_PROPERTY "ExternalFacingPort" - -#define PCIE_HOTPLUG_IN_D3_UUID "6211E2C0-58A3-4AF3-90E1-927A4E0C55A4" -#define PCIE_HOTPLUG_IN_D3_PROPERTY "HotPlugSupportInD3" - -/* - * This UUID and the resulting ACPI Device Property is defined by the - * Power Management for Storage Hardware Devices: - * - * https://docs.microsoft.com/en-us/windows-hardware/design/component-guidelines/power-management-for-storage-hardware-devices-intro - */ -#define PCIE_RTD3_STORAGE_UUID "5025030F-842F-4AB4-A561-99A5189762D0" -#define PCIE_RTD3_STORAGE_PROPERTY "StorageD3Enable" - /* PCIe Root Port registers for link status and L23 control. */ #define PCH_PCIE_CFG_LSTS 0x52 /* Link Status Register */ #define PCH_PCIE_CFG_SPR 0xe0 /* Scratchpad */ @@ -367,7 +348,7 @@ static void pcie_rtd3_acpi_fill_ssdt(const struct device *dev) FIELDLIST_NAMESTR(ACPI_REG_PCI_L23_RDY_DETECT, 1), }; int pcie_rp; - struct acpi_dp *dsd, *pkg; + struct acpi_dp *dsd; if (!is_dev_enabled(parent)) { printk(BIOS_ERR, "%s: root port not enabled\n", __func__); @@ -474,16 +455,11 @@ static void pcie_rtd3_acpi_fill_ssdt(const struct device *dev) /* Indicate to the OS that device supports hotplug in D3. */ dsd = acpi_dp_new_table("_DSD"); - pkg = acpi_dp_new_table(PCIE_HOTPLUG_IN_D3_UUID); - acpi_dp_add_integer(pkg, PCIE_HOTPLUG_IN_D3_PROPERTY, 1); - acpi_dp_add_package(dsd, pkg); + acpi_device_add_hotplug_support_in_d3(dsd); /* Indicate to the OS if the device provides an External facing port. */ - if (config->add_acpi_external_facing_port) { - pkg = acpi_dp_new_table(PCIE_EXTERNAL_PORT_UUID); - acpi_dp_add_integer(pkg, PCIE_EXTERNAL_PORT_PROPERTY, 1); - acpi_dp_add_package(dsd, pkg); - } + if (config->add_acpi_external_facing_port) + acpi_device_add_external_facing_port(dsd); /* Indicate to the OS if the device has DMA property. */ if (config->add_acpi_dma_property) @@ -502,11 +478,7 @@ static void pcie_rtd3_acpi_fill_ssdt(const struct device *dev) acpigen_write_STA(ACPI_STATUS_DEVICE_ALL_ON); acpigen_write_name_integer("_S0W", ACPI_DEVICE_SLEEP_D3_COLD); - dsd = acpi_dp_new_table("_DSD"); - pkg = acpi_dp_new_table(PCIE_RTD3_STORAGE_UUID); - acpi_dp_add_integer(pkg, PCIE_RTD3_STORAGE_PROPERTY, 1); - acpi_dp_add_package(dsd, pkg); - acpi_dp_write(dsd); + acpi_device_add_storage_d3_enable(NULL); acpigen_pop_len(); /* Device */ -- cgit v1.2.3