Without them the BS_DEV_RESOURCES stage won't traverse the bridge and the graphics controller would be left without resources assigned. Even worse, the resources would stay based in offset 0 which confuses the MTRR setting code and causes a good chunk of the DRAM to be set to type write combining. With the patch applied, the resources are set: Show resources in subtree (Root Device)...After assigning values. ... PCI: 00:01.0 child on link 0 PCI: 01:00.0 + PCI: 00:01.0 resource base ffff size 0 align 0 gran 0 limit ffff flags 60080100 index 0 + PCI: 00:01.0 resource base f8000000 size 4000000 align 26 gran 0 limit fbffffff flags 60081200 index 1 + PCI: 00:01.0 resource base fc000000 size 1010000 align 24 gran 0 limit fd00ffff flags 60080200 index 2 PCI: 01:00.0 - PCI: 01:00.0 resource base 0 size 4000000 align 26 gran 26 limit ffffffff flags 1200 index 10 - PCI: 01:00.0 resource base 0 size 1000000 align 24 gran 24 limit ffffffff flags 200 index 14 - PCI: 01:00.0 resource base 0 size 10000 align 16 gran 16 limit ffffffff flags 2200 index 30 + PCI: 01:00.0 resource base f8000000 size 4000000 align 26 gran 26 limit fbffffff flags 60001200 index 10 + PCI: 01:00.0 resource base fc000000 size 1000000 align 24 gran 24 limit fcffffff flags 60000200 index 14 + PCI: 01:00.0 resource base fd000000 size 10000 align 16 gran 16 limit fd00ffff flags 60002200 index 30 And the caching mode is set properly: MTRR: Physical address space: -0x0000000000000000 - 0x0000000004000000 size 0x04000000 type 1 -0x0000000004000000 - 0x000000000e000000 size 0x0a000000 type 6 -0x000000000e000000 - 0x0000000100000000 size 0xf2000000 type 0 +0x0000000000000000 - 0x00000000000a0000 size 0x000a0000 type 6 +0x00000000000a0000 - 0x00000000000c0000 size 0x00020000 type 0 +0x00000000000c0000 - 0x000000000e000000 size 0x0df40000 type 6 +0x000000000e000000 - 0x00000000f8000000 size 0xea000000 type 0 +0x00000000f8000000 - 0x00000000fc000000 size 0x04000000 type 1 +0x00000000fc000000 - 0x0000000100000000 size 0x04000000 type 0 The problem was also spot and discussed here: http://coreboot.coreboot.narkive.com/E9eGauzH/via-c7-on-bcom-winnet-p680-l1-l2-cache-very-slow Change-Id: Idb4979b206838dd6455b2a16de14dc74f83af921 Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> Reviewed-on: https://review.coreboot.org/18894 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
199 lines
5.5 KiB
C
199 lines
5.5 KiB
C
/*
|
|
* This file is part of the coreboot project.
|
|
*
|
|
* Copyright (C) 2007 Corey Osgood <corey.osgood@gmail.com>
|
|
*
|
|
* 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; either version 2 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*/
|
|
|
|
#include <console/console.h>
|
|
#include <arch/io.h>
|
|
#include <stdint.h>
|
|
#include <device/device.h>
|
|
#include <device/pci.h>
|
|
#include <device/pci_ids.h>
|
|
#include "northbridge.h"
|
|
#include "cn700.h"
|
|
|
|
/* This is the main AGP device, and only one used when configured for AGP 2.0 */
|
|
static void agp_init(device_t dev)
|
|
{
|
|
u32 reg32;
|
|
|
|
/* Some of this may not be necessary (should be handled by the OS). */
|
|
printk(BIOS_DEBUG, "Enabling AGP.\n");
|
|
|
|
/* Allow R/W access to AGP registers. */
|
|
pci_write_config8(dev, 0x4d, 0x15);
|
|
|
|
/* Setup PCI latency timer. */
|
|
pci_write_config8(dev, 0xd, 0x8);
|
|
|
|
/*
|
|
* Set to AGP 3.0 Mode, which should theoretically render the rest of
|
|
* the registers set here pointless.
|
|
*/
|
|
pci_write_config8(dev, 0x84, 0xb);
|
|
|
|
/* AGP Request Queue Size */
|
|
pci_write_config8(dev, 0x4a, 0x1f);
|
|
|
|
/*
|
|
* AGP Hardware Support (default 0xc4)
|
|
* 7: AGP SBA Enable (1 to Enable)
|
|
* 6: AGP Enable
|
|
* 5: Reserved
|
|
* 4: Fast Write Enable
|
|
* 3: AGP8X Mode Enable
|
|
* 2: AGP4X Mode Enable
|
|
* 1: AGP2X Mode Enable
|
|
* 0: AGP1X Mode Enable
|
|
*/
|
|
pci_write_config8(dev, 0x4b, 0xc4);
|
|
|
|
/* Enable AGP Backdoor */
|
|
pci_write_config8(dev, 0xb5, 0x03);
|
|
|
|
/* Set aperture to 32 MB. */
|
|
/* TODO: Use config option, explain how it works. */
|
|
pci_write_config32(dev, 0x94, 0x00010f38);
|
|
/* Set GART Table Base Address (31:12). */
|
|
pci_write_config32(dev, 0x98, (0x1558 << 12));
|
|
/* Set AGP Aperture Base. */
|
|
pci_write_config32(dev, 0x10, 0xf8000008);
|
|
|
|
/* Enable CPU/PMSTR GART Access. */
|
|
reg32 = pci_read_config8(dev, 0xbf);
|
|
reg32 |= 0x80;
|
|
pci_write_config8(dev, 0xbf, reg32);
|
|
|
|
/* Enable AGP Aperture. */
|
|
reg32 = pci_read_config32(dev, 0x94);
|
|
reg32 |= (3 << 7);
|
|
pci_write_config32(dev, 0x90, reg32);
|
|
|
|
/* AGP Control */
|
|
pci_write_config8(dev, 0xbc, 0x21);
|
|
pci_write_config8(dev, 0xbd, 0xd2);
|
|
|
|
/*
|
|
* AGP Pad, driving strength, and delay control. All this should be
|
|
* constant, seeing as the VGA controller is onboard.
|
|
*/
|
|
pci_write_config8(dev, 0x40, 0xc7);
|
|
pci_write_config8(dev, 0x41, 0xdb);
|
|
pci_write_config8(dev, 0x42, 0x10);
|
|
pci_write_config8(dev, 0x43, 0xdb);
|
|
pci_write_config8(dev, 0x44, 0x24);
|
|
|
|
/* AGPC CKG Control */
|
|
pci_write_config8(dev, 0xc0, 0x02);
|
|
pci_write_config8(dev, 0xc1, 0x02);
|
|
}
|
|
|
|
static const struct device_operations agp_operations = {
|
|
.read_resources = DEVICE_NOOP,
|
|
.set_resources = pci_dev_set_resources,
|
|
.enable_resources = pci_dev_enable_resources,
|
|
.init = agp_init,
|
|
.ops_pci = 0,
|
|
};
|
|
|
|
static const struct pci_driver agp_driver __pci_driver = {
|
|
.ops = &agp_operations,
|
|
.vendor = PCI_VENDOR_ID_VIA,
|
|
.device = PCI_DEVICE_ID_VIA_CN700_AGP,
|
|
};
|
|
|
|
/*
|
|
* This is the AGP 3.0 "bridge" @Bus 0 Device 1 Func 0. When using AGP 3.0, the
|
|
* config in this device takes presidence. We configure both just to be safe.
|
|
*/
|
|
static void agp_bridge_init(device_t dev)
|
|
{
|
|
printk(BIOS_DEBUG, "Setting up AGP bridge device\n");
|
|
|
|
pci_write_config16(dev, 0x4, 0x0007);
|
|
|
|
/* Secondary Bus Number */
|
|
pci_write_config8(dev, 0x19, 0x01);
|
|
/* Subordinate Bus Number */
|
|
pci_write_config8(dev, 0x1a, 0x01);
|
|
/* I/O Base */
|
|
pci_write_config8(dev, 0x1c, 0xd0);
|
|
/* I/O Limit */
|
|
pci_write_config8(dev, 0x1d, 0xd0);
|
|
|
|
/* Memory Base */
|
|
pci_write_config16(dev, 0x20, 0xfb00);
|
|
/* Memory Limit */
|
|
pci_write_config16(dev, 0x22, 0xfcf0);
|
|
/* Prefetchable Memory Base */
|
|
pci_write_config16(dev, 0x24, 0xf400);
|
|
/* Prefetchable Memory Limit */
|
|
pci_write_config16(dev, 0x26, 0xf7f0);
|
|
/* Enable VGA Compatible Memory/IO Range */
|
|
pci_write_config8(dev, 0x3e, 0x08);
|
|
|
|
/* Second PCI Bus Control (see datasheet) */
|
|
pci_write_config8(dev, 0x40, 0x83);
|
|
pci_write_config8(dev, 0x41, 0x43);
|
|
pci_write_config8(dev, 0x42, 0xe2);
|
|
pci_write_config8(dev, 0x43, 0x44);
|
|
pci_write_config8(dev, 0x44, 0x34);
|
|
pci_write_config8(dev, 0x45, 0x72);
|
|
}
|
|
|
|
static void agp_bridge_read_resources(device_t dev)
|
|
{
|
|
struct resource *resource;
|
|
|
|
resource = new_resource(dev, 0);
|
|
if (resource) {
|
|
resource->base = 0;
|
|
resource->size = 0;
|
|
resource->limit = 0xffffUL;
|
|
resource->flags = IORESOURCE_IO | IORESOURCE_BRIDGE;
|
|
}
|
|
|
|
resource = new_resource(dev, 1);
|
|
if (resource) {
|
|
resource->base = 0;
|
|
resource->size = 0;
|
|
resource->limit = 0xfffffffffULL;
|
|
resource->flags = IORESOURCE_MEM | IORESOURCE_PREFETCH;
|
|
resource->flags |= IORESOURCE_BRIDGE;
|
|
}
|
|
|
|
resource = new_resource(dev, 2);
|
|
if (resource) {
|
|
resource->base = 0;
|
|
resource->size = 0;
|
|
resource->limit = 0xffffffffULL;
|
|
resource->flags = IORESOURCE_MEM | IORESOURCE_BRIDGE;
|
|
}
|
|
}
|
|
|
|
static const struct device_operations agp_bridge_operations = {
|
|
.read_resources = agp_bridge_read_resources,
|
|
.set_resources = pci_dev_set_resources,
|
|
.enable_resources = pci_bus_enable_resources,
|
|
.init = agp_bridge_init,
|
|
.scan_bus = pci_scan_bridge,
|
|
.ops_pci = 0,
|
|
};
|
|
|
|
static const struct pci_driver agp_bridge_driver __pci_driver = {
|
|
.ops = &agp_bridge_operations,
|
|
.vendor = PCI_VENDOR_ID_VIA,
|
|
.device = PCI_DEVICE_ID_VIA_CN700_BRIDGE,
|
|
};
|