[coreboot-gerrit] New patch to review for coreboot: 1fa51de AGESA fam12: Fix entry to cimx/sb700

[Kyösti Mälkki (kyosti.malkki@gmail.com)]

Kyösti Mälkki (kyosti.malkki at 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 at 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 at 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__);
 }