soc/amd/common,stoneyridge: drop invalid hda_soc_ssdt_quirks

Drop the hda_soc_ssdt_quirks function since it doesn't apply for any of
the SoCs supported by the Stoneyridge code which was the only SoC
implementing it. This code was added when commit 91a7abf25c72
("soc/amd/hda: Move HDA PCI device from DSDT to SSDT") rewrote the code
originally added in commit 1587dc8a2b4d ("soc/amd/stoneyridge: Add
northbridge support") as a copy from northbridge/amd/pi/00670F00. This
code was moved around in commit 6580408a7e0a ("amd/pi/hudson: Move audio
to northbridge"), since the HDA controller was moved from the FCH to the
northbridge complex. When the controller was moved, the PCI config space
interface also changed, so those bits are no longer the DisableNoSnoop,
DisableNoSnoopOverride, and EnableNoSnoopRequest bits of the Misc
Control register of the HDA controller, but some bits within the
ClassCodeW field of the ACGAZ Mirrot Reg Ctrl 0 register.

BKDG #55072 Rev 3.04 (Stoneyridge), BKDG #50742 Rev 3.08 (family 15h
model 60h-6fh / 00670F00), and BKDG #52740 Rev 3.05 (family 16h model
30h-3fh) were used as a reference. Only the SoC with BKDG #52740 still
has the HDA controller in the FCH; the other two have it in the
northbridge.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I77fc76752b1c7de62ba8a196f15c198f55be3074
Reviewed-on: https://review.coreboot.org/c/coreboot/+/78940
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

1 files changed, 0 insertions, 47 deletions

diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c
index 081bc1f47d..1e601cf4b7 100644
--- a/src/soc/amd/stoneyridge/northbridge.c
+++ b/src/soc/amd/stoneyridge/northbridge.c
@@ -3,7 +3,6 @@
 #include <assert.h>
 #include <amdblocks/acpi.h>
 #include <amdblocks/biosram.h>
-#include <amdblocks/hda.h>
 #include <device/pci_ops.h>
 #include <arch/hpet.h>
 #include <arch/ioapic.h>
@@ -408,49 +407,3 @@ void SetNbMidParams(GNB_MID_CONFIGURATION *params)
 	params->iGpuVgaMode = 0;
 	params->GnbIoapicAddress = IO_APIC2_ADDR;
 }
-
-void hda_soc_ssdt_quirks(const struct device *dev)
-{
-	const char *scope = acpi_device_path(dev);
-	static const struct fieldlist list[] = {
-		FIELDLIST_OFFSET(0x42),
-		FIELDLIST_NAMESTR("NSDI", 1),
-		FIELDLIST_NAMESTR("NSDO", 1),
-		FIELDLIST_NAMESTR("NSEN", 1),
-	};
-	struct opregion opreg = OPREGION("AZPD", PCI_CONFIG, 0x0, 0x100);
-
-	assert(scope);
-
-	acpigen_write_scope(scope);
-
-	/*
-	 * OperationRegion(AZPD, PCI_Config, 0x00, 0x100)
-	 * Field (AZPD, AnyAcc, NoLock, Preserve) {
-	 *	Offset (0x42),
-	 *	NSDI, 1,
-	 *	NSDO, 1,
-	 *	NSEN, 1,
-	 * }
-	 */
-	acpigen_write_opregion(&opreg);
-	acpigen_write_field(opreg.name, list, ARRAY_SIZE(list),
-			    FIELD_ANYACC | FIELD_NOLOCK | FIELD_PRESERVE);
-
-	/*
-	 * Method (_INI, 0, NotSerialized) {
-	 *	Store (Zero, NSEN)
-	 *	Store (One, NSDO)
-	 *	Store (One, NSDI)
-	 * }
-	 */
-	acpigen_write_method("_INI", 0);
-
-	acpigen_write_store_op_to_namestr(ZERO_OP, "NSEN");
-	acpigen_write_store_op_to_namestr(ONE_OP, "NSDO");
-	acpigen_write_store_op_to_namestr(ONE_OP, "NSDI");
-
-	acpigen_pop_len(); /* Method _INI */
-
-	acpigen_pop_len(); /* Scope */
-}