Fine-tune the V-link bus between K8T890 and VT8237R and set

it to 8X transfer rate (up to 1066 MB/s) similar code placed here would be
needed for VT8237A/S etc. Using VIA recommended values despite they are for
K8T890CF, this is K8T890CE (still dont know what is exactly different).

This patch enables the parity error reporting on V-Link, so it enables NMI
generation for the SERR# errors. The NMI may not be generated, maybe port
61h needs some tuning too.

Signed-off-by: Rudolf Marek <r.marek@assembler.cz>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2958 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

diff --git a/src/southbridge/via/k8t890/Config.lb b/src/southbridge/via/k8t890/Config.lb
index e18c1a20bc2b3a95984a25ac38912078df8abfb8..2930e9b72f6fabc86ca806594e39b0c36c36a6d3 100644 (file)
--- a/src/southbridge/via/k8t890/Config.lb
+++ b/src/southbridge/via/k8t890/Config.lb
@@ -23,3 +23,4 @@ driver k8t890_host.o
 driver k8t890_host_ctrl.o
 driver k8t890_pcie.o
 driver k8t890_traf_ctrl.o
+driver k8t890_error.o

diff --git a/src/southbridge/via/k8t890/k8t890_ctrl.c b/src/southbridge/via/k8t890/k8t890_ctrl.c
index c8ca2122d0edd4a91fb75b7591e9ce652a5f545e..c57c165195c5cd6b1433a5dc0b35f77c6f6cbfe1 100644 (file)
--- a/src/southbridge/via/k8t890/k8t890_ctrl.c
+++ b/src/southbridge/via/k8t890/k8t890_ctrl.c
@@ -23,6 +23,56 @@
 #include <device/pci_ids.h>
 #include <console/console.h>
 
+/**
+ * Setup the V-Link for VT8237R, 8X mode.
+ *
+ * For K8T890CF VIA recommends what is in VIA column, AW is award 8X:
+ *
+ *                                              REG   DEF   AW  VIA-8X VIA-4X
+ *                                              -----------------------------
+ * NB V-Link Manual Driving Control strobe      0xb5  0x46  0x46  0x88  0x88
+ * NB V-Link Manual Driving Control - Data      0xb6  0x46  0x46  0x88  0x88
+ * NB V-Link Receiving Strobe Delay             0xb7  0x02  0x02  0x61  0x01
+ * NB V-Link Compensation Control bit4,0 (b5,b6) 0xb4  0x10  0x10  0x11  0x11
+ * SB V-Link Strobe Drive Control               0xb9  0x00  0xa5  0x98  0x98
+ * SB V-Link Data drive Control????             0xba  0x00  0xbb  0x77  0x77
+ * SB V-Link Receive Strobe Delay????           0xbb  0x04  0x11  0x11  0x11
+ * SB V-Link Compensation Control bit0 (use b9)         0xb8  0x00  0x01  0x01  0x01
+ * V-Link CKG Control                           0xb0  0x05  0x05  0x06  0x03
+ * V-Link CKG Control                           0xb1  0x05  0x05  0x01  0x03
+ */
+static void ctrl_init(struct device *dev)
+{
+       u8 reg;
+       device_t devsb = dev_find_device(PCI_VENDOR_ID_VIA,
+                                        PCI_DEVICE_ID_VIA_VT8237R_LPC, 0);
+
+       if (!devsb)
+               return;
+
+       pci_write_config8(dev, 0xb5, 0x88);
+       pci_write_config8(dev, 0xb6, 0x88);
+       pci_write_config8(dev, 0xb7, 0x61);
+
+       reg = pci_read_config8(dev, 0xb4);
+       reg |= 0x11;
+       pci_write_config8(dev, 0xb4, reg);
+
+       pci_write_config8(dev, 0xb9, 0x98);
+       pci_write_config8(dev, 0xba, 0x77);
+       pci_write_config8(dev, 0xbb, 0x11);
+
+       reg = pci_read_config8(dev, 0xb8);
+       reg |= 0x1;
+       pci_write_config8(dev, 0xb8, reg);
+
+       pci_write_config8(dev, 0xb0, 0x06);
+       pci_write_config8(dev, 0xb1, 0x01);
+
+       /* Program V-link 8X 16bit full duplex, parity enabled. */
+       pci_write_config8(dev, 0x48, 0xa3);
+}
+
 static void ctrl_enable(struct device *dev)
 {
        u8 regm, regm2, regm3;
@@ -36,11 +86,6 @@ static void ctrl_enable(struct device *dev)
        /* PCI CFG Address bits[27:24] are used as extended register address
           bit[11:8] */
        pci_write_config8(dev, 0x47, 0x30);
-
-       /* FIXME: Program V-link 8X 16bit full duplex, this needs to be fixed
-          for other than VT8237R SB */
-       pci_write_config8(dev, 0x48, 0x23);
-
        /* Magic init. This is not well documented :/ */
        pci_write_config8(dev, 0x70, 0xc2);
 
@@ -92,6 +137,7 @@ static struct device_operations ctrl_ops = {
        .set_resources = pci_dev_set_resources,
        .enable_resources = pci_dev_enable_resources,
        .enable = ctrl_enable,
+       .init = ctrl_init,
        .ops_pci = 0,
 };
 

diff --git a/src/southbridge/via/k8t890/k8t890_error.c b/src/southbridge/via/k8t890/k8t890_error.c
new file mode 100644 (file)
index 0000000..49d32bf
--- /dev/null
+++ b/src/southbridge/via/k8t890/k8t890_error.c
@@ -0,0 +1,47 @@
+/*
+ * This file is part of the LinuxBIOS project.
+ *
+ * Copyright (C) 2007 Rudolf Marek <r.marek@assembler.cz>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License v2 as published by
+ * the Free Software Foundation.
+ *
+ * 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.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ */
+
+#include <device/device.h>
+#include <device/pci.h>
+#include <device/pci_ids.h>
+#include <console/console.h>
+
+static void error_enable(struct device *dev)
+{
+       /*
+        * bit0 - Enable V-link parity error reporting in 0x50 bit0 (RWC)
+        * bit6 - Parity Error/SERR# Report Through V-Link to SB
+        * bit7 - Parity Error/SERR# Report Through NMI
+        */
+       pci_write_config8(dev, 0x58, 0x81);
+}
+
+static const struct device_operations error_ops = {
+       .read_resources = pci_dev_read_resources,
+       .set_resources = pci_dev_set_resources,
+       .enable_resources = pci_dev_enable_resources,
+       .enable = error_enable,
+       .ops_pci = 0,
+};
+
+static const struct pci_driver northbridge_driver __pci_driver = {
+       .ops = &error_ops,
+       .vendor = PCI_VENDOR_ID_VIA,
+       .device = PCI_DEVICE_ID_VIA_K8T890CE_1,
+};