Arthur Heymans has uploaded this change for review.

View Change

{soc/amd,sb/amd/hudson}: Fix generating the ACPI mcfg

The last argument for acpi_fill_mcfg is the last PCI bus, not the
total number of busses.

Change-Id: I8887e14128dbe54688eb6e803d6694b7c29956c1
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
---
M src/soc/amd/picasso/northbridge.c
M src/soc/amd/stoneyridge/northbridge.c
M src/southbridge/amd/pi/hudson/lpc.c
3 files changed, 3 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/35872/1
diff --git a/src/soc/amd/picasso/northbridge.c b/src/soc/amd/picasso/northbridge.c
index 36135f9..756537b 100644
--- a/src/soc/amd/picasso/northbridge.c
+++ b/src/soc/amd/picasso/northbridge.c
@@ -167,7 +167,7 @@
CONFIG_MMCONF_BASE_ADDRESS,
0,
0,
- CONFIG_MMCONF_BUS_NUMBER);
+ CONFIG_MMCONF_BUS_NUMBER - 1);

return current;
}
diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c
index 044a1b0..2304e73 100644
--- a/src/soc/amd/stoneyridge/northbridge.c
+++ b/src/soc/amd/stoneyridge/northbridge.c
@@ -183,7 +183,7 @@
CONFIG_MMCONF_BASE_ADDRESS,
0,
0,
- CONFIG_MMCONF_BUS_NUMBER);
+ CONFIG_MMCONF_BUS_NUMBER - 1);

return current;
}
diff --git a/src/southbridge/amd/pi/hudson/lpc.c b/src/southbridge/amd/pi/hudson/lpc.c
index 02123a1..cf73a8c 100644
--- a/src/southbridge/amd/pi/hudson/lpc.c
+++ b/src/southbridge/amd/pi/hudson/lpc.c
@@ -336,7 +336,7 @@
CONFIG_MMCONF_BASE_ADDRESS,
0,
0,
- CONFIG_MMCONF_BUS_NUMBER);
+ CONFIG_MMCONF_BUS_NUMBER - 1);
return current;
}


To view, visit change 35872. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8887e14128dbe54688eb6e803d6694b7c29956c1
Gerrit-Change-Number: 35872
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-MessageType: newchange