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 <[EMAIL PROTECTED]>
Acked-by: Ronald G. Minnich <[EMAIL PROTECTED]>


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;


--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to