Author: rminnich Date: 2008-11-14 16:58:59 +0100 (Fri, 14 Nov 2008) New Revision: 1023
Modified: coreboot-v3/mainboard/kontron/986lcd-m/Makefile coreboot-v3/mainboard/kontron/986lcd-m/initram.c coreboot-v3/northbridge/intel/i945/Makefile coreboot-v3/northbridge/intel/i945/raminit.c coreboot-v3/northbridge/intel/i945/rcven.c coreboot-v3/northbridge/intel/i945/stage1.c Log: These are all cleanups to get it closer to building.
Lots more to do.
Signed-off-by: Ronald G. Minnich rminnich@gmail.com Acked-by: Ronald G. Minnich rminnich@gmail.com
Modified: coreboot-v3/mainboard/kontron/986lcd-m/Makefile =================================================================== --- coreboot-v3/mainboard/kontron/986lcd-m/Makefile 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/mainboard/kontron/986lcd-m/Makefile 2008-11-14 15:58:59 UTC (rev 1023) @@ -26,7 +26,6 @@
INITRAM_SRC= $(src)/mainboard/$(MAINBOARDDIR)/initram.c \ - $(src)/northbridge/intel/i945/raminit.c \
STAGE2_CHIPSET_SRC=
Modified: coreboot-v3/mainboard/kontron/986lcd-m/initram.c =================================================================== --- coreboot-v3/mainboard/kontron/986lcd-m/initram.c 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/mainboard/kontron/986lcd-m/initram.c 2008-11-14 15:58:59 UTC (rev 1023) @@ -35,6 +35,8 @@ #include <io.h> #include <mc146818rtc.h> #include <spd.h> +#include "../../../northbridge/intel/i945/ich7.h" +#include "../../../northbridge/intel/i945/i945.h"
#define RC0 ((1<<0)<<8)
@@ -174,10 +176,12 @@ */ int main(void) { + int boot_mode; void i945_early_initialization(void); void enable_smbus(void); int fixup_i945_errata(void); void i945_late_initialization(void); + void sdram_initialize(int boot_mode);
if (MCHBAR16(SSKPD) == 0xCAFE) { printk(BIOS_DEBUG, "soft reset detected.\n"); @@ -212,7 +216,7 @@ /* Initialize the internal PCIe links before we go into stage2 */ i945_late_initialization();
-#if DEFAULT_CONSOLE_LOGLEVEL > 8 +#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8 #if defined(DEBUG_RAM_SETUP) sdram_dump_mchbar_registers(); #endif
Modified: coreboot-v3/northbridge/intel/i945/Makefile =================================================================== --- coreboot-v3/northbridge/intel/i945/Makefile 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/northbridge/intel/i945/Makefile 2008-11-14 15:58:59 UTC (rev 1023) @@ -21,6 +21,12 @@
ifeq ($(CONFIG_NORTHBRIDGE_INTEL_I945),y)
+INITRAM_SRC += \ + $(src)/northbridge/intel/i945/raminit.c \ + $(src)/northbridge/intel/i945/rcven.c \ + $(src)/lib/clog2.c + + STAGE2_CHIPSET_SRC += $(src)/northbridge/intel/i945/northbridge.c \
endif
Modified: coreboot-v3/northbridge/intel/i945/raminit.c =================================================================== --- coreboot-v3/northbridge/intel/i945/raminit.c 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/northbridge/intel/i945/raminit.c 2008-11-14 15:58:59 UTC (rev 1023) @@ -31,6 +31,7 @@ #include <mc146818rtc.h> #include <lib.h> #include <spd.h> +#include <io.h> #include "raminit.h" #include "i945.h"
@@ -72,11 +73,11 @@ }
-static void ram_read32(u32 offset) +static void ram_readl(u32 offset) { printk(BIOS_DEBUG," ram read: %08x\n", offset);
- read32(offset); + readl((void *)offset); }
#ifdef DEBUG_RAM_SETUP @@ -97,7 +98,7 @@ { u32 reg32;
- reg32 = pci_conf1_read_config32(PCI_DEV(0, 0x00, 0), 0xe4); + reg32 = pci_conf1_read_config32(PCI_BDF(0, 0x00, 0), 0xe4); reg32 &= (7 << 0);
switch (reg32) { @@ -121,7 +122,7 @@ { u32 reg32;
- reg32 = pci_conf1_read_config8(PCI_DEV(0, 0x00,0), 0xe4); + reg32 = pci_conf1_read_config8(PCI_BDF(0, 0x00,0), 0xe4); reg32 >>= 25; reg32 &= 1;
@@ -137,7 +138,7 @@ { u32 reg32;
- reg32 = pci_conf1_read_config8(PCI_DEV(0, 0x00,0), 0xe4); + reg32 = pci_conf1_read_config8(PCI_BDF(0, 0x00,0), 0xe4); reg32 >>= 24; reg32 &= 1;
@@ -148,7 +149,7 @@ { u8 reg8;
- reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x00, 0), 0xe5); /* CAPID0 + 5 */ + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x00, 0), 0xe5); /* CAPID0 + 5 */ reg8 &= (1 << 7); return (!reg8); @@ -158,7 +159,7 @@ { u8 reg8;
- reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x00, 0), 0xe8); /* CAPID0 + 8 */ + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x00, 0), 0xe8); /* CAPID0 + 8 */ reg8 &= (1 << 0); return (reg8 != 0); @@ -168,7 +169,7 @@ { u8 reg8;
- reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x00, 0), 0xe5); + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x00, 0), 0xe5); reg8 &= (1 << 0); return (reg8 != 0); @@ -178,25 +179,25 @@ { u8 reg8;
- reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x1f, 0), 0xa2); + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x1f, 0), 0xa2); if (reg8 & ((1<<7)|(1<<2))) { if (reg8 & (1<<2)) { printk(BIOS_DEBUG, "SLP S4# Assertion Width Violation.\n"); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa2, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa2, reg8); }
if (reg8 & (1<<7)) { printk(BIOS_DEBUG, "DRAM initialization was interrupted.\n"); reg8 &= ~(1<<7); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa2, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa2, reg8); }
/* Set SLP_S3# Assertion Stretch Enable */ - reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x1f, 0), 0xa4); /* GEN_PMCON_3 */ + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x1f, 0), 0xa4); /* GEN_PMCON_3 */ reg8 |= (1 << 3); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa4, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa4, reg8);
printk(BIOS_DEBUG, "Reset required.\n"); outb(0x00, 0xcf9); @@ -205,9 +206,9 @@ }
/* Set DRAM initialization bit in ICH7 */ - reg8 = pci_conf1_read_config8(PCI_DEV(0, 0x1f, 0), 0xa2); + reg8 = pci_conf1_read_config8(PCI_BDF(0, 0x1f, 0), 0xa2); reg8 |= (1<<7); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa2, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa2, reg8); }
@@ -309,7 +310,7 @@ sysinfo->dimm[i] = SYSINFO_DIMM_X8DS; break; default: - printk_debug ("Unsupported.\n"); + printk(BIOS_DEBUG, "Unsupported.\n"); } break; case 0x10: @@ -323,7 +324,7 @@ sysinfo->dimm[i] = SYSINFO_DIMM_X16SS; break; default: - printk_debug ("Unsupported.\n"); + printk(BIOS_DEBUG, "Unsupported.\n"); } break; default: @@ -342,7 +343,7 @@ * not bother until someone needs this enough to cope with it. */ if (!(dimm_mask & ((1 << DIMM_SOCKETS) - 1))) { - printk_err("Channel 0 has no memory populated. This setup is not usable. Please move the DIMM.\n"); + printk(BIOS_ERR, "Channel 0 has no memory populated. This setup is not usable. Please move the DIMM.\n"); } }
@@ -1053,7 +1054,7 @@ u32 chan0dll = 0, chan1dll = 0; int i;
- printk_debug ("Programming DLL Timings... \n"); + printk(BIOS_DEBUG, "Programming DLL Timings... \n"); MCHBAR16(DQSMT) &= ~( (3 << 12) | (1 << 10) | ( 0xf << 0) ); MCHBAR16(DQSMT) |= (1 << 13) | (0xc << 0); @@ -1073,6 +1074,11 @@ } }
+static int i945_silicon_revision(void) +{ + return pci_conf1_read_config8(PCI_BDF(0, 0x00, 0), 8); +} + static void sdram_force_rcomp(void) { u32 reg32; @@ -1103,8 +1109,8 @@ u8 reg8; u32 reg32;
- printk_debug ("Initializing System Memory IO... \n"); - + printk(BIOS_DEBUG, "Initializing System Memory IO... \n"); + reg8 = MCHBAR8(C0HCTC); reg8 &= ~0x1f; reg8 |= ( 1 << 0); @@ -1142,7 +1148,7 @@ { u32 reg32;
- printk_debug ("Enabling System Memory IO... \n"); + printk(BIOS_DEBUG, "Enabling System Memory IO... \n"); reg32 = MCHBAR32(RCVENMT); reg32 &= ~(0x3f << 6); @@ -1220,7 +1226,8 @@ value = spd_read_byte(device, SPD_NUM_BANKS_PER_SDRAM); /* banks */ if (value < 0) goto hw_err; if ((value & 0xff) == 0) goto val_err; - sz.side1 += log2(value & 0xff); +#warning log2 floor or ceiling -- which one? + sz.side1 += log2f(value & 0xff);
/* Get the module data width and convert it to a power of two */ value = spd_read_byte(device, SPD_MODULE_DATA_WIDTH_MSB); /* (high byte) */ @@ -1232,7 +1239,8 @@ if (low < 0) goto hw_err; value = value | (low & 0xff); if ((value != 72) && (value != 64)) goto val_err; - sz.side1 += log2(value); +#warning log2 floor or ceiling -- which one? + sz.side1 += log2f(value);
/* side 2 */ value = spd_read_byte(device, SPD_NUM_DIMM_BANKS); /* number of physical banks */ @@ -1251,7 +1259,7 @@ /* Don't die here, I have not come across any of these to test what * actually happens. */ - printk_err("Assymetric DIMMs are not supported by this chipset\n"); + printk(BIOS_ERR, "Assymetric DIMMs are not supported by this chipset\n");
sz.side2 -= (rows & 0x0f); /* Subtract out rows on side 1 */ sz.side2 += ((rows >> 4) & 0x0f); /* Add in rows on side 2 */ @@ -1315,7 +1323,7 @@ int i; int cum0, cum1, tolud;
- printk_debug ("Setting RAM size... \n"); + printk(BIOS_DEBUG, "Setting RAM size... \n");
cum0 = 0; for(i = 0; i < 2 * DIMM_SOCKETS; i++) { @@ -1345,13 +1353,13 @@ tolud = (cum0 + cum1) << 1; else tolud = (cum1 ? cum1 : cum0) << 1; - pci_conf1_write_config16(PCI_DEV(0,0,0), TOLUD, tolud); + pci_conf1_write_config16(PCI_BDF(0,0,0), TOLUD, tolud); printk(BIOS_DEBUG, "C0DRB = 0x%08x\n", MCHBAR32(C0DRB0)); printk(BIOS_DEBUG, "C1DRB = 0x%08x\n", MCHBAR32(C1DRB0)); printk(BIOS_DEBUG, "TOLUD = 0x%04x\n", tolud);
- pci_conf1_write_config16(PCI_DEV(0,0,0), TOM, tolud>>3); + pci_conf1_write_config16(PCI_BDF(0,0,0), TOM, tolud>>3);
return 0; } @@ -1362,7 +1370,7 @@ int i, value; u16 dra0=0, dra1=0, dra = 0;
- printk_debug ("Setting row attributes... \n"); + printk(BIOS_DEBUG, "Setting row attributes... \n"); for(i=0; i < 2 * DIMM_SOCKETS; i++) { u16 device; u8 columnsrows; @@ -1429,7 +1437,7 @@ if (sysinfo->banks[i] != 8) continue;
- printk_spew("DIMM%d has 8 banks.\n"); + printk(BIOS_SPEW, "DIMM%d has 8 banks.\n", i);
if (i & 1) MCHBAR16(off32) |= 0x50; @@ -1779,11 +1787,11 @@ #define CDCLK_200MHz 0x00 #define CDCLK_320MHz 0x40
- printk_debug ("Setting Graphics Frequency... \n"); + printk(BIOS_DEBUG, "Setting Graphics Frequency... \n");
- reg16 = pci_conf1_read_config16(PCI_DEV(0,2,0), GCFC); + reg16 = pci_conf1_read_config16(PCI_BDF(0,2,0), GCFC); reg16 |= (1<<11) | (1<<9); - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16);
/* Program CPCTL according to FSB speed */ reg16 = MCHBAR16(CPCTL); @@ -1803,7 +1811,7 @@ MCHBAR16(CPCTL) = reg16;
/* Get graphics frequency capabilities */ - reg8 = (pci_conf1_read_config8(PCI_DEV(0, 0x00,0), 0xe5) & 0x0e) >> 1; + reg8 = (pci_conf1_read_config8(PCI_BDF(0, 0x00,0), 0xe5) & 0x0e) >> 1; freq = CRCLK_250MHz; switch (reg8) { @@ -1820,7 +1828,7 @@
if (freq != CRCLK_400MHz) { - reg8 = (pci_conf1_read_config8(PCI_DEV(0, 0x00,0), 0xe7) & 0x70) >> 4; + reg8 = (pci_conf1_read_config8(PCI_BDF(0, 0x00,0), 0xe7) & 0x70) >> 4; if (reg8==2) freq = CRCLK_166MHz; } @@ -1857,28 +1865,28 @@ sysinfo->clkcfg_bit7=0; }
- reg16 = pci_conf1_read_config16(PCI_DEV(0,2,0), GCFC); + reg16 = pci_conf1_read_config16(PCI_BDF(0,2,0), GCFC); reg16 &= ~( (7 << 0) | (1 << 13) ); reg16 |= freq; - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16);
- reg16 = pci_conf1_read_config16(PCI_DEV(0,2,0), GCFC); + reg16 = pci_conf1_read_config16(PCI_BDF(0,2,0), GCFC); reg16 &= ~( (1<<7) | (7<<4) ); if (MCHBAR32(DFT_STRAP1) & (1 << 20)) { reg16 |= CDCLK_200MHz; } else { reg16 |= CDCLK_320MHz; } - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16);
- reg16 = pci_conf1_read_config16(PCI_DEV(0,2,0), GCFC); + reg16 = pci_conf1_read_config16(PCI_BDF(0,2,0), GCFC); reg16 &= ~( (1<<10) | (1<<8) ); - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16); reg16 |= (1<<10) | (1<<8); - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16);
reg16 &= 0xf0ff; - pci_conf1_write_config16(PCI_DEV(0,2,0), GCFC, reg16); + pci_conf1_write_config16(PCI_BDF(0,2,0), GCFC, reg16); }
static void sdram_program_memory_frequency(struct sys_info *sysinfo) @@ -1886,7 +1894,7 @@ u32 clkcfg; u8 reg8;
- printk_debug ("Setting Memory Frequency... "); + printk(BIOS_DEBUG, "Setting Memory Frequency... "); clkcfg = MCHBAR32(CLKCFG);
@@ -1913,7 +1921,7 @@ }
if (MCHBAR32(CLKCFG) == clkcfg) { - printk_debug ("ok (unchanged)\n"); + printk(BIOS_DEBUG, "ok (unchanged)\n"); return; }
@@ -1924,9 +1932,9 @@ */ goto cache_code; vco_update: - reg8 = pci_conf1_read_config8(PCI_DEV(0,0x1f,0), 0xa2); + reg8 = pci_conf1_read_config8(PCI_BDF(0,0x1f,0), 0xa2); reg8 &= ~(1 << 7); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa2, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa2, reg8);
clkcfg &= ~(1 << 10); @@ -1958,7 +1966,7 @@ out:
printk(BIOS_DEBUG, "CLKCFG=0x%08x, ", MCHBAR32(CLKCFG)); - printk_debug ("ok\n"); + printk(BIOS_DEBUG, "ok\n"); }
static void sdram_program_clock_crossing(void) @@ -2306,13 +2314,13 @@ MCHBAR32(FSBPMC4) &= ~(1 << 4); }
- reg8 = pci_conf1_read_config8(PCI_DEV(0,0x0,0), 0xfc); + reg8 = pci_conf1_read_config8(PCI_BDF(0,0x0,0), 0xfc); reg8 |= (1 << 4); - pci_conf1_write_config8(PCI_DEV(0, 0x0, 0), 0xfc, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x0, 0), 0xfc, reg8);
- reg8 = pci_conf1_read_config8(PCI_DEV(0,0x2,0), 0xc1); + reg8 = pci_conf1_read_config8(PCI_BDF(0,0x2,0), 0xc1); reg8 |= (1 << 2); - pci_conf1_write_config8(PCI_DEV(0, 0x2, 0), 0xc1, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x2, 0), 0xc1, reg8);
if (integrated_graphics) { MCHBAR16(MIPMC4) = 0x0468; @@ -2349,10 +2357,9 @@ MCHBAR8(TCO0) = 0x00; }
-#include "rcven.c" - static void sdram_program_receive_enable(struct sys_info *sysinfo) { + void receive_enable_adjust(struct sys_info *sysinfo); MCHBAR32(REPC) |= (1 << 0);
receive_enable_adjust(sysinfo); @@ -2528,22 +2535,22 @@ /* Apply NOP */ printk(BIOS_DEBUG,"Apply NOP\n"); do_ram_command(RAM_COMMAND_NOP); - ram_read32(bankaddr); + ram_readl(bankaddr);
/* Precharge all banks */ printk(BIOS_DEBUG,"All Banks Precharge\n"); do_ram_command(RAM_COMMAND_PRECHARGE); - ram_read32(bankaddr); + ram_readl(bankaddr);
/* Extended Mode Register Set (2) */ printk(BIOS_DEBUG,"Extended Mode Register Set(2)\n"); do_ram_command(RAM_COMMAND_EMRS | RAM_EMRS_2); - ram_read32(bankaddr); + ram_readl(bankaddr);
/* Extended Mode Register Set (3) */ printk(BIOS_DEBUG,"Extended Mode Register Set(3)\n"); do_ram_command(RAM_COMMAND_EMRS | RAM_EMRS_3); - ram_read32(bankaddr); + ram_readl(bankaddr);
/* Extended Mode Register Set */ printk(BIOS_DEBUG,"Extended Mode Register Set\n"); @@ -2556,7 +2563,7 @@ } else { tmpaddr |= RTT_ODT_150_OHM; } - ram_read32(tmpaddr); + ram_readl(tmpaddr);
/* Mode Register Set: Reset DLLs */ printk(BIOS_DEBUG,"MRS: Reset DLLs\n"); @@ -2568,20 +2575,20 @@ tmpaddr |= (1 << 12); else tmpaddr |= (1 << 11); - ram_read32(tmpaddr); + ram_readl(tmpaddr);
/* Precharge all banks */ printk(BIOS_DEBUG,"All Banks Precharge\n"); do_ram_command(RAM_COMMAND_PRECHARGE); - ram_read32(bankaddr); + ram_readl(bankaddr);
/* CAS before RAS Refresh */ printk(BIOS_DEBUG,"CAS before RAS\n"); do_ram_command(RAM_COMMAND_CBR);
/* CBR wants two READs */ - ram_read32(bankaddr); - ram_read32(bankaddr); + ram_readl(bankaddr); + ram_readl(bankaddr);
/* Mode Register Set: Enable DLLs */ printk(BIOS_DEBUG,"MRS: Enable DLLs\n"); @@ -2589,7 +2596,7 @@
tmpaddr = bankaddr; tmpaddr |= mrsaddr; - ram_read32(tmpaddr); + ram_readl(tmpaddr); /* Extended Mode Register Set */ printk(BIOS_DEBUG,"Extended Mode Register Set: ODT/OCD\n"); @@ -2604,7 +2611,7 @@ } else { tmpaddr |= RTT_ODT_150_OHM | EMRS_OCD_DEFAULT; } - ram_read32(tmpaddr); + ram_readl(tmpaddr);
/* Extended Mode Register Set */ printk(BIOS_DEBUG,"Extended Mode Register Set: OCD Exit\n"); @@ -2618,7 +2625,7 @@ } else { tmpaddr |= RTT_ODT_150_OHM; } - ram_read32(tmpaddr); + ram_readl(tmpaddr); } }
@@ -2652,7 +2659,7 @@
sdram_detect_errors();
- printk_debug ("Setting up RAM controller.\n"); + printk(BIOS_DEBUG, "Setting up RAM controller.\n");
memset(&sysinfo, 0, sizeof(sysinfo));
@@ -2769,9 +2776,9 @@ sdram_enable_rcomp();
/* Tell ICH7 that we're done */ - reg8 = pci_conf1_read_config8(PCI_DEV(0,0x1f,0), 0xa2); + reg8 = pci_conf1_read_config8(PCI_BDF(0,0x1f,0), 0xa2); reg8 &= ~(1 << 7); - pci_conf1_write_config8(PCI_DEV(0, 0x1f, 0), 0xa2, reg8); + pci_conf1_write_config8(PCI_BDF(0, 0x1f, 0), 0xa2, reg8);
printk(BIOS_DEBUG, "RAM initialization finished.\n");
Modified: coreboot-v3/northbridge/intel/i945/rcven.c =================================================================== --- coreboot-v3/northbridge/intel/i945/rcven.c 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/northbridge/intel/i945/rcven.c 2008-11-14 15:58:59 UTC (rev 1023) @@ -25,14 +25,14 @@ #include <spd_ddr2.h> #include <cpu.h> #include <msr.h> -#include <amd/k8/k8.h> -#include <amd/k8/sysconf.h> #include <device/pci.h> #include <pci_ops.h> #include <mc146818rtc.h> #include <lib.h> #include <spd.h> +#include <io.h> #include "raminit.h" +#include "../../../northbridge/intel/i945/i945.h"
/** * sample the strobes signal @@ -57,8 +57,8 @@ }
for (i = 0; i < 28; i++) { - read32(addr); - read32(addr + 0x80); + readl((void *)addr); + readl((void *)(addr + 0x80)); }
reg32 = MCHBAR32(RCVENMT);
Modified: coreboot-v3/northbridge/intel/i945/stage1.c =================================================================== --- coreboot-v3/northbridge/intel/i945/stage1.c 2008-11-14 15:25:22 UTC (rev 1022) +++ coreboot-v3/northbridge/intel/i945/stage1.c 2008-11-14 15:58:59 UTC (rev 1023) @@ -33,11 +33,6 @@ #include <lib.h> #include "i945.h"
-static int i945_silicon_revision(void) -{ - return pci_conf1_read_config8(PCI_BDF(0, 0x00, 0), 8); -} - static void i945_detect_chipset(void) { u8 reg8;