From bc76109df205cc279c659c93186b004d43a8ddc8 Mon Sep 17 00:00:00 2001 From: Kapil Porwal Date: Thu, 24 Nov 2022 17:58:34 +0530 Subject: [PATCH] {soc/intel/cmn/pcie, mb/google/volteer}: Rename `is_external` variable Name a variable based on its utility. `is_external` variable adds `ExternalFacingPort` _DSD property to an ACPI device hence rename it to `add_acpi_external_facing_port`. BUG=b:259716145 TEST=Build google/rex with this flag and verify it in SSDT at runtime. SSDT snippet: Name (_DSD, Package (0x04) // _DSD: Device-Specific Data { ToUUID ("6211e2c0-58a3-4af3-90e1-927a4e0c55a4"), Package (0x01) { Package (0x02) { "HotPlugSupportInD3", One } }, ToUUID ("efcc06cc-73ac-4bc3-bff0-76143807c389"), Package (0x01) { Package (0x02) { "ExternalFacingPort", One } } }) Signed-off-by: Kapil Porwal Change-Id: I65100283ed9b65037c9890f28ecab41fcfa25d83 Reviewed-on: https://review.coreboot.org/c/coreboot/+/69970 Reviewed-by: Eric Lai Reviewed-by: Subrata Banik Tested-by: build bot (Jenkins) --- src/mainboard/google/volteer/variants/baseboard/devicetree.cb | 2 +- src/soc/intel/common/block/pcie/rtd3/chip.h | 2 +- src/soc/intel/common/block/pcie/rtd3/rtd3.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb index 9f3987e672..3a54d3b728 100644 --- a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb @@ -489,7 +489,7 @@ chip soc/intel/tigerlake chip soc/intel/common/block/pcie/rtd3 register "reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_H3)" register "srcclk_pin" = "3" - register "is_external" = "1" + register "add_acpi_external_facing_port" = "1" device generic 1 on probe DB_SD SD_RTS5261 end diff --git a/src/soc/intel/common/block/pcie/rtd3/chip.h b/src/soc/intel/common/block/pcie/rtd3/chip.h index 652aafffd7..8b62406558 100644 --- a/src/soc/intel/common/block/pcie/rtd3/chip.h +++ b/src/soc/intel/common/block/pcie/rtd3/chip.h @@ -44,7 +44,7 @@ struct soc_intel_common_block_pcie_rtd3_config { * Add device property indicating the device provides an external PCI port * for the OS to apply security restrictions. */ - bool is_external; + bool add_acpi_external_facing_port; /* * Allow a device to add the RuntimeD3Storage property even if the detected diff --git a/src/soc/intel/common/block/pcie/rtd3/rtd3.c b/src/soc/intel/common/block/pcie/rtd3/rtd3.c index 7a64734b6e..85ee24fbb5 100644 --- a/src/soc/intel/common/block/pcie/rtd3/rtd3.c +++ b/src/soc/intel/common/block/pcie/rtd3/rtd3.c @@ -479,7 +479,7 @@ static void pcie_rtd3_acpi_fill_ssdt(const struct device *dev) acpi_dp_add_package(dsd, pkg); /* Indicate to the OS if the device provides an External facing port. */ - if (config->is_external) { + 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);