diff options
| author | Robert Zieba <robertzieba@google.com> | 2022-07-11 10:50:25 -0600 |
|---|---|---|
| committer | Martin L Roth <gaumless@tutanota.com> | 2022-07-20 22:23:40 +0000 |
| commit | 5609f7a684c698fff9da764c1868f97f4ee2ba33 (patch) | |
| tree | 223f1a24255193569336eb259eeab1d06f41c3d1 | |
| parent | 65f558f576ac716b0639d8d18834b776a2a9f90b (diff) | |
drivers/pcie/generic: Clean up driver
This removes unneeded and unused functionality in the driver as part of
an effort to make the driver more generic and useful. The things that
have been removed are: `DmaProperty` and its associated `is_untrusted`
config, `_DSD` generation, and the companion device functionality. This
driver isn't currently used anywhere so there won't be any issues from
removing the above functionality.
BUG=b:237682766
TEST=Built and booted coreboot on Skyrim device
Change-Id: I0abd9148ab66ea9426069102ecc8c2fa77fea98e
Signed-off-by: Robert Zieba <robertzieba@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/65797
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/drivers/pcie/generic/chip.h | 9 | ||||
| -rw-r--r-- | src/drivers/pcie/generic/generic.c | 27 |
2 files changed, 7 insertions, 29 deletions
diff --git a/src/drivers/pcie/generic/chip.h b/src/drivers/pcie/generic/chip.h index 5d762ec250..1d9f0a62ed 100644 --- a/src/drivers/pcie/generic/chip.h +++ b/src/drivers/pcie/generic/chip.h @@ -6,15 +6,6 @@ #include <types.h> struct drivers_pcie_generic_config { - bool is_untrusted; - /* - * This needs to be pointed to the device instance in the device tree when - * there is already a device with the root port so that the ACPI code to be - * generated will be added to that existing device. - * By default, an ACPI device named 'DEV0' is created under the root port if - * this does not reference to a device. - */ - DEVTREE_CONST struct device *companion_dev; }; #endif /* _PCIE_GENERIC_H_ */ diff --git a/src/drivers/pcie/generic/generic.c b/src/drivers/pcie/generic/generic.c index 0a9a243518..a8dd1d9745 100644 --- a/src/drivers/pcie/generic/generic.c +++ b/src/drivers/pcie/generic/generic.c @@ -9,17 +9,12 @@ static const char *pcie_generic_acpi_name(const struct device *dev) { - struct drivers_pcie_generic_config *config = dev->chip_info; - - if (config->companion_dev) - return acpi_device_name(config->companion_dev); return "DEV0"; } static void pcie_generic_fill_ssdt(const struct device *dev) { struct drivers_pcie_generic_config *config; - struct acpi_dp *dsd; if (!is_dev_enabled(dev)) return; @@ -27,28 +22,20 @@ static void pcie_generic_fill_ssdt(const struct device *dev) pci_rom_ssdt(dev); config = dev->chip_info; - if (!config || !config->is_untrusted || !dev->bus || !dev->bus->dev) + if (!config || !dev->bus || !dev->bus->dev) return; const char *scope; const char *name; - /* Code will be generated under companion device instead if present. */ - if (config->companion_dev) - scope = acpi_device_path(config->companion_dev); - else - scope = acpi_device_path(dev->bus->dev); + scope = acpi_device_path(dev->bus->dev); name = acpi_device_name(dev); + acpigen_write_scope(scope); - if (!config->companion_dev) { - acpigen_write_device(name); - acpigen_write_ADR_pci_device(dev); - } - dsd = acpi_dp_new_table("_DSD"); - acpi_dp_add_integer(dsd, "DmaProperty", 1); - acpi_dp_write(dsd); - if (!config->companion_dev) - acpigen_write_device_end(); + acpigen_write_device(name); + acpigen_write_ADR_pci_device(dev); + + acpigen_write_device_end(); acpigen_write_scope_end(); printk(BIOS_INFO, "%s.%s: Enable ACPI properties for %s (%s)\n", scope, name, |