[coreboot] r1023 - in coreboot-v3: mainboard/kontron/986lcd-m northbridge/intel/i945

svn at coreboot.org svn at coreboot.org
Fri Nov 14 16:59:00 CET 2008


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 at gmail.com>
Acked-by: Ronald G. Minnich <rminnich at 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;





More information about the coreboot mailing list