Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/75671?usp=email )
Change subject: soc/amd/stoneyridge/acpi/northbridge: drop _STA method from PCI0 scope ......................................................................
soc/amd/stoneyridge/acpi/northbridge: drop _STA method from PCI0 scope
The PCI root complex itself isn't on an enumerable bus, so without providing an _STA method, the device will still be assumed to be present and visible, so this won't change behavior. This also brings Stoneyridge more in line with the newer SoCs.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I663c7bcba89ffe25d0819d83461cb95e10f49028 Reviewed-on: https://review.coreboot.org/c/coreboot/+/75671 Reviewed-by: Arthur Heymans arthur@aheymans.xyz Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/stoneyridge/acpi/northbridge.asl 1 file changed, 0 insertions(+), 5 deletions(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved
diff --git a/src/soc/amd/stoneyridge/acpi/northbridge.asl b/src/soc/amd/stoneyridge/acpi/northbridge.asl index af62abd..6bf2293 100644 --- a/src/soc/amd/stoneyridge/acpi/northbridge.asl +++ b/src/soc/amd/stoneyridge/acpi/northbridge.asl @@ -11,11 +11,6 @@ Return(0) }
-Method(_STA, 0, NotSerialized) -{ - Return(0x0F) /* Status is visible */ -} - Method(_PRT,0, NotSerialized) { If(PICM)