summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2023-05-05 20:46:11 +0200
committerFelix Held <felix-coreboot@felixheld.de>2023-06-07 00:16:58 +0000
commite4b65cc945bb5256fdb65041b9f4d20c05155cd8 (patch)
tree083b3aa4a2afbd78f521ae749c6c6b69661159ec /src
parent784c9c693fd2398e4283bdcd8051a027268238e0 (diff)
soc/amd/common/data_fabric/domain: write _BBN method in SSDT
Instead of having PCI0's _BBN method in the DSDT that always returns 0, use acpigen_write_BBN to generate the _BBN method that returns the first PCI bus number in the PCI domain/host bridge. TEST=On mandolin the _BBN method in the _SB/PCI0 scope is now in the SSDT instead of the DSDT, but still returns 0. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: I8badeb0064b498d3f18217ea24bff73676913b02 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74992 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Raul Rangel <rrangel@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
-rw-r--r--src/soc/amd/common/block/data_fabric/domain.c3
-rw-r--r--src/soc/amd/picasso/acpi/northbridge.asl5
2 files changed, 3 insertions, 5 deletions
diff --git a/src/soc/amd/common/block/data_fabric/domain.c b/src/soc/amd/common/block/data_fabric/domain.c
index aec1596fc1..c032fdb4bb 100644
--- a/src/soc/amd/common/block/data_fabric/domain.c
+++ b/src/soc/amd/common/block/data_fabric/domain.c
@@ -243,6 +243,9 @@ void amd_pci_domain_fill_ssdt(const struct device *domain)
}
acpigen_write_resourcetemplate_footer();
+
+ acpigen_write_BBN(domain->link_list->secondary);
+
/* Scope */
acpigen_pop_len();
}
diff --git a/src/soc/amd/picasso/acpi/northbridge.asl b/src/soc/amd/picasso/acpi/northbridge.asl
index 688f138123..2f73ae4384 100644
--- a/src/soc/amd/picasso/acpi/northbridge.asl
+++ b/src/soc/amd/picasso/acpi/northbridge.asl
@@ -6,11 +6,6 @@ Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
/* Describe the Northbridge devices */
-Method(_BBN, 0, NotSerialized) /* Bus number = 0 */
-{
- Return(0)
-}
-
Method(_STA, 0, NotSerialized)
{
Return(0x0f) /* Status is visible */