From ba09695b58f7254d646618d1207840e33ca3d1d8 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Thu, 22 Apr 2010 09:22:15 +0000 Subject: fix compilation remaining geode boards Signed-off-by: Stefan Reinauer Acked-by: Stefan Reinauer git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5475 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/mainboard/amd/rumba/devicetree.cb | 2 + src/mainboard/lippert/frontrunner/devicetree.cb | 2 + src/northbridge/amd/gx2/chipsetinit.c | 223 ++++++++++++------------ src/northbridge/amd/gx2/grphinit.c | 4 + src/northbridge/amd/gx2/northbridge.c | 16 +- src/northbridge/amd/gx2/northbridge.h | 11 +- src/northbridge/amd/gx2/northbridgeinit.c | 1 - src/southbridge/amd/cs5536/chip.h | 1 - src/southbridge/amd/cs5536/cs5536.c | 26 +-- 9 files changed, 148 insertions(+), 138 deletions(-) diff --git a/src/mainboard/amd/rumba/devicetree.cb b/src/mainboard/amd/rumba/devicetree.cb index 8c7de64ce7..0c0c8f0c31 100644 --- a/src/mainboard/amd/rumba/devicetree.cb +++ b/src/mainboard/amd/rumba/devicetree.cb @@ -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 diff --git a/src/mainboard/lippert/frontrunner/devicetree.cb b/src/mainboard/lippert/frontrunner/devicetree.cb index 7f16174d60..28c6af67a1 100644 --- a/src/mainboard/lippert/frontrunner/devicetree.cb +++ b/src/mainboard/lippert/frontrunner/devicetree.cb @@ -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 diff --git a/src/northbridge/amd/gx2/chipsetinit.c b/src/northbridge/amd/gx2/chipsetinit.c index cbb4e65620..620f56452d 100644 --- a/src/northbridge/amd/gx2/chipsetinit.c +++ b/src/northbridge/amd/gx2/chipsetinit.c @@ -12,9 +12,8 @@ #include #include #include -#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 @@ struct msrinit { 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 @@ struct msrinit SB_MASTER_CONF_TABLE[] = { }; /* 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 @@ struct acpiinit acpi_init_table[] = { {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 @@ pmChipsetInit(void) { /* 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 @@ struct FLASH_DEVICE { 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 @@ struct FLASH_DEVICE FlashInitTable[] = { #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 @@ uint32_t FlashPort[] = { * 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 @@ static void ChipsetFlashSetup(void) 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 @@ ChipsetGeodeLinkInit(void){ 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 @@ chipsetinit (struct northbridge_amd_gx2_config *nb){ 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 @@ chipsetinit (struct northbridge_amd_gx2_config *nb){ } #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 @@ chipsetinit (struct northbridge_amd_gx2_config *nb){ 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 @@ chipsetinit (struct northbridge_amd_gx2_config *nb){ 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 @@ chipsetinit (struct northbridge_amd_gx2_config *nb){ wrmsr(csi->msrnum, msr); // MSR - see table above } } - } + diff --git a/src/northbridge/amd/gx2/grphinit.c b/src/northbridge/amd/gx2/grphinit.c index 85df9d8c1b..a30ba78373 100644 --- a/src/northbridge/amd/gx2/grphinit.c +++ b/src/northbridge/amd/gx2/grphinit.c @@ -1,7 +1,11 @@ #include #include #include +#include +#include "chip.h" +#include "northbridge.h" +// FIXME handle UMA properly. #define VIDEO_MB 8 // MB of video memory diff --git a/src/northbridge/amd/gx2/northbridge.c b/src/northbridge/amd/gx2/northbridge.c index 190529ce04..e74a8e33f8 100644 --- a/src/northbridge/amd/gx2/northbridge.c +++ b/src/northbridge/amd/gx2/northbridge.c @@ -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 @@ setup_gx2_cache(void) } /* 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 @@ static void ram_resource(device_t dev, unsigned long index, 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 @@ static void tolm_test(void *gp, struct device *dev, struct resource *new) *best_p = best; } -#if 0 static u32 find_pci_tolm(struct bus *bus) { struct resource *min; @@ -396,6 +394,8 @@ static u32 find_pci_tolm(struct bus *bus) return tolm; } #endif + +// FIXME handle UMA correctly. #define FRAMEBUFFERK 4096 static void pci_domain_set_resources(device_t dev) @@ -484,19 +484,17 @@ extern uint64_t high_tables_base, high_tables_size; 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(); diff --git a/src/northbridge/amd/gx2/northbridge.h b/src/northbridge/amd/gx2/northbridge.h index 750ed3b42c..cd3af80cd5 100644 --- a/src/northbridge/amd/gx2/northbridge.h +++ b/src/northbridge/amd/gx2/northbridge.h @@ -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 */ diff --git a/src/northbridge/amd/gx2/northbridgeinit.c b/src/northbridge/amd/gx2/northbridgeinit.c index 84ecbf09ed..4ad5af23bc 100644 --- a/src/northbridge/amd/gx2/northbridgeinit.c +++ b/src/northbridge/amd/gx2/northbridgeinit.c @@ -135,7 +135,6 @@ ShadowInit(struct gliutable *gl) */ /* 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) { diff --git a/src/southbridge/amd/cs5536/chip.h b/src/southbridge/amd/cs5536/chip.h index 7d9db4e3b3..ad9e736dec 100644 --- a/src/southbridge/amd/cs5536/chip.h +++ b/src/southbridge/amd/cs5536/chip.h @@ -40,7 +40,6 @@ struct southbridge_amd_cs5536_config { 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 */ diff --git a/src/southbridge/amd/cs5536/cs5536.c b/src/southbridge/amd/cs5536/cs5536.c index 2a3662c517..f5de337bbe 100644 --- a/src/southbridge/amd/cs5536/cs5536.c +++ b/src/southbridge/amd/cs5536/cs5536.c @@ -40,7 +40,7 @@ struct msrinit { }; /* 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 @@ struct msrinit SB_MASTER_CONF_TABLE[] = { }; /* 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 @@ struct acpiinit { 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 @@ struct FLASH_DEVICE { 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 @@ struct FLASH_DEVICE FlashInitTable[] = { #define FlashInitTableLen (ARRAY_SIZE(FlashInitTable)) -u32 FlashPort[] = { +static u32 FlashPort[] = { MDD_LBAR_FLSH0, MDD_LBAR_FLSH1, MDD_LBAR_FLSH2, @@ -512,12 +512,16 @@ static void enable_USB_port4(struct southbridge_amd_cs5536_config *sb) } } -/* ***************************************************************************/ -/* **/ -/* * 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; -- cgit v1.2.3