From 8b942e75d2f3c0ccf0198019a706a1c156008b27 Mon Sep 17 00:00:00 2001 From: Uwe Hermann Date: Tue, 13 Nov 2007 16:24:15 +0000 Subject: Random minor cosmetical or coding style fixes (trivial). Signed-off-by: Uwe Hermann Acked-by: Uwe Hermann git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2966 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/mainboard/pcengines/alix1c/Options.lb | 1 - src/mainboard/pcengines/alix1c/cache_as_ram_auto.c | 67 ++++++++++--------- src/mainboard/pcengines/alix1c/irq_tables.c | 77 ++++++++++++---------- src/mainboard/pcengines/alix1c/mainboard.c | 7 +- 4 files changed, 81 insertions(+), 71 deletions(-) (limited to 'src/mainboard') diff --git a/src/mainboard/pcengines/alix1c/Options.lb b/src/mainboard/pcengines/alix1c/Options.lb index c4fce0a4c6..71ef274069 100644 --- a/src/mainboard/pcengines/alix1c/Options.lb +++ b/src/mainboard/pcengines/alix1c/Options.lb @@ -104,7 +104,6 @@ default CONFIG_TSC_X86RDTSC_CALIBRATE_WITH_TIMER2=1 ## default HAVE_PIRQ_TABLE=1 default IRQ_SLOT_COUNT=5 -#object irq_tables.o ## ## Build code to export a CMOS option table diff --git a/src/mainboard/pcengines/alix1c/cache_as_ram_auto.c b/src/mainboard/pcengines/alix1c/cache_as_ram_auto.c index 7f4898fac9..331e6cd274 100644 --- a/src/mainboard/pcengines/alix1c/cache_as_ram_auto.c +++ b/src/mainboard/pcengines/alix1c/cache_as_ram_auto.c @@ -38,7 +38,7 @@ #define POST_CODE(x) outb(x, 0x80) #define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1) -/* The alix1c has no SMBUS; the setup is hard-wired. */ +/* The ALIX1.C has no SMBus; the setup is hard-wired. */ void cs5536_enable_smbus(void) { } @@ -46,7 +46,8 @@ void cs5536_enable_smbus(void) #include "southbridge/amd/cs5536/cs5536_early_setup.c" #include "superio/winbond/w83627hf/w83627hf_early_serial.c" -/* the part is a hynix hy5du121622ctp-d43 +/* The part is a Hynix hy5du121622ctp-d43. + * * HY 5D U 12 16 2 2 C T P D43 * Hynix * DDR SDRAM (5D) @@ -62,18 +63,17 @@ void cs5536_enable_smbus(void) * Lead Free (P) * DDR400 3-3-3 (D43) */ -/* spd array */ -static u8 spdbytes[] = { +/* SPD array */ +static const u8 spdbytes[] = { [SPD_ACCEPTABLE_CAS_LATENCIES] = 0x10, [SPD_BANK_DENSITY] = 0x40, [SPD_DEVICE_ATTRIBUTES_GENERAL] = 0xff, [SPD_MEMORY_TYPE] = 7, - [SPD_MIN_CYCLE_TIME_AT_CAS_MAX] = 10, /* This is a guess for tRAC value */ - [SPD_MODULE_ATTRIBUTES] = 0xff, /* fix me later when we figure out */ + [SPD_MIN_CYCLE_TIME_AT_CAS_MAX] = 10, /* A guess for the tRAC value */ + [SPD_MODULE_ATTRIBUTES] = 0xff, /* FIXME later when we figure out. */ [SPD_NUM_BANKS_PER_SDRAM] = 4, [SPD_PRIMARY_SDRAM_WIDTH] = 8, - /* alix1c is 1 bank. */ - [SPD_NUM_DIMM_BANKS] = 1, + [SPD_NUM_DIMM_BANKS] = 1, /* ALIX1.C is 1 bank. */ [SPD_NUM_COLUMNS] = 0xa, [SPD_NUM_ROWS] = 3, [SPD_REFRESH] = 0x3a, @@ -91,7 +91,7 @@ static u8 spd_read_byte(u8 device, u8 address) print_debug("spd_read_byte dev "); print_debug_hex8(device); - if (device != (0x50<<1)){ + if (device != (0x50 << 1)) { print_debug(" returns 0xff\n"); return 0xff; } @@ -101,14 +101,17 @@ static u8 spd_read_byte(u8 device, u8 address) print_debug(" returns "); print_debug_hex8(spdbytes[address]); print_debug("\r\n"); + return spdbytes[address]; } -#define ManualConf 0 /* Do automatic strapped PLL config */ -#define PLLMSRhi 0x00001490 /* manual settings for the PLL */ -#define PLLMSRlo 0x02000030 -#define DIMM0 0xA0 -#define DIMM1 0xA2 +#define ManualConf 0 /* Do automatic strapped PLL config */ +#define PLLMSRhi 0x00001490 /* Manual settings for the PLL */ +#define PLLMSRlo 0x02000030 + +#define DIMM0 0xa0 +#define DIMM1 0xa2 + #include "northbridge/amd/lx/raminit.h" #include "northbridge/amd/lx/pll_reset.c" #include "northbridge/amd/lx/raminit.c" @@ -119,9 +122,10 @@ static u8 spd_read_byte(u8 device, u8 address) static void msr_init(void) { msr_t msr; + /* Setup access to the MC for under 1MB. Note MC not setup yet. */ msr.hi = 0x24fffc02; - msr.lo = 0x10010000; + msr.lo = 0x10010000; wrmsr(CPU_RCONF_DEFAULT, msr); msr.hi = 0x20000000; @@ -129,14 +133,13 @@ static void msr_init(void) wrmsr(MSR_GLIU0 + 0x20, msr); msr.hi = 0x20000000; - msr.lo = 0xfff00; + msr.lo = 0xfff00; wrmsr(MSR_GLIU1 + 0x20, msr); - } +/** Early mainboard specific GPIO setup. */ static void mb_gpio_init(void) { - /* Early mainboard specific GPIO setup */ } void cache_as_ram_main(void) @@ -144,7 +147,9 @@ void cache_as_ram_main(void) static const struct mem_controller memctrl[] = { {.channel0 = {0x50}}, }; + extern void RestartCAR(); + POST_CODE(0x01); SystemPreInit(); @@ -152,9 +157,8 @@ void cache_as_ram_main(void) cs5536_early_setup(); - /* NOTE: must do this AFTER the early_setup! - * it is counting on some early MSR setup - * for cs5536 + /* NOTE: Must do this AFTER cs5536_early_setup()! + * It is counting on some early MSR setup for the CS5536. */ cs5536_disable_internal_uart(); w83627hf_enable_serial(SERIAL_DEV, TTYS0_BASE); @@ -169,17 +173,18 @@ void cache_as_ram_main(void) sdram_initialize(1, memctrl); /* Check memory */ - /* enable this only if you are having questions */ - /* ram_check(0x00000000, 640 * 1024);*/ + /* Enable this only if you are having questions. */ + /* ram_check(0, 640 * 1024); */ - /* Switch from Cache as RAM to real RAM - * There are two ways we could think about this. + /* Switch from Cache as RAM to real RAM. + * + * There are two ways we could think about this. * * 1. If we are using the auto.inc ROMCC way, the stack is * going to be re-setup in the code following this code. Just * wbinvd the stack to clear the cache tags. We don't care * where the stack used to be. - * + * * 2. This file is built as a normal .c -> .o and linked in * etc. The stack might be used to return etc. That means we * care about what is in the stack. If we are smart we set @@ -190,16 +195,16 @@ void cache_as_ram_main(void) * located somewhere other than where LB would like it, you * need to write some code to do a copy from cache to RAM * - * We use method 1 on Norwich and on this board too. + * We use method 1 on Norwich and on this board too. */ POST_CODE(0x02); print_err("POST 02\n"); __asm__("wbinvd\n"); print_err("Past wbinvd\n"); - /* we are finding the return does not work on this - * board. Explicitly call the label that is after the call to - * us. This is gross, but sometimes at this level it is the - * only way out + + /* We are finding the return does not work on this board. Explicitly + * call the label that is after the call to us. This is gross, but + * sometimes at this level it is the only way out. */ done_cache_as_ram_main(); } diff --git a/src/mainboard/pcengines/alix1c/irq_tables.c b/src/mainboard/pcengines/alix1c/irq_tables.c index 55d2823085..1db35cba45 100644 --- a/src/mainboard/pcengines/alix1c/irq_tables.c +++ b/src/mainboard/pcengines/alix1c/irq_tables.c @@ -20,7 +20,6 @@ #include #include #include -#include #include "../../../southbridge/amd/cs5536/cs5536.h" /* Platform IRQs */ @@ -41,7 +40,11 @@ #define L_PIRQC 3 /* Means Slot INTx# Connects To Chipset INTC# */ #define L_PIRQD 4 /* Means Slot INTx# Connects To Chipset INTD# */ -/* ALIX 1c interrupt wiring. Devices are: +/* + * ALIX1.C interrupt wiring. + * + * Devices are: + * * 00:01.0 Host bridge: Advanced Micro Devices [AMD] CS5536 [Geode companion] Host Bridge (rev 31) * 00:01.2 Entertainment encryption device: Advanced Micro Devices [AMD] Geode LX AES Security Block * 00:0d.0 Ethernet controller: VIA Technologies, Inc. VT6105M [Rhine-III] (rev 96) @@ -51,8 +54,11 @@ * 00:0f.3 Multimedia audio controller: Advanced Micro Devices [AMD] CS5536 [Geode companion] Audio (rev 01) * 00:0f.4 USB Controller: Advanced Micro Devices [AMD] CS5536 [Geode companion] OHC (rev 02) * 00:0f.5 USB Controller: Advanced Micro Devices [AMD] CS5536 [Geode companion] EHC (rev 02) + * * The only devices that interrupt are: - * What device IRQ PIN PIN WIRED TO + * + * What Device IRQ PIN PIN WIRED TO + * ------------------------------------------------- * AES 00:01.2 0a 01 A A * 3VPCI 00:0c.0 0a 01 A A * eth0 00:0d.0 0b 01 A B @@ -61,39 +67,40 @@ * usb 00:0f.4 0b 04 D D * usb 00:0f.5 0b 04 D D * - * The only swizzled interrupt is eth0, where INTA is wired to interrupt controller line B -*/ + * The only swizzled interrupt is eth0, where INTA is wired to interrupt controller line B. + */ + const struct irq_routing_table intel_irq_routing_table = { -PIRQ_SIGNATURE, /* u32 signature */ -PIRQ_VERSION, /* u16 version */ -32 + 16 * IRQ_SLOT_COUNT, -0x00, /* Where the interrupt router lies (bus) */ -(0x0F << 3) | 0x0, /* Where the interrupt router lies (dev) */ -0x00, /* IRQs devoted exclusively to PCI usage */ -0x100B, /* Vendor */ -0x002B, /* Device */ -0, /* Crap (miniport) */ -{0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, /* u8 rfu[11] */ -0x00, /* u8 checksum , this has to set to - * some value that would give 0 - * after the sum of all bytes - * for this structure - * (including checksum) - */ -{ - /* If you change the number of entries, - * change the IRQ_SLOT_COUNT above! - */ - /* bus, dev|fn, {link, bitmap}, {link, bitmap}, {link, bitmap}, {link, bitmap}, slot, rfu */ - {0x00, (0x01 << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {0x00, 0x00}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0}, /* cpu */ - /* PCI SLOT */ - {0x00, (0x0C << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x4, 0x0}, /* slot1 */ - /* ONBOARD ETHER */ - {0x00, (0x0D << 3) | 0x0, {{L_PIRQB, M_PIRQB}, {0x00, 0x00}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0}, /* ethernet */ - /* MINI PCI */ - {0x00, (0x0E << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x1, 0x0}, /* mini slot2 */ - /* Chipset slots -- f.3 wires to B, and f.4 and f.5 wires to D */ - {0x00, (0x0F << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x0, 0x0}, /* chipset */ + PIRQ_SIGNATURE, + PIRQ_VERSION, + 32 + 16 * IRQ_SLOT_COUNT,/* Max. number of devices on the bus */ + 0x00, /* Where the interrupt router lies (bus) */ + (0x0F << 3) | 0x0, /* Where the interrupt router lies (dev) */ + 0x00, /* IRQs devoted exclusively to PCI usage */ + 0x100B, /* Vendor */ + 0x002B, /* Device */ + 0, /* Crap (miniport) */ + {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, /* u8 rfu[11] */ + 0x00, /* Checksum */ + { + /* If you change the number of entries, change IRQ_SLOT_COUNT above! */ + + /* bus, dev|fn, {link, bitmap}, {link, bitmap}, {link, bitmap}, {link, bitmap}, slot, rfu */ + + /* CPU */ + {0x00, (0x01 << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {0x00, 0x00}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0}, + + /* PCI (slot 1) */ + {0x00, (0x0C << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x4, 0x0}, + + /* On-board ethernet */ + {0x00, (0x0D << 3) | 0x0, {{L_PIRQB, M_PIRQB}, {0x00, 0x00}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0}, + + /* Mini PCI (slot 2) */ + {0x00, (0x0E << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x1, 0x0}, + + /* Chipset slots -- f.3 wires to B, and f.4 and f.5 wires to D. */ + {0x00, (0x0F << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x0, 0x0}, } }; diff --git a/src/mainboard/pcengines/alix1c/mainboard.c b/src/mainboard/pcengines/alix1c/mainboard.c index 1343725ab7..743f2800fc 100644 --- a/src/mainboard/pcengines/alix1c/mainboard.c +++ b/src/mainboard/pcengines/alix1c/mainboard.c @@ -21,14 +21,13 @@ #include #include #include -#include #include #include #include #include "../../../southbridge/amd/cs5536/cs5536.h" #include "chip.h" -/* Print the platform configuration */ +/* Print the platform configuration. */ void print_conf(void) { #if DEFAULT_CONSOLE_LOGLEVEL >= BIOS_ERR int i; @@ -156,8 +155,8 @@ void print_conf(void) { static void init(struct device *dev) { - printk_debug("ALIX1C ENTER %s\n", __FUNCTION__); - printk_debug("ALIX1C EXIT %s\n", __FUNCTION__); + printk_debug("ALIX1.C ENTER %s\n", __FUNCTION__); + printk_debug("ALIX1.C EXIT %s\n", __FUNCTION__); } static void enable_dev(struct device *dev) -- cgit v1.2.3