From 30140a59f7c34b583b670401a205338e0c8e3311 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Wed, 11 Mar 2009 16:20:39 +0000 Subject: i945 northbridge update - lots of PCIe updates - various bug fixes to early init - some fixes for typos and warnings - initial support for PCIe x16 - some minor fixes to memory init code - some subsystem vendor id patches, to be consistent with ICH7 Signed-off-by: Stefan Reinauer Acked-by: Peter Stuge git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3997 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/northbridge/intel/i945/Config.lb | 3 +- src/northbridge/intel/i945/early_init.c | 300 ++++++++++++++++++++++++++----- src/northbridge/intel/i945/gma.c | 93 ++++++++++ src/northbridge/intel/i945/northbridge.c | 13 +- src/northbridge/intel/i945/pcie_config.c | 2 +- src/northbridge/intel/i945/raminit.c | 11 +- src/northbridge/intel/i945/raminit.h | 2 +- 7 files changed, 373 insertions(+), 51 deletions(-) create mode 100644 src/northbridge/intel/i945/gma.c (limited to 'src/northbridge/intel/i945') diff --git a/src/northbridge/intel/i945/Config.lb b/src/northbridge/intel/i945/Config.lb index 744437988b..15a9e8f572 100644 --- a/src/northbridge/intel/i945/Config.lb +++ b/src/northbridge/intel/i945/Config.lb @@ -1,7 +1,7 @@ # # This file is part of the coreboot project. # -# Copyright (C) 2007-2008 coresystems GmbH +# Copyright (C) 2007-2009 coresystems GmbH # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -19,3 +19,4 @@ config chip.h driver northbridge.o +driver gma.o diff --git a/src/northbridge/intel/i945/early_init.c b/src/northbridge/intel/i945/early_init.c index a829644ef8..a7b06b079d 100644 --- a/src/northbridge/intel/i945/early_init.c +++ b/src/northbridge/intel/i945/early_init.c @@ -1,7 +1,7 @@ /* * This file is part of the coreboot project. * - * Copyright (C) 2007-2008 coresystems GmbH + * Copyright (C) 2007-2009 coresystems GmbH * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -162,7 +162,6 @@ static void i945_setup_egress_port(void) reg32 &= 0xffffff01; EPBAR32(EPVC0RCTL) = reg32; - reg32 = EPBAR32(EPPVCCAP1); reg32 &= ~(7 << 0); reg32 |= 1; @@ -195,7 +194,7 @@ static void i945_setup_egress_port(void) } /* Is internal graphics enabled? */ - if (pci_read_config8(PCI_DEV(0, 0x0, 0), 54) & ((1 << 4) | (1 << 3))) { /* DEVEN */ + if (pci_read_config8(PCI_DEV(0, 0x0, 0), 0x54) & ((1 << 4) | (1 << 3))) { /* DEVEN */ MCHBAR32(MMARB1) |= (1 << 17); } @@ -204,12 +203,12 @@ static void i945_setup_egress_port(void) reg32 &= ~(7 << 24); reg32 |= (1 << 24); EPBAR32(EPVC1RCTL) = reg32; - + reg32 = EPBAR32(EPVC1RCTL); reg32 &= 0xffffff01; reg32 |= (1 << 7); EPBAR32(EPVC1RCTL) = reg32; - + EPBAR32(PORTARB + 0x00) = 0x01000001; EPBAR32(PORTARB + 0x04) = 0x00040000; EPBAR32(PORTARB + 0x08) = 0x00001000; @@ -218,7 +217,7 @@ static void i945_setup_egress_port(void) EPBAR32(PORTARB + 0x14) = 0x00040000; EPBAR32(PORTARB + 0x18) = 0x00001000; EPBAR32(PORTARB + 0x1c) = 0x00000040; - + EPBAR32(EPVC1RCTL) |= (1 << 16); EPBAR32(EPVC1RCTL) |= (1 << 16); @@ -248,11 +247,11 @@ static void i945_setup_egress_port(void) static void ich7_setup_dmi_rcrb(void) { u16 reg16; + u32 reg32; - reg16 = RCBA16(LCTL); reg16 &= ~(3 << 0); - reg16 |= 1; + reg16 |= 3; RCBA16(LCTL) = reg16; RCBA32(V0CTL) = 0x80000001; @@ -267,12 +266,33 @@ static void ich7_setup_dmi_rcrb(void) RCBA32(RPFN) = 0x00543210; - pci_write_config16(PCI_DEV(0, 0x1c, 0), 0x42, 0x0141); - pci_write_config16(PCI_DEV(0, 0x1c, 4), 0x42, 0x0141); - pci_write_config16(PCI_DEV(0, 0x1c, 5), 0x42, 0x0141); + pci_write_config16(PCI_DEV(0, 0x1c, 0), 0x42, 0x0141); + pci_write_config16(PCI_DEV(0, 0x1c, 4), 0x42, 0x0141); + pci_write_config16(PCI_DEV(0, 0x1c, 5), 0x42, 0x0141); + + pci_write_config32(PCI_DEV(0, 0x1c, 4), 0x54, 0x00480ce0); + pci_write_config32(PCI_DEV(0, 0x1c, 5), 0x54, 0x00500ce0); + + reg32 = RCBA32(V1CTL); + reg32 &= ~( (0x7f << 1) | (7 << 17) | (7 << 24) ); + reg32 |= (0x40 << 1) | (4 << 17) | (1 << 24) | (1 << 31); + RCBA32(V1CTL) = reg32; - pci_write_config32(PCI_DEV(0, 0x1c, 4), 0x54, 0x00480ce0); - pci_write_config32(PCI_DEV(0, 0x1c, 5), 0x54, 0x00500ce0); + RCBA32(ESD) |= (2 << 16); + + RCBA32(ULD) |= (1 << 24) | (1 << 16); + + RCBA32(ULBA) = DEFAULT_DMIBAR; + + RCBA32(RP1D) |= (2 << 16); + RCBA32(RP2D) |= (2 << 16); + RCBA32(RP3D) |= (2 << 16); + RCBA32(RP4D) |= (2 << 16); + RCBA32(HDD) |= (2 << 16); + RCBA32(RP5D) |= (2 << 16); + RCBA32(RP6D) |= (2 << 16); + + RCBA32(LCAP) |= (3 << 10); } static void i945_setup_dmi_rcrb(void) @@ -280,23 +300,25 @@ static void i945_setup_dmi_rcrb(void) u32 reg32; u32 timeout; + int activate_aspm = 1; + printk_debug("Setting up DMI RCRB\n"); /* Virtual Channel 0 Configuration */ reg32 = DMIBAR32(DMIVC0RCTL0); reg32 &= 0xffffff01; DMIBAR32(DMIVC0RCTL0) = reg32; - + reg32 = DMIBAR32(DMIPVCCAP1); reg32 &= ~(7 << 0); reg32 |= 1; DMIBAR32(DMIPVCCAP1) = reg32; - + reg32 = DMIBAR32(DMIVC1RCTL); reg32 &= ~(7 << 24); reg32 |= (1 << 24); /* NOTE: This ID must match ICH7 side */ DMIBAR32(DMIVC1RCTL) = reg32; - + reg32 = DMIBAR32(DMIVC1RCTL); reg32 &= 0xffffff01; reg32 |= (1 << 7); @@ -318,20 +340,23 @@ static void i945_setup_dmi_rcrb(void) reg32 = DMIBAR32(DMILCAP); reg32 &= ~(7 << 12); - reg32 |= (2 << 12); + reg32 |= (2 << 12); reg32 &= ~(7 << 15); - - reg32 |= (2 << 15); + reg32 |= (2 << 15); DMIBAR32(DMILCAP) = reg32; reg32 = DMIBAR32(DMICC); - reg32 &= 0x00ffffff; + reg32 &= 0x00ffffff; reg32 &= ~(3 << 0); - reg32 |= (1 << 0); + reg32 |= (1 << 0); + + reg32 &= ~(3 << 20); + reg32 |= (1 << 20); + DMIBAR32(DMICC) = reg32; - if (0) { + if (activate_aspm) { DMIBAR32(DMILCTL) |= (3 << 0); } #endif @@ -353,7 +378,7 @@ static void i945_setup_dmi_rcrb(void) #endif DMIBAR32(0x204) = reg32; - if (pci_read_config8(PCI_DEV(0, 0x0, 0), 54) & ((1 << 4) | (1 << 3))) { /* DEVEN */ + if (pci_read_config8(PCI_DEV(0, 0x0, 0), 0x54) & ((1 << 4) | (1 << 3))) { /* DEVEN */ DMIBAR32(0x200) |= (1 << 21); } else { DMIBAR32(0x200) &= ~(1 << 21); @@ -408,11 +433,11 @@ static void i945_setup_dmi_rcrb(void) printk_debug("timeout!\n"); else printk_debug("ok\n"); - + DMIBAR32(0x1c4) = 0xffffffff; DMIBAR32(0x1d0) = 0xffffffff; DMIBAR32(0x228) = 0xffffffff; - + DMIBAR32(0x308) = DMIBAR32(0x308); DMIBAR32(0x314) = DMIBAR32(0x314); DMIBAR32(0x324) = DMIBAR32(0x324); @@ -421,7 +446,7 @@ static void i945_setup_dmi_rcrb(void) DMIBAR32(0x338) = DMIBAR32(0x338); if (i945_silicon_revision() == 1 && ((MCHBAR8(0xe08) & (1 << 5)) == 1)) { - if ((MCHBAR32(0x214) & 0xf) != 0x3) { + if ((MCHBAR32(0x214) & 0xf) != 0x3) { printk_info ("DMI link requires A1 stepping workaround. Rebooting.\n"); reg32 = MCHBAR32(MMARB1); @@ -438,7 +463,191 @@ static void i945_setup_pci_express_x16(void) u32 timeout; u32 reg32; u16 reg16; +#if SETUP_PCIE_X16_LINK + u8 reg8; + + printk_debug("Enabling PCI Express x16 Link\n"); + + reg16 = pci_read_config16(PCI_DEV(0, 0x00, 0), DEVEN); + reg16 |= DEVEN_D1F0; + pci_write_config16(PCI_DEV(0, 0x00, 0), DEVEN, reg16); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x208); + reg32 &= ~(1 << 8); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x208, reg32); + + MCHBAR16(UPMC1) &= ~( (1 << 5) | (1 << 0) ); + + /* Initialze PEG_CAP */ + reg16 = pcie_read_config16(PCI_DEV(0, 0x01, 0), 0xa2); + reg16 |= (1 << 8); + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0xa2, reg16); + + /* Setup SLOTCAP */ + /* TODO: These values are mainboard dependent and should + * be set from Config.lb or Options.lb. + */ + /* NOTE: SLOTCAP becomes RO after the first write! */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xb4); + reg32 &= 0x0007ffff; // TODO + reg32 &= 0xfffe007f; // TODO + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xb4, reg32); + + /* Wait for training to succeed */ + printk_debug("Wait for PCIe x16 link training ..."); + timeout = 0x7fffff; + while ((((pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x214) >> 16) & 4) != 3) && --timeout) ; + if (!timeout) { + printk_debug("timeout!\n"); + + printk_debug("Restrain PCIe port to x1\n"); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x214); + reg32 &= ~(0xf << 1); + reg32 |=1; + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x214, reg32); + + reg16 = pcie_read_config16(PCI_DEV(0, 0x01, 0), 0x3e); + + reg16 |= (1 << 6); + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0x3e, reg16); + reg16 &= ~(1 << 6); + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0x3e, reg16); + + printk_debug("Wait for PCIe x1 link training ..."); + timeout = 0x7fffff; + while ((((pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x214) >> 16) & 4) != 3) && --timeout) ; + if (!timeout) { + printk_debug("timeout!\n"); + printk_debug("Disabling PCIe x16 port completely.\n"); + goto disable_pciexpress_x16_link; + } else { + printk_debug("ok\n"); + } + } else { + printk_debug("ok\n"); + } + + reg16 = pcie_read_config16(PCI_DEV(0, 0x01, 0), 0xb2); + reg16 >>= 4; + reg16 &= 0x3f; + + printk_debug("PCIe x%d link training succeeded.\n", reg16); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x204); + reg32 &= 0xfffffc00; + if (reg16 == 1) { + reg32 |= 0x32b; + // TODO + } else if (reg16 == 16) { + reg32 |= 0x0f4; + // TODO + } + + /* Enable GPEs */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xec); + reg32 |= (1 << 2) | (1 << 1) | (1 << 0); /* PMEGPE, HPGPE, GENGPE */ + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x114, reg32); + + /* Virtual Channel Configuration: Only VC0 on PCIe x16 */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x114); + reg32 &= 0xffffff01; + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x114, reg32); + + /* Extended VC count */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x104); + reg32 &= ~(7 << 0); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x104, reg32); + + /* Active State Power Management ASPM */ + + /* TODO */ + + /* Clear error bits */ + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0x06, 0xffff); + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0x1e, 0xffff); + pcie_write_config16(PCI_DEV(0, 0x01, 0), 0xaa, 0xffff); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x1c4, 0xffffffff); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x1d0, 0xffffffff); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x1f0, 0xffffffff); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x228, 0xffffffff); + + /* Program R/WO registers */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x308); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x308, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x314); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x314, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x324); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x324, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x328); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x328, reg32); + + reg8 = pcie_read_config8(PCI_DEV(0, 0x01, 0), 0xb4); + pcie_write_config8(PCI_DEV(0, 0x01, 0), 0xb4, reg8); + + /* Additional PCIe graphics setup */ + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xf0); + reg32 |= (3 << 26); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xf0, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xf0); + reg32 |= (3 << 24); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xf0, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0x200); + reg32 &= ~(3 << 26); + reg32 |= (2 << 26); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0x200, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xe80); + if (i945_silicon_revision() >= 2) { + reg32 |= (1 << 12); + } else { + reg32 &= ~(1 << 12); + } + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xe80, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xeb4); + reg32 &= ~(1 << 31); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xeb4, reg32); + + reg32 = pcie_read_config32(PCI_DEV(0, 0x01, 0), 0xfc); + reg32 |= (1 << 31); + pcie_write_config32(PCI_DEV(0, 0x01, 0), 0xfc, reg32); + + if (i945_silicon_revision() >= 3) { + static const u32 reglist[] = { + 0xec0, 0xed4, 0xee8, 0xefc, 0xf10, 0xf24, + 0xf38, 0xf4c, 0xf60, 0xf74, 0xf88, 0xf9c, + 0xfb0, 0xfc4, 0xfd8, 0xfec + }; + + int i; + for (i=0; i +#include +#include +#include + +static void gma_func0_init(struct device *dev) +{ + u32 reg32; + + /* IGD needs to be Bus Master */ + reg32 = pci_read_config32(dev, PCI_COMMAND); + pci_write_config32(dev, PCI_COMMAND, reg32 | PCI_COMMAND_MASTER); + + pci_dev_init(dev); +} + +static void gma_func1_init(struct device *dev) +{ + u32 reg32; + + /* IGD needs to be Bus Master, also enable IO accesss */ + reg32 = pci_read_config32(dev, PCI_COMMAND); + pci_write_config32(dev, PCI_COMMAND, reg32 | + PCI_COMMAND_MASTER | PCI_COMMAND_IO); +} + +static void gma_set_subsystem(device_t dev, unsigned vendor, unsigned device) +{ + if (!vendor || !device) { + pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, + pci_read_config32(dev, PCI_VENDOR_ID)); + } else { + pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, + ((device & 0xffff) << 16) | (vendor & 0xffff)); + } +} + +static struct pci_operations gma_pci_ops = { + .set_subsystem = gma_set_subsystem, +}; + +static struct device_operations gma_func0_ops = { + .read_resources = pci_dev_read_resources, + .set_resources = pci_dev_set_resources, + .enable_resources = pci_dev_enable_resources, + .init = gma_func0_init, + .scan_bus = 0, + .enable = 0, + .ops_pci = &gma_pci_ops, +}; + + +static struct device_operations gma_func1_ops = { + .read_resources = pci_dev_read_resources, + .set_resources = pci_dev_set_resources, + .enable_resources = pci_dev_enable_resources, + .init = gma_func1_init, + .scan_bus = 0, + .enable = 0, + .ops_pci = &gma_pci_ops, +}; + +static const struct pci_driver i945_gma_func0_driver __pci_driver = { + .ops = &gma_func0_ops, + .vendor = PCI_VENDOR_ID_INTEL, + .device = 0x27a2, +}; + +static const struct pci_driver i945_gma_func1_driver __pci_driver = { + .ops = &gma_func1_ops, + .vendor = PCI_VENDOR_ID_INTEL, + .device = 0x27a6, +}; + diff --git a/src/northbridge/intel/i945/northbridge.c b/src/northbridge/intel/i945/northbridge.c index ad1d938710..81fd58e07f 100644 --- a/src/northbridge/intel/i945/northbridge.c +++ b/src/northbridge/intel/i945/northbridge.c @@ -224,8 +224,8 @@ static void mc_read_resources(device_t dev) resource->flags = IORESOURCE_MEM | IORESOURCE_FIXED | IORESOURCE_STORED | IORESOURCE_ASSIGNED; - printk_debug("Adding PCIe enhanced config space BAR 0x%08x-0x%08x.\n", - resource->base, (resource->base + resource->size)); + printk_debug("Adding PCIe enhanced config space BAR 0x%08lx-0x%08lx.\n", + (unsigned long)(resource->base), (unsigned long)(resource->base + resource->size)); } static void mc_set_resources(device_t dev) @@ -245,8 +245,13 @@ static void mc_set_resources(device_t dev) static void intel_set_subsystem(device_t dev, unsigned vendor, unsigned device) { - pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, - ((device & 0xffff) << 16) | (vendor & 0xffff)); + if (!vendor || !device) { + pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, + pci_read_config32(dev, PCI_VENDOR_ID)); + } else { + pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, + ((device & 0xffff) << 16) | (vendor & 0xffff)); + } } static struct pci_operations intel_pci_ops = { diff --git a/src/northbridge/intel/i945/pcie_config.c b/src/northbridge/intel/i945/pcie_config.c index 94a088bf9d..5870b3453f 100644 --- a/src/northbridge/intel/i945/pcie_config.c +++ b/src/northbridge/intel/i945/pcie_config.c @@ -1,7 +1,7 @@ /* * This file is part of the coreboot project. * - * Copyright (C) 2007-2008 coresystems GmbH + * Copyright (C) 2007-2009 coresystems GmbH * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c index 8220e9bbd3..25ba6c25bf 100644 --- a/src/northbridge/intel/i945/raminit.c +++ b/src/northbridge/intel/i945/raminit.c @@ -2160,8 +2160,10 @@ static void sdram_post_jedec_initialization(struct sys_info *sysinfo) reg32 = MCHBAR32(DCC); #if CHANNEL_XOR_RANDOMIZATION reg32 &= ~(1 << 10); -#endif + reg32 |= (1 << 9); +#else reg32 &= ~(1 << 9); +#endif MCHBAR32(DCC) = reg32; } @@ -2242,7 +2244,12 @@ static void sdram_power_management(struct sys_info *sysinfo) } MCHBAR16(CPCTL) = reg16; +#if 0 + /* This is set later in the game */ if ((MCHBAR32(ECO) & (1 << 16)) != 0) { +#else + if (i945_silicon_revision() != 0) { +#endif switch (sysinfo->fsb_frequency) { case 667: MCHBAR32(HGIPMC2) = 0x0d590d59; break; case 533: MCHBAR32(HGIPMC2) = 0x155b155b; break; @@ -2306,7 +2313,7 @@ static void sdram_power_management(struct sys_info *sysinfo) /* stepping 0 and 1 */ MCHBAR32(FSBPMC4) &= ~(1 << 4); } else { - MCHBAR32(FSBPMC4) &= ~(1 << 4); + MCHBAR32(FSBPMC4) |= (1 << 4); } reg8 = pci_read_config8(PCI_DEV(0,0x0,0), 0xfc); diff --git a/src/northbridge/intel/i945/raminit.h b/src/northbridge/intel/i945/raminit.h index 33ed700826..deb07a6755 100644 --- a/src/northbridge/intel/i945/raminit.h +++ b/src/northbridge/intel/i945/raminit.h @@ -5,7 +5,7 @@ * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; version 2 of the License + * the Free Software Foundation; version 2 of the License. * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of -- cgit v1.2.3