[XS] Change in coreboot[master]: soc/amd/common/data_fabric/domain: write _BBN method in SSDT

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74992?usp=email )

Change subject: soc/amd/common/data_fabric/domain: write _BBN method in SSDT
......................................................................

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 
Change-Id: I8badeb0064b498d3f18217ea24bff73676913b02
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74992
Reviewed-by: Arthur Heymans 
Reviewed-by: Nico Huber 
Reviewed-by: Raul Rangel 
Tested-by: build bot (Jenkins) 
---
M src/soc/amd/common/block/data_fabric/domain.c
M src/soc/amd/picasso/acpi/northbridge.asl
2 files changed, 3 insertions(+), 5 deletions(-)

Approvals:
  Arthur Heymans: Looks good to me, approved
  build bot (Jenkins): Verified
  Raul Rangel: Looks good to me, approved
  Nico Huber: Looks good to me, but someone else must approve

diff --git a/src/soc/amd/common/block/data_fabric/domain.c b/src/soc/amd/common/block/data_fabric/domain.c
index aec1596..c032fdb 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 @@
    }

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 688f138..2f73ae4 100644
--- a/src/soc/amd/picasso/acpi/northbridge.asl
+++ b/src/soc/amd/picasso/acpi/northbridge.asl
@@ -6,11 +6,6 @@

/* Describe the Northbridge devices */

-Method(_BBN, 0, NotSerialized)	/* Bus number = 0 */
-{
-	Return(0)
-}
-
 Method(_STA, 0, NotSerialized)
 {
    Return(0x0f)	/* Status is visible */