Kyösti Mälkki (kyosti.malkki@gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6222
-gerrit
commit 1fa51de89a7d9100b7fa106d736858f0ba860154 Author: Kyösti Mälkki kyosti.malkki@gmail.com Date: Mon Jul 7 18:16:55 2014 +0300
AGESA fam12: Fix entry to cimx/sb700
Move SB700 calls to match comments and changes already made for family14 et al.
Change-Id: I20a84e487ba346f63dd4454447077e0d2fd12c89 Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com --- src/northbridge/amd/agesa/family12/northbridge.c | 31 ++++++++---------------- 1 file changed, 10 insertions(+), 21 deletions(-)
diff --git a/src/northbridge/amd/agesa/family12/northbridge.c b/src/northbridge/amd/agesa/family12/northbridge.c index 65212dc..94de010 100644 --- a/src/northbridge/amd/agesa/family12/northbridge.c +++ b/src/northbridge/amd/agesa/family12/northbridge.c @@ -767,10 +767,16 @@ printk(BIOS_DEBUG, " adsr - leaving this lovely routine.\n");
static void domain_enable_resources(device_t dev) { - /* Must be called after PCI enumeration and resource allocation */ - printk(BIOS_DEBUG, "\nFam12h - northbridge.c - %s - Start.\n",__func__); -// AGESAWRAPPER(amdinitmid); - printk(BIOS_DEBUG, "Fam12h - northbridge.c - %s - End.\n",__func__); + printk(BIOS_DEBUG, "\nFam12h - northbridge.c - %s - Start.\n",__func__); + + /* Must be called after PCI enumeration and resource allocation */ +#if CONFIG_AMD_SB_CIMX + sb_After_Pci_Init(); + sb_Mid_Post_Init(); +#endif + + AGESAWRAPPER(amdinitmid); + printk(BIOS_DEBUG, "Fam12h - northbridge.c - %s - End.\n",__func__); }
@@ -807,23 +813,6 @@ static void cpu_bus_init(device_t dev) { printk(BIOS_DEBUG, "\nFam12h - northbridge.c - %s - Start.\n",__func__); initialize_cpus(dev->link_list); - -#if CONFIG_SOUTHBRIDGE_AMD_CIMX_SB900 - /* Must be called after PCI enumeration and resource allocation */ - printk(BIOS_DEBUG, "\nFam12h - northbridge.c - %s - sb_After_Pci_Init - Start.\n",__func__); - sb_After_Pci_Init (); - printk(BIOS_DEBUG, "Fam12h - northbridge.c - %s - sb_After_Pci_Init - End.\n",__func__); -#endif // #if CONFIG_SOUTHBRIDGE_AMD_CIMX_SB900 - -#if CONFIG_SOUTHBRIDGE_AMD_CIMX_SB900 - /* Must be called after PCI enumeration and resource allocation */ - printk(BIOS_DEBUG, "\nFam12h - northbridge.c - %s - sb_Mid_Post_Init - Start.\n",__func__); - sb_Mid_Post_Init (); - printk(BIOS_DEBUG, "Fam12h - northbridge.c - %s - sb_Mid_Post_Init - End.\n",__func__); -#endif // #if CONFIG_SOUTHBRIDGE_AMD_CIMX_SB900 - - /* Must be called after PCI enumeration and resource allocation */ - AGESAWRAPPER(amdinitmid); printk(BIOS_DEBUG, "Fam12h - northbridge.c - %s - End.\n",__func__); }
-
Kyösti Mälkki