[commit] r5475 - in trunk/src: mainboard/amd/rumba mainboard/lippert/frontrunner northbridge/amd/gx2 southbridge/amd/cs5536

[repository service]

Author: stepan
Date: Thu Apr 22 11:22:15 2010
New Revision: 5475
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5475

Log:
fix compilation remaining geode boards

Signed-off-by: Stefan Reinauer 
Acked-by: Stefan Reinauer

Modified:
   trunk/src/mainboard/amd/rumba/devicetree.cb
   trunk/src/mainboard/lippert/frontrunner/devicetree.cb
   trunk/src/northbridge/amd/gx2/chipsetinit.c
   trunk/src/northbridge/amd/gx2/grphinit.c
   trunk/src/northbridge/amd/gx2/northbridge.c
   trunk/src/northbridge/amd/gx2/northbridge.h
   trunk/src/northbridge/amd/gx2/northbridgeinit.c
   trunk/src/southbridge/amd/cs5536/chip.h
   trunk/src/southbridge/amd/cs5536/cs5536.c

Modified: trunk/src/mainboard/amd/rumba/devicetree.cb
==============================================================================
--- trunk/src/mainboard/amd/rumba/devicetree.cb	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/mainboard/amd/rumba/devicetree.cb	Thu Apr 22 11:22:15 2010	(r5475)
@@ -1,4 +1,6 @@
 chip northbridge/amd/gx2
+	register "setupflash" = "0"
+  	#register "irqmap" = "0xaa5b"
    device apic_cluster 0 on
    	chip cpu/amd/model_gx2
    		device apic 0 on end

Modified: trunk/src/mainboard/lippert/frontrunner/devicetree.cb
==============================================================================
--- trunk/src/mainboard/lippert/frontrunner/devicetree.cb	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/mainboard/lippert/frontrunner/devicetree.cb	Thu Apr 22 11:22:15 2010	(r5475)
@@ -1,4 +1,6 @@
 chip northbridge/amd/gx2
+  register "setupflash" = "0"
+  #register "irqmap" = "0xaa5b"
   device pci_domain 0 on 
     device pci 0.0 on end
       chip southbridge/amd/cs5535

Modified: trunk/src/northbridge/amd/gx2/chipsetinit.c
==============================================================================
--- trunk/src/northbridge/amd/gx2/chipsetinit.c	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/northbridge/amd/gx2/chipsetinit.c	Thu Apr 22 11:22:15 2010	(r5475)
@@ -12,9 +12,8 @@
 #include <cpu/amd/gx2def.h>
 #include <cpu/x86/msr.h>
 #include <cpu/x86/cache.h>
-#include "../../../southbridge/amd/cs5536/cs5536.h"
+#include "southbridge/amd/cs5536/cs5536.h"

-extern int sizeram(void);
 /* the structs in this file only set msr.lo. But ... that may not always be true */

struct msrinit {
@@ -22,13 +21,13 @@
    msr_t msr;
 };

-/*  Master Configuration Register for Bus Masters.*/
-struct msrinit SB_MASTER_CONF_TABLE[] = {
-	{USB1_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000}},	/*  NOTE: Must be 1st entry in table*/
-	{USB2_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000}},
-	{ATA_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00048f000}},
-	{AC97_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000}},
-	{MDD_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00000f000}},
+/*  Master Configuration Register for Bus Masters. */
+static struct msrinit SB_MASTER_CONF_TABLE[] = {
+	{ USB1_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000} },	/*  NOTE: Must be 1st entry in table */
+	{ USB2_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000} },
+	{ ATA_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00048f000} },
+	{ AC97_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00008f000} },
+	{ MDD_SB_GLD_MSR_CONF,	{.hi=0,.lo=0x00000f000} },
 /* GLPCI_SB_GLD_MSR_CONF,	0x0FFFFFFFF*/
 /* GLCP_SB_GLD_MSR_CONF,	0x0FFFFFFFF*/
 /* GLIU_SB_GLD_MSR_CONF,	0x0*/
@@ -36,37 +35,38 @@
 };

/*  5535_A3 Clock Gating*/
-struct msrinit CS5535_CLOCK_GATING_TABLE[] = {
-	{      USB1_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{      USB2_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{      GLIU_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000004}},
-	{      GLPCI_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{      GLCP_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000004}},
-	{      MDD_SB_GLD_MSR_PM,	{.hi=0,.lo=0x050554111}},
-	{      ATA_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{	AC97_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{0,{0,0}}
+static struct msrinit CS5535_CLOCK_GATING_TABLE[] = {
+	{ USB1_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ USB2_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ GLIU_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000004} },
+	{ GLPCI_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ GLCP_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000004} },
+	{ MDD_SB_GLD_MSR_PM,	{.hi=0, .lo=0x050554111} },
+	{ ATA_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ AC97_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ 0,			{.hi=0, .lo=0x000000000} }
 };

/*  5536 Clock Gating*/
-struct msrinit CS5536_CLOCK_GATING_TABLE[] = {
+static struct msrinit CS5536_CLOCK_GATING_TABLE[] = {
 /* MSR		  Setting*/
-	{      GLIU_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000004}},
-	{      GLPCI_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{      GLCP_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000004}},
-	{      MDD_SB_GLD_MSR_PM,	{.hi=0,.lo=0x050554111}}, /*  SMBus clock gating errata (PBZ 2226 & SiBZ 3977)*/
-	{      ATA_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{      AC97_SB_GLD_MSR_PM,	{.hi=0,.lo=0x000000005}},
-	{0,{0,0}}
+	{ GLIU_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000004} },
+	{ GLPCI_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ GLCP_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000004} },
+	{ MDD_SB_GLD_MSR_PM,	{.hi=0, .lo=0x050554111} }, /*  SMBus clock gating errata (PBZ 2226 & SiBZ 3977) */
+	{ ATA_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ AC97_SB_GLD_MSR_PM,	{.hi=0, .lo=0x000000005} },
+	{ 0,			{.hi=0, .lo=0x000000000} }
 };

+#ifdef UNUSED_CODE
 struct acpiinit {
    unsigned short ioreg; 
    unsigned long regdata;
    unsigned short iolen;
 };

-struct acpiinit acpi_init_table[] = {
+static struct acpiinit acpi_init_table[] = {
    {ACPI_BASE+0x00, 0x01000000, 4},
    {ACPI_BASE+0x08, 0, 4},
    {ACPI_BASE+0x0C, 0, 4},
@@ -81,34 +81,28 @@
    {PM_WKXD, 0x0000000A0, 4},
    {0,0,0}
 };
+#endif

/* return 1 if we are a 5536-based system */
-static int is_5536(void){
+static int is_5536(void)
+{
    msr_t msr;
    msr = rdmsr(GLIU_SB_GLD_MSR_CAP);
    msr.lo >>= 20;
    printk(BIOS_DEBUG, "is_5536: msr.lo is 0x%x(==5 means 5536)\n", msr.lo&0xf);
    return ((msr.lo&0xf) == 5);
 }
-/* ***************************************************************************/
-/* **/
-/* *	pmChipsetInit*/
-/* **/
-/* *	Program ACPI LBAR and initialize ACPI registers.*/
-/* *  */
-/* **/
-/* *	Entry:*/
-/* *		None*/
-/* **/
-/* *	Exit:*/
-/* *		None*/
-/* **/
-/* *	Destroys:*/
-/* *		None*/
-/* **/
-/* ***************************************************************************/
-static void
-pmChipsetInit(void) {
+
+#ifdef UNUSED_CODE
+/*****************************************************************************
+ *
+ *	pmChipsetInit
+ *
+ *	Program ACPI LBAR and initialize ACPI registers.
+ *
+ *****************************************************************************/
+static void pmChipsetInit(void)
+{
    unsigned long val = 0;
    unsigned short port;

@@ -129,40 +123,43 @@
    	
    /*  PM_SED*/
    port =  (PMLogic_BASE + 0x014);
-/* 	mov		eax, 0x057642	; 100ms, works*/
    val =  0x04601		; /*  5ms*/
    outl(val, port);
    
    /*  PM_SIDD*/
    port =  (PMLogic_BASE + 0x020);
-/* 	mov		eax, 0x0AEC84	; 200ms, works*/
    val =  0x08C02		; /*  10ms*/
    outl(val, port);
    
-	/*  GPIO24 OUT_AUX1 function is the external signal for 5535's vsb_working_aux*/
-	/*  which is de-asserted when 5535 enters Standby(S3 or S5) state.*/
-	/*  On Hawk, GPIO24 controls all voltage rails except Vmem and Vstandby.  This means*/
-	/*  GX2 will be fully de-powered if this control de-asserts in S3/S5.*/
-	/* */
-	/*  GPIO24 is setup in preChipsetInit for two reasons*/
-	/*  1. GPIO24 at reset defaults to disabled, since this signal is vsb_work_aux on*/
-	/*     Hawk it controls the FET's for all voltage rails except Vstanby & Vmem.*/
-	/*     BIOS needs to enable GPIO24 as OUT_AUX1 & OUTPUT_EN early so it is driven*/
-	/*     by 5535.*/
-	/*  2. Non-PM builds will require GPIO24 enabled for instant-off power button*/
-	/* */
-
-	/*  GPIO11 OUT_AUX1 function is the external signal for 5535's slp_clk_n which is asserted*/
-	/*  when 5535 enters Sleep(S1) state.*/
-	/*  On Hawk, GPIO11 is connected to control input of external clock generator*/
-	/*  for 14MHz, PCI, USB & LPC clocks.*/
-	/*  Programming of GPIO11 will be done by VSA PM code.  During VSA Init. BIOS writes*/
-	/*  PM Core Virual Register indicating if S1 Clocks should be On or Off. This is based*/
-	/*  on a Setup item.  We do not want to leave GPIO11 enabled because of a Hawk board*/
-	/*  problem.  With GPIO11 enabled in S3, something is back-driving GPIO11 causing it to*/
-	/*  float to 1.6-1.7V.*/
-
+	/* GPIO24 OUT_AUX1 function is the external signal for 5535's
+	 * vsb_working_aux which is de-asserted when 5535 enters Standby (S3 or
+	 * S5) state.  On Hawk, GPIO24 controls all voltage rails except Vmem
+	 * and Vstandby.  This means GX2 will be fully de-powered if this
+	 * control de-asserts in S3/S5.
+	 */
+
+	/* GPIO24 is setup in preChipsetInit for two reasons
+	 * 1. GPIO24 at reset defaults to disabled, since this signal is
+	 *    vsb_work_aux on Hawk it controls the FET's for all voltage
+	 *    rails except Vstanby & Vmem.  BIOS needs to enable GPIO24 as
+	 *    OUT_AUX1 & OUTPUT_EN early so it is driven by 5535.
+	 * 2. Non-PM builds will require GPIO24 enabled for instant-off power
+	 *    button
+	 */
+
+	/* GPIO11 OUT_AUX1 function is the external signal for 5535's
+	 * slp_clk_n which is asserted when 5535 enters Sleep(S1) state.
+	 * On Hawk, GPIO11 is connected to control input of external clock
+	 * generator for 14MHz, PCI, USB & LPC clocks.
+	 * Programming of GPIO11 will be done by VSA PM code.  During VSA
+	 * Init. BIOS writes PM Core Virual Register indicating if S1 Clocks
+	 * should be On or Off. This is based on a Setup item.  We do not want
+	 * to leave GPIO11 enabled because of a Hawk board problem.  With 
+	 * GPIO11 enabled in S3, something is back-driving GPIO11 causing it
+	 * to float to 1.6-1.7V.
+	 */
 }
+#endif

struct FLASH_DEVICE {
    unsigned char fType;		/* Flash type: NOR or NAND */
@@ -170,7 +167,7 @@
    unsigned long fMask;		/* Flash size/mask */
 };

-struct FLASH_DEVICE FlashInitTable[] = {
+static struct FLASH_DEVICE FlashInitTable[] = {
    { FLASH_TYPE_NAND, FLASH_IF_MEM, FLASH_MEM_4K },	/* CS0, or Flash Device 0 */
    { FLASH_TYPE_NONE, 0, 0 },	/* CS1, or Flash Device 1 */
    { FLASH_TYPE_NONE, 0, 0 },	/* CS2, or Flash Device 2 */
@@ -179,12 +176,12 @@

#define FlashInitTableLen (ARRAY_SIZE(FlashInitTable))

-uint32_t FlashPort[] = {
+static uint32_t FlashPort[] = {
    MDD_LBAR_FLSH0,
    MDD_LBAR_FLSH1,
    MDD_LBAR_FLSH2,
    MDD_LBAR_FLSH3
-	};
+};

/***************************************************************************
  *
@@ -194,10 +191,6 @@
  *	correct size info.  Call this routine only if flash needs to be 
  *	configured (don't call it if you want IDE).
  *
- *	Entry:
- *	Exit:
- *	Destroys:
- *
  **************************************************************************/
 static void ChipsetFlashSetup(void)
 {
@@ -242,34 +235,34 @@
    	printk(BIOS_DEBUG, "WRMSR(0x%08X, %08X_%08X)\n", MDD_PIN_OPT, msr.hi, msr.lo);
    	wrmsr(MDD_PIN_OPT, msr);
    }
-	printk(BIOS_DEBUG, "ChipsetFlashSetup--\n");

+	printk(BIOS_DEBUG, "ChipsetFlashSetup--\n");
 }

-/* ***************************************************************************/
-/* **/
-/* *	ChipsetGeodeLinkInit*/
-/* *	Handle chipset specific GeodeLink settings here. */
-/* *	Called from GeodeLink init code.*/
-/* **/
-/* *	Entry:*/
-/* *	Exit:*/
-/* *	Destroys: GS*/
-/* **/
-/* ***************************************************************************/
+/****************************************************************************
+ * 
+ * 	ChipsetGeodeLinkInit
+ *
+ * 	Handle chipset specific GeodeLink settings here.
+ * 	Called from GeodeLink init code.
+ * 
+ ****************************************************************************/
 static void 
-ChipsetGeodeLinkInit(void){
+ChipsetGeodeLinkInit(void)
+{
    msr_t msr;
    unsigned long msrnum;
    unsigned long totalmem;

if (is_5536())
    	return;
+
    /*  SWASIF for A1 DMA */
-	/*  Set all memory to  "just above systop" PCI so DMA will work*/
-	/*  check A1*/
+	/*  Set all memory to  "just above systop" PCI so DMA will work */
+
+	/*  check A1 */
    msrnum = MSR_SB_GLCP + 0x17;
    msr = rdmsr(msrnum);
    if ((msr.lo&0xff) == 0x11)
@@ -280,13 +273,14 @@
    totalmem = ~totalmem;
    totalmem &= 0xfffff;
    msr.lo = totalmem;
-	msr.hi = 0x20000000;				/*  Port 1 (PCI)*/
-	msrnum = MSR_SB_GLIU + 0x20;		/*  */;
+	msr.hi = 0x20000000;		/*  Port 1 (PCI) */
+	msrnum = MSR_SB_GLIU + 0x20;
    wrmsr(msrnum, msr);
 }

void
-chipsetinit (struct northbridge_amd_gx2_config *nb){
+gx2_chipsetinit (struct northbridge_amd_gx2_config *nb)
+{
    msr_t msr;
    struct msrinit *csi;
    int i;
@@ -294,7 +288,10 @@

outb( P80_CHIPSET_INIT, 0x80);
    ChipsetGeodeLinkInit();
-#if 0
+
+	printk(BIOS_DEBUG, "Companion is a %s\n", is_5536()?"CS5536":"CS5535");
+
+#ifdef UNUSED_CODE
    /* we hope NEVER to be in coreboot when S3 resumes 
    if (! IsS3Resume()) */
    {
@@ -313,9 +310,8 @@
    }
 #endif

-
    if (!is_5536()) {
-		/*  Setup USB. Need more details. #118.18*/
+		/*  Setup USB. Need more details. #118.18 */
    	msrnum = MSR_SB_USB1 + 8;
    	msr.lo =  0x00012090;
    	msr.hi = 0;
@@ -328,24 +324,23 @@
    outl	(GPIOL_2_SET, GPIOL_INPUT_ENABLE);
    outl	(GPIOL_2_SET, GPIOL_IN_AUX1_SELECT);

-	/*  Allow IO read and writes during a ATA DMA operation.*/
-	/*   This could be done in the HD rom but do it here for easier debugging.*/
+	/*  Allow IO read and writes during a ATA DMA operation. */
+	/*   This could be done in the HD rom but do it here for easier debugging. */
    
    msrnum = ATA_SB_GLD_MSR_ERR;
    msr = rdmsr(msrnum);
    msr.lo &= ~0x100;
    wrmsr(msrnum, msr);

-	/*  Enable Post Primary IDE.*/
+	/*  Enable Post Primary IDE. */
    msrnum = GLPCI_SB_CTRL;
    msr = rdmsr(msrnum);
    msr.lo |=  GLPCI_CRTL_PPIDE_SET;
    wrmsr(msrnum, msr);

-
-	/*  Set up Master Configuration Register*/
-	/*  If 5536, use same master config settings as 5535, except for OHCI MSRs*/
-	if (is_5536()) 
+	/*  Set up Master Configuration Register */
+	/*  If 5536, use same master config settings as 5535, except for OHCI MSRs */
+	if (is_5536())
    	i = 2;
    else
    	i = 0;
@@ -357,17 +352,15 @@
    	wrmsr(csi->msrnum, msr); // MSR - see table above
    }

+	/*  Flash Setup */
+	printk(BIOS_INFO, "%sDOING ChipsetFlashSetup()!\n",
+			nb->setupflash ? "" : "NOT ");

-	/*  Flash Setup*/
-	printk(BIOS_ERR, "%sDOING ChipsetFlashSetup()!!!!!!!!!!!!!!!!!!\n", nb->setupflash? " " : "NOT");
    if (nb->setupflash)
    	ChipsetFlashSetup();

+	/*  Set up Hardware Clock Gating */

-
-	/* */
-	/*  Set up Hardware Clock Gating*/
-	/* */
    /* if (getnvram(TOKEN_SB_CLK_GATE) != TVALUE_DISABLE) */
    {
    	if (is_5536())
@@ -381,5 +374,5 @@
    		wrmsr(csi->msrnum, msr);	// MSR - see table above
    	}
    }
-
 }
+

Modified: trunk/src/northbridge/amd/gx2/grphinit.c
==============================================================================
--- trunk/src/northbridge/amd/gx2/grphinit.c	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/northbridge/amd/gx2/grphinit.c	Thu Apr 22 11:22:15 2010	(r5475)
@@ -1,7 +1,11 @@
 #include <arch/io.h>
 #include <stdint.h>
 #include <cpu/amd/vr.h>
+#include <device/device.h>
+#include "chip.h"
+#include "northbridge.h"

+// FIXME handle UMA properly.
 #define VIDEO_MB	8					// MB of video memory

Modified: trunk/src/northbridge/amd/gx2/northbridge.c
==============================================================================
--- trunk/src/northbridge/amd/gx2/northbridge.c	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/northbridge/amd/gx2/northbridge.c	Thu Apr 22 11:22:15 2010	(r5475)
@@ -17,8 +17,6 @@
 #include "../../../southbridge/amd/cs5536/cs5536.h"
 #define VIDEO_MB 8

-extern void graphics_init(void);
-
 #define NORTHBRIDGE_FILE "northbridge.c"

/* todo: add a resource record. We don't do this here because this may be called when 
@@ -190,7 +188,7 @@
 }

/* we have to do this here. We have not found a nicer way to do it */
-void
+static void
 setup_gx2(void)
 {

@@ -371,6 +369,7 @@
                 IORESOURCE_FIXED | IORESOURCE_STORED | IORESOURCE_ASSIGNED;
 }

+#if 0
 static void tolm_test(void *gp, struct device *dev, struct resource *new)
 {
    struct resource **best_p = gp;
@@ -382,7 +381,6 @@
    *best_p = best;
 }

-#if 0
 static u32 find_pci_tolm(struct bus *bus)
 {
    struct resource *min;
@@ -396,6 +394,8 @@
    return tolm;
 }
 #endif
+
+// FIXME handle UMA correctly.
 #define FRAMEBUFFERK 4096

static void pci_domain_set_resources(device_t dev)
@@ -484,19 +484,17 @@
 static void enable_dev(struct device *dev)
 {
    printk(BIOS_DEBUG, "gx2 north: enable_dev\n");
-	void northbridgeinit(void);
-	void chipsetinit(struct northbridge_amd_gx2_config *nb);
    void do_vsmbios(void);
+
         /* Set the operations if it is a special bus type */
         if (dev->path.type == DEVICE_PATH_PCI_DOMAIN) {
    	struct northbridge_amd_gx2_config *nb = (struct northbridge_amd_gx2_config *)dev->chip_info;
-		extern void cpubug(void);
    	u32 tomk;
    	printk(BIOS_DEBUG, "DEVICE_PATH_PCI_DOMAIN\n");
    	/* cpubug MUST be called before setup_gx2(), so we force the issue here */
    	northbridgeinit();
-		cpubug();	
-		chipsetinit(nb);
+		cpubug();
+		gx2_chipsetinit(nb);
    	setup_gx2();
    	do_vsmbios();
    	graphics_init();

Modified: trunk/src/northbridge/amd/gx2/northbridge.h
==============================================================================
--- trunk/src/northbridge/amd/gx2/northbridge.h	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/northbridge/amd/gx2/northbridge.h	Thu Apr 22 11:22:15 2010	(r5475)
@@ -1,6 +1,15 @@
 #ifndef NORTHBRIDGE_AMD_GX2_H
 #define NORTHBRIDGE_AMD_GX2_H

-extern unsigned int gx2_scan_root_bus(device_t root, unsigned int max);
+#if !defined(__ROMCC__)  && !defined(ASSEMBLY)
+#if defined(__PRE_RAM__)
+#else
+unsigned int gx2_scan_root_bus(device_t root, unsigned int max);
+int sizeram(void);
+void gx2_chipsetinit (struct northbridge_amd_gx2_config *nb);
+void graphics_init(void);
+void northbridgeinit(void);
+#endif
+#endif

#endif /* NORTHBRIDGE_AMD_GX2_H */

Modified: trunk/src/northbridge/amd/gx2/northbridgeinit.c
==============================================================================
--- trunk/src/northbridge/amd/gx2/northbridgeinit.c	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/northbridge/amd/gx2/northbridgeinit.c	Thu Apr 22 11:22:15 2010	(r5475)
@@ -135,7 +135,6 @@
    */
 /* yes, this duplicates later code, but it seems that is how they want it done. 
   */
-extern int sizeram(void);
 static void
 SysmemInit(struct gliutable *gl)
 {

Modified: trunk/src/southbridge/amd/cs5536/chip.h
==============================================================================
--- trunk/src/southbridge/amd/cs5536/chip.h	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/southbridge/amd/cs5536/chip.h	Thu Apr 22 11:22:15 2010	(r5475)
@@ -40,7 +40,6 @@
    unsigned int com2_address;	/* e.g. 0x2F8 */
    unsigned int com2_irq;	/* e.g. 3 */
    unsigned int unwanted_vpci[MAX_UNWANTED_VPCI];	/* the following allow you to disable unwanted virtualized PCI devices */
-
 };

#endif				/* _SOUTHBRIDGE_AMD_CS5536 */

Modified: trunk/src/southbridge/amd/cs5536/cs5536.c
==============================================================================
--- trunk/src/southbridge/amd/cs5536/cs5536.c	Thu Apr 22 02:52:42 2010	(r5474)
+++ trunk/src/southbridge/amd/cs5536/cs5536.c	Thu Apr 22 11:22:15 2010	(r5475)
@@ -40,7 +40,7 @@
 };

/*	Master Configuration Register for Bus Masters.*/
-struct msrinit SB_MASTER_CONF_TABLE[] = {
+static struct msrinit SB_MASTER_CONF_TABLE[] = {
    {USB2_SB_GLD_MSR_CONF, {.hi = 0,.lo = 0x00008f000}},
    {ATA_SB_GLD_MSR_CONF,  {.hi = 0,.lo = 0x00048f000}},
    {AC97_SB_GLD_MSR_CONF, {.hi = 0,.lo = 0x00008f000}},
@@ -49,7 +49,7 @@
 };

/*	5536 Clock Gating*/
-struct msrinit CS5536_CLOCK_GATING_TABLE[] = {
+static struct msrinit CS5536_CLOCK_GATING_TABLE[] = {
    /* MSR		  Setting*/
    {GLIU_SB_GLD_MSR_PM,  {.hi = 0,.lo = 0x000000004}},
    {GLPCI_SB_GLD_MSR_PM, {.hi = 0,.lo = 0x000000005}},
@@ -65,7 +65,7 @@
    u32 regdata;
 };

-struct acpiinit acpi_init_table[] = {
+static struct acpiinit acpi_init_table[] = {
    {ACPI_IO_BASE + 0x00, 0x01000000},
    {ACPI_IO_BASE + 0x08, 0},
    {ACPI_IO_BASE + 0x0C, 0},
@@ -86,7 +86,7 @@
    unsigned long fMask;	/* Flash size/mask */
 };

-struct FLASH_DEVICE FlashInitTable[] = {
+static struct FLASH_DEVICE FlashInitTable[] = {
    {FLASH_TYPE_NAND, FLASH_IF_MEM, FLASH_MEM_4K},	/* CS0, or Flash Device 0 */
    {FLASH_TYPE_NONE, 0, 0},	/* CS1, or Flash Device 1 */
    {FLASH_TYPE_NONE, 0, 0},	/* CS2, or Flash Device 2 */
@@ -95,7 +95,7 @@

#define FlashInitTableLen (ARRAY_SIZE(FlashInitTable))

-u32 FlashPort[] = {
+static u32 FlashPort[] = {
    MDD_LBAR_FLSH0,
    MDD_LBAR_FLSH1,
    MDD_LBAR_FLSH2,
@@ -512,12 +512,16 @@
    }
 }

-/* ***************************************************************************/
-/* **/
-/* *	ChipsetInit */
-/*			Called from northbridge init (Pre-VSA). */
-/* **/
-/* ***************************************************************************/
+/****************************************************************************
+ * 
+ * 	ChipsetInit 
+ *
+ *	Called from northbridge init (Pre-VSA).
+ *
+ *	NOTE! This function is NOT called if the CS5536 is combined with
+ *	an AMD Geode GX2. It's ONLY used on Geode LX based systems.
+ * 
+ ****************************************************************************/
 void chipsetinit(void)
 {
    device_t dev;