From 7f7b01d46747e0147cac4427a2667dd6bf213a91 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Wed, 5 Oct 2022 14:47:00 +0200 Subject: [PATCH] soc/amd/sata.c: Hook up directly in devicetree Cezanne has two SATA controllers, but doesn't select SOC_AMD_COMMON_BLOCK_SATA, so it's not added to the SATA devices in the Cezanne chipset devicetree. Change-Id: If7f0a9638151cf981d891464a2c3a0ec5fc9c780 Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/68142 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth Reviewed-by: Fred Reitberger --- src/soc/amd/common/block/sata/sata.c | 19 +------------------ src/soc/amd/picasso/chipset.cb | 2 +- src/soc/amd/stoneyridge/chipset_cz.cb | 2 +- src/soc/amd/stoneyridge/chipset_st.cb | 2 +- 4 files changed, 4 insertions(+), 21 deletions(-) diff --git a/src/soc/amd/common/block/sata/sata.c b/src/soc/amd/common/block/sata/sata.c index 19a7d3edd5..6a271682c2 100644 --- a/src/soc/amd/common/block/sata/sata.c +++ b/src/soc/amd/common/block/sata/sata.c @@ -3,7 +3,6 @@ #include #include #include -#include #include static const char *sata_acpi_name(const struct device *dev) @@ -11,7 +10,7 @@ static const char *sata_acpi_name(const struct device *dev) return "STCR"; } -static struct device_operations sata_ops = { +struct device_operations amd_sata_ops = { .read_resources = pci_dev_read_resources, .set_resources = pci_dev_set_resources, .enable_resources = pci_dev_enable_resources, @@ -20,19 +19,3 @@ static struct device_operations sata_ops = { .acpi_name = sata_acpi_name, .acpi_fill_ssdt = acpi_device_write_pci_dev, }; - -static const unsigned short pci_device_ids[] = { - PCI_DID_AMD_CZ_SATA, - PCI_DID_AMD_CZ_SATA_AHCI, - PCI_DID_AMD_FAM17H_SATA_AHCI_VER0, - PCI_DID_AMD_FAM17H_SATA_AHCI_VER1, - PCI_DID_AMD_FAM17H_SATA_AHCI_RAID_VER0, - PCI_DID_AMD_FAM17H_SATA_AHCI_RAID_VER1, - 0 -}; - -static const struct pci_driver sata0_driver __pci_driver = { - .ops = &sata_ops, - .vendor = PCI_VID_AMD, - .devices = pci_device_ids, -}; diff --git a/src/soc/amd/picasso/chipset.cb b/src/soc/amd/picasso/chipset.cb index b4479d7cc1..ab286de864 100644 --- a/src/soc/amd/picasso/chipset.cb +++ b/src/soc/amd/picasso/chipset.cb @@ -30,7 +30,7 @@ chip soc/amd/picasso end device pci 08.2 alias internal_bridge_b off # internal bridge to bus B ops amd_internal_pcie_gpp_ops - device pci 0.0 alias sata off end + device pci 0.0 alias sata off ops amd_sata_ops end device pci 0.1 alias xgbe_0 off end device pci 0.2 alias xgbe_1 off end end diff --git a/src/soc/amd/stoneyridge/chipset_cz.cb b/src/soc/amd/stoneyridge/chipset_cz.cb index dc510e171e..dbbbd4d3cf 100644 --- a/src/soc/amd/stoneyridge/chipset_cz.cb +++ b/src/soc/amd/stoneyridge/chipset_cz.cb @@ -26,7 +26,7 @@ chip soc/amd/stoneyridge device pci 09.0 alias hda_bridge off end # host audio bridge device pci 09.2 alias hda off end # main HD Audio Controller device pci 10.0 alias xhci off ops stoneyridge_usb_ops end - device pci 11.0 alias sata off end + device pci 11.0 alias sata off ops amd_sata_ops end device pci 12.0 alias ehci off ops stoneyridge_usb_ops end device pci 14.0 alias smbus on ops amd_smbus_ops end # primary FCH function device pci 14.3 alias lpc_bridge on ops amd_lpc_ops end diff --git a/src/soc/amd/stoneyridge/chipset_st.cb b/src/soc/amd/stoneyridge/chipset_st.cb index 4d9af9092a..4c244d0f0d 100644 --- a/src/soc/amd/stoneyridge/chipset_st.cb +++ b/src/soc/amd/stoneyridge/chipset_st.cb @@ -20,7 +20,7 @@ chip soc/amd/stoneyridge device pci 09.0 alias hda_bridge off end # host audio bridge device pci 09.2 alias hda off end # main HD Audio Controller device pci 10.0 alias xhci off ops stoneyridge_usb_ops end - device pci 11.0 alias sata off end + device pci 11.0 alias sata off ops amd_sata_ops end device pci 12.0 alias ehci off ops stoneyridge_usb_ops end device pci 14.0 alias smbus on ops amd_smbus_ops end # primary FCH function device pci 14.3 alias lpc_bridge on ops amd_lpc_ops end