We define IO_APIC_ADDR in <arch/ioapic.h>, let's use it.
[coreboot.git] / src / southbridge / via / vt8237r / vt8237r_lpc.c
index cd4464b6daa4b14cc647268972fd7170a86362d9..2de64f03ce1dd0e1598a37954bd1ab5169d18a4f 100644 (file)
@@ -1,11 +1,12 @@
 /*
- * This file is part of the LinuxBIOS project.
+ * This file is part of the coreboot project.
  *
- * Copyright (C) 2007 Rudolf Marek <r.marek@assembler.cz>
+ * Copyright (C) 2007, 2008 Rudolf Marek <r.marek@assembler.cz>
+ * Copyright (C) 2009 Jon Harrison <bothlyn@blueyonder.co.uk>
  *
  * 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.
+ * it under the terms of the GNU General Public License as published by
+ * 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
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
-#include <device/pci_ops.h>
 #include <device/pci_ids.h>
 #include <pc80/mc146818rtc.h>
+#include <arch/ioapic.h>
 #include <cpu/x86/lapic.h>
+#include <pc80/keyboard.h>
+#include <pc80/i8259.h>
 #include <stdlib.h>
 #include "vt8237r.h"
 #include "chip.h"
 
-#define ALL            (0xff << 24)
-#define NONE           (0)
-#define DISABLED       (1 << 16)
-#define ENABLED                (0 << 16)
-#define TRIGGER_EDGE   (0 << 15)
-#define TRIGGER_LEVEL  (1 << 15)
-#define POLARITY_HIGH  (0 << 13)
-#define POLARITY_LOW   (1 << 13)
-#define PHYSICAL_DEST  (0 << 11)
-#define LOGICAL_DEST   (1 << 11)
-#define ExtINT         (7 << 8)
-#define NMI            (4 << 8)
-#define SMI            (2 << 8)
-#define INT            (1 << 8)
-
-extern void dump_south(device_t dev);
-
-struct ioapicreg {
-       u32 reg;
-       u32 value_low;
-       u32 value_high;
-} ioapic_table[] = {
-       /* IO-APIC virtual wire mode configuration. */
-       /* mask, trigger, polarity, destination, delivery, vector */
-       {0,  ENABLED | TRIGGER_EDGE | POLARITY_HIGH | PHYSICAL_DEST |
-            ExtINT, NONE},
-       {1,  DISABLED, NONE},
-       {2,  DISABLED, NONE},
-       {3,  DISABLED, NONE},
-       {4,  DISABLED, NONE},
-       {5,  DISABLED, NONE},
-       {6,  DISABLED, NONE},
-       {7,  DISABLED, NONE},
-       {8,  DISABLED, NONE},
-       {9,  DISABLED, NONE},
-       {10, DISABLED, NONE},
-       {11, DISABLED, NONE},
-       {12, DISABLED, NONE},
-       {13, DISABLED, NONE},
-       {14, DISABLED, NONE},
-       {15, DISABLED, NONE},
-       {16, DISABLED, NONE},
-       {17, DISABLED, NONE},
-       {18, DISABLED, NONE},
-       {19, DISABLED, NONE},
-       {20, DISABLED, NONE},
-       {21, DISABLED, NONE},
-       {22, DISABLED, NONE},
-       {23, DISABLED, NONE},
-};
+static void southbridge_init_common(struct device *dev);
+
+#if CONFIG_EPIA_VT8237R_INIT
+                   /* Interrupts for  INT# A   B   C   D */
+static const unsigned char pciIrqs[4]  = { 10, 11, 12, 0};
 
-static void setup_ioapic(u32 ioapic_base)
+            /* Interrupt Assignments for Pins   1   2   3   4  */
+static const unsigned char sataPins[4] =     { 'A','B','C','D'};
+static const unsigned char vgaPins[4] =      { 'A','B','C','D'};
+static const unsigned char usbPins[4] =      { 'A','B','C','D'};
+static const unsigned char enetPins[4] =     { 'A','B','C','D'};
+static const unsigned char vt8237Pins[4] =   { 'A','B','C','D'};
+static const unsigned char slotPins[4] =     { 'C','D','A','B'};
+static const unsigned char riserPins[4] =    { 'D','C','B','A'};
+
+static unsigned char *pin_to_irq(const unsigned char *pin)
 {
-       u32 value_low, value_high, val;
-       volatile u32 *l;
+       static unsigned char Irqs[4];
        int i;
+       for (i = 0 ; i < 4 ; i++)
+               Irqs[i] = pciIrqs[ pin[i] - 'A' ];
 
-       /* All delivered to CPU0. */
-       ioapic_table[0].value_high = (lapicid()) << (56 - 32);
-       l = (unsigned long *) ioapic_base;
-
-       /* Set APIC to FSB message bus. */
-       l[0] = 0x3;
-       val = l[4];
-       l[4] = (val & 0xFFFFFE) | 1;
-
-       /* Set APIC ADDR - this will be VT8237R_APIC_ID. */
-       l[0] = 0;
-       val = l[4];
-       l[4] = (val & 0xF0FFFF) | (VT8237R_APIC_ID << 24);
-
-       for (i = 0; i < ARRAY_SIZE(ioapic_table); i++) {
-               l[0] = (ioapic_table[i].reg * 2) + 0x10;
-               l[4] = ioapic_table[i].value_low;
-               value_low = l[4];
-               l[0] = (ioapic_table[i].reg * 2) + 0x11;
-               l[4] = ioapic_table[i].value_high;
-               value_high = l[4];
-
-               if ((i == 0) && (value_low == 0xffffffff)) {
-                       printk_warning("IO APIC not responding.\n");
-                       return;
-               }
-       }
+       return Irqs;
 }
+#endif
 
 /** Set up PCI IRQ routing, route everything through APIC. */
 static void pci_routing_fixup(struct device *dev)
 {
+#if CONFIG_EPIA_VT8237R_INIT
+       device_t pdev;
+#endif
+
        /* PCI PNP Interrupt Routing INTE/F - disable */
        pci_write_config8(dev, 0x44, 0x00);
 
        /* PCI PNP Interrupt Routing INTG/H - disable */
        pci_write_config8(dev, 0x45, 0x00);
 
+       /* Gate Interrupts until RAM Writes are flushed */
+       pci_write_config8(dev, 0x49, 0x20);
+
+#if CONFIG_EPIA_VT8237R_INIT
+
+       /* Share INTE-INTH with INTA-INTD as per stock BIOS. */
+       pci_write_config8(dev, 0x46, 0x00);
+
+       /* setup PCI IRQ routing (For PCI Slot)*/
+       pci_write_config8(dev, 0x55, pciIrqs[0] << 4);
+       pci_write_config8(dev, 0x56, pciIrqs[1] | (pciIrqs[2] << 4) );
+       pci_write_config8(dev, 0x57, pciIrqs[3] << 4);
+
+       /* PCI Routing Fixup */
+
+       //Setup MiniPCI Slot
+       pci_assign_irqs(0, 0x14, pin_to_irq(slotPins));
+
+       // Via 2 slot riser card 2nd slot
+       pci_assign_irqs(0, 0x13, pin_to_irq(riserPins));
+
+       //Setup USB
+       pci_assign_irqs(0, 0x10, pin_to_irq(usbPins));
+
+       //Setup VT8237R Sound
+       pci_assign_irqs(0, 0x11, pin_to_irq(vt8237Pins));
+
+       //Setup Ethernet
+       pci_assign_irqs(0, 0x12, pin_to_irq(enetPins));
+
+       //Setup VGA
+       pci_assign_irqs(1, 0x00, pin_to_irq(vgaPins));
+
+       /* APIC Routing Fixup */
+
+       // Setup SATA
+       pdev = dev_find_device(PCI_VENDOR_ID_VIA,
+                               PCI_DEVICE_ID_VIA_VT6420_SATA, 0);
+       pci_write_config8(pdev, PCI_INTERRUPT_PIN, 0x02);
+       pci_assign_irqs(0, 0x0f, pin_to_irq(sataPins));
+
+
+       // Setup PATA Override
+       pdev = dev_find_device(PCI_VENDOR_ID_VIA,
+                               PCI_DEVICE_ID_VIA_82C586_1, 0);
+       pci_write_config8(pdev, PCI_INTERRUPT_PIN, 0x01);
+       pci_write_config8(pdev, PCI_INTERRUPT_LINE, 0xFF);
+
+#else
        /* Route INTE-INTH through registers above, no map to INTA-INTD. */
        pci_write_config8(dev, 0x46, 0x10);
 
@@ -140,14 +136,22 @@ static void pci_routing_fixup(struct device *dev)
 
        /* PCI INTD# Routing */
        pci_write_config8(dev, 0x57, 0x00);
+#endif
 }
 
+
+
 /**
  * Set up the power management capabilities directly into ACPI mode.
  * This avoids having to handle any System Management Interrupts (SMIs).
  */
-void setup_pm(device_t dev)
+
+extern u8 acpi_slp_type;
+
+
+static void setup_pm(device_t dev)
 {
+       u16 tmp;
        /* Debounce LID and PWRBTN# Inputs for 16ms. */
        pci_write_config8(dev, 0x80, 0x20);
 
@@ -157,39 +161,47 @@ void setup_pm(device_t dev)
        /* Set ACPI to 9, must set IRQ 9 override to level! Set PSON gating. */
        pci_write_config8(dev, 0x82, 0x40 | VT8237R_ACPI_IRQ);
 
+#if CONFIG_EPIA_VT8237R_INIT
+       /* Primary interupt channel, define wake events 0=IRQ0 15=IRQ15 1=en. */
+       pci_write_config16(dev, 0x84, 0x3052);
+#else
        /* Primary interupt channel, define wake events 0=IRQ0 15=IRQ15 1=en. */
        pci_write_config16(dev, 0x84, 0x30b2);
 
+#endif
        /* SMI output level to low, 7.5us throttle clock */
        pci_write_config8(dev, 0x8d, 0x18);
 
        /* GP Timer Control 1s */
        pci_write_config8(dev, 0x93, 0x88);
 
-       /* 7 = SMBus clock from RTC 32.768KHz
-        * 5 = Internal PLL reset from susp
-        * 2 = GPO2 is GPIO
+       /*
+        * 7 = SMBus clock from RTC 32.768KHz
+        * 5 = Internal PLL reset from susp disabled
+        * 2 = GPO2 is SUSA#
         */
-       pci_write_config8(dev, 0x94, 0xa4);
+       pci_write_config8(dev, 0x94, 0xa0);
 
-       /* 7 = stp to sust delay 1msec
+       /*
+        * 7 = stp to sust delay 1msec
         * 6 = SUSST# Deasserted Before PWRGD for STD
-        * 3 = GPO26/GPO27 is GPO 
+        * 5 = Keyboard/Mouse Swap
+        * 4 = PWRGOOD reset on VT8237A/S
+        * 3 = GPO26/GPO27 is GPO
         * 2 = Disable Alert on Lan
+        * 1 = SUSCLK/GPO4
+        * 0 = USB Wakeup
         */
+
+#if CONFIG_EPIA_VT8237R_INIT
+       pci_write_config8(dev, 0x95, 0xc2);
+#else
        pci_write_config8(dev, 0x95, 0xcc);
+#endif
 
        /* Disable GP3 timer. */
        pci_write_config8(dev, 0x98, 0);
 
-       /* Enable SATA LED, disable special CPU Frequency Change -
-        * GPIO28 GPIO22 GPIO29 GPIO23 are GPIOs.
-        */
-       pci_write_config8(dev, 0xe5, 0x9);
-
-       /* REQ5 as PCI request input - should be together with INTE-INTH. */
-       pci_write_config8(dev, 0xe4, 0x4);
-
        /* Enable ACPI accessm RTC signal gated with PSON. */
        pci_write_config8(dev, 0x81, 0x84);
 
@@ -221,10 +233,129 @@ void setup_pm(device_t dev)
        outb(0x0, VT8237R_ACPI_IO_BASE + 0x42);
 
        /* SCI is generated for RTC/pwrBtn/slpBtn. */
-       outw(0x001, VT8237R_ACPI_IO_BASE + 0x04);
+       tmp = inw(VT8237R_ACPI_IO_BASE + 0x04);
+#if CONFIG_HAVE_ACPI_RESUME == 1
+       acpi_slp_type = ((tmp & (7 << 10)) >> 10) == 1 ? 3 : 0 ;
+       printk(BIOS_DEBUG, "SLP_TYP type was %x %x\n", tmp, acpi_slp_type);
+#endif
+       /* clear sleep */
+       tmp &= ~(7 << 10);
+       tmp |= 1;
+       outw(tmp, VT8237R_ACPI_IO_BASE + 0x04);
 }
 
 static void vt8237r_init(struct device *dev)
+{
+       u8 enables;
+
+#if CONFIG_EPIA_VT8237R_INIT
+       printk(BIOS_SPEW, "Entering vt8237r_init, for EPIA.\n");
+       /*
+        * TODO: Looks like stock BIOS can do this but causes a hang
+        * Enable SATA LED, disable special CPU Frequency Change -
+        * GPIO28 GPIO22 GPIO29 GPIO23 are GPIOs.
+        * Setup to match EPIA default
+        * PCS0# on Pin U1
+        */
+       enables = pci_read_config8(dev, 0xe5);
+       enables |= 0x23;
+       pci_write_config8(dev, 0xe5, enables);
+
+       /*
+        * Enable Flash Write Access.
+        * Note EPIA-N Does not use REQ5 or PCISTP#(Hang)
+        */
+       enables = pci_read_config8(dev, 0xe4);
+       enables |= 0x2B;
+       pci_write_config8(dev, 0xe4, enables);
+
+       /* Enables Extra RTC Ports */
+       enables = pci_read_config8(dev, 0x4E);
+       enables |= 0x80;
+       pci_write_config8(dev, 0x4E, enables);
+
+#else
+       printk(BIOS_SPEW, "Entering vt8237r_init.\n");
+       /*
+        * Enable SATA LED, disable special CPU Frequency Change -
+        * GPIO28 GPIO22 GPIO29 GPIO23 are GPIOs.
+        */
+       pci_write_config8(dev, 0xe5, 0x09);
+
+       /* REQ5 as PCI request input - should be together with INTE-INTH. */
+       pci_write_config8(dev, 0xe4, 0x4);
+#endif
+
+       /* Set bit 3 of 0x4f (use INIT# as CPU reset). */
+       enables = pci_read_config8(dev, 0x4f);
+       enables |= 0x08;
+       pci_write_config8(dev, 0x4f, enables);
+
+#if CONFIG_EPIA_VT8237R_INIT
+       /*
+        * Set Read Pass Write Control Enable
+        */
+       pci_write_config8(dev, 0x48, 0x0c);
+#else
+       /*
+        * Set Read Pass Write Control Enable
+        * (force A2 from APIC FSB to low).
+        */
+       pci_write_config8(dev, 0x48, 0x8c);
+#endif
+
+       southbridge_init_common(dev);
+
+#if !CONFIG_EPIA_VT8237R_INIT
+       /* FIXME: Intel needs more bit set for C2/C3. */
+
+       /*
+        * Allow SLP# signal to assert LDTSTOP_L.
+        * Will work for C3 and for FID/VID change.
+        */
+       outb(0x1, VT8237R_ACPI_IO_BASE + 0x11);
+#endif
+
+       printk(BIOS_SPEW, "Leaving %s.\n", __func__);
+}
+
+static void vt8237s_init(struct device *dev)
+{
+       u32 tmp;
+
+       /* Put SPI base VT8237S_SPI_MEM_BASE. */
+       tmp = pci_read_config32(dev, 0xbc);
+       pci_write_config32(dev, 0xbc,
+                          (VT8237S_SPI_MEM_BASE >> 8) | (tmp & 0xFF000000));
+
+       /*
+        * REQ5 as PCI request input - should be together with INTE-INTH.
+        */
+       pci_write_config8(dev, 0xe4, 0x04);
+
+       /* Reduce further the STPCLK/LDTSTP signal to 5us. */
+       pci_write_config8(dev, 0xec, 0x4);
+
+       /* Host Bus Power Management Control, maybe not needed */
+       pci_write_config8(dev, 0x8c, 0x5);
+
+       /* Enable HPET at VT8237R_HPET_ADDR., does not work correctly on R. */
+       pci_write_config32(dev, 0x68, (VT8237R_HPET_ADDR | 0x80));
+
+       southbridge_init_common(dev);
+
+       /* FIXME: Intel needs more bit set for C2/C3. */
+
+       /*
+        * Allow SLP# signal to assert LDTSTOP_L.
+        * Will work for C3 and for FID/VID change. FIXME FIXME, pre rev A2.
+        */
+       outb(0xff, VT8237R_ACPI_IO_BASE + 0x50);
+
+       dump_south(dev);
+}
+
+static void vt8237_common_init(struct device *dev)
 {
        u8 enables, byte;
 
@@ -233,17 +364,31 @@ static void vt8237r_init(struct device *dev)
        byte |= PCI_COMMAND_WAIT;
        pci_write_config8(dev, PCI_COMMAND, byte);
 
+/* EPIA-N(L) Uses CN400 for BIOS Access */
+#if !CONFIG_EPIA_VT8237R_INIT
        /* Enable the internal I/O decode. */
        enables = pci_read_config8(dev, 0x6C);
        enables |= 0x80;
        pci_write_config8(dev, 0x6C, enables);
 
-       /* FIXME: Map 4MB of flash into the address space,
-        * this should be in CAR call.
+       /*
+        * ROM decode
+        * bit range
+        *   7 000E0000h-000EFFFFh
+        *   6 FFF00000h-FFF7FFFFh
+        *   5 FFE80000h-FFEFFFFFh
+        *   4 FFE00000h-FFE7FFFFh
+        *   3 FFD80000h-FFDFFFFFh
+        *   2 FFD00000h-FFD7FFFFh
+        *   1 FFC80000h-FFCFFFFFh
+        *   0 FFC00000h-FFC7FFFFh
+        * So 0x7f here sets ROM decode to FFC00000-FFFFFFFF or 4Mbyte.
         */
-       /* pci_write_config8(dev, 0x41, 0x7f); */
+       pci_write_config8(dev, 0x41, 0x7f);
+#endif
 
-       /* Set bit 6 of 0x40 (I/O recovery time).
+       /*
+        * Set bit 6 of 0x40 (I/O recovery time).
         * IMPORTANT FIX - EISA = ECLR reg at 0x4d0! Decoding must be on so
         * that PCI interrupts can be properly marked as level triggered.
         */
@@ -259,31 +404,52 @@ static void vt8237r_init(struct device *dev)
        /* Delay transaction control */
        pci_write_config8(dev, 0x43, 0xb);
 
-       /* I/O recovery time */
-       pci_write_config8(dev, 0x4c, 0x44);
+#if CONFIG_EPIA_VT8237R_INIT
+       /* I/O recovery time, default IDE routing */
+       pci_write_config8(dev, 0x4c, 0x04);
 
        /* ROM memory cycles go to LPC. */
        pci_write_config8(dev, 0x59, 0x80);
 
-       /* Bypass APIC De-Assert Message, INTE#, INTF#, INTG#, INTH# as PCI. */
-       pci_write_config8(dev, 0x5B, 0xb);
+       /*
+        * Bit | Meaning
+        * -------------
+        *   3 | Bypass APIC De-Assert Message (1=Enable)
+        *   1 | possibly "INTE#, INTF#, INTG#, INTH# as PCI"
+        *     | bit 1=1 works for Aaron at VIA, bit 1=0 works for jakllsch
+        *   0 | Dynamic Clock Gating Main Switch (1=Enable)
+        */
+       pci_write_config8(dev, 0x5b, 0x9);
 
-       /* Set Read Pass Write Control Enable (force A2 from APIC FSB to low). */
-       pci_write_config8(dev, 0x48, 0x8c);
+       /* Set 0x58 to 0x42 APIC On and RTC Write Protect.*/
+       pci_write_config8(dev, 0x58, 0x42);
+
+       /* Enable serial IRQ, 6PCI clocks. */
+       pci_write_config8(dev, 0x52, 0x9);
+#else
+       /* I/O recovery time, default IDE routing */
+       pci_write_config8(dev, 0x4c, 0x44);
+
+       /* ROM memory cycles go to LPC. */
+       pci_write_config8(dev, 0x59, 0x80);
+
+       /*
+        * Bit | Meaning
+        * -------------
+        *   3 | Bypass APIC De-Assert Message (1=Enable)
+        *   1 | possibly "INTE#, INTF#, INTG#, INTH# as PCI"
+        *     | bit 1=1 works for Aaron at VIA, bit 1=0 works for jakllsch
+        *   0 | Dynamic Clock Gating Main Switch (1=Enable)
+        */
+       pci_write_config8(dev, 0x5b, 0xb);
 
        /* Set 0x58 to 0x43 APIC and RTC. */
        pci_write_config8(dev, 0x58, 0x43);
 
-       /* Set bit 3 of 0x4f (use INIT# as CPU reset). */
-       enables = pci_read_config8(dev, 0x4f);
-       enables |= 0x08;
-       pci_write_config8(dev, 0x4f, enables);
-
        /* Enable serial IRQ, 6PCI clocks. */
        pci_write_config8(dev, 0x52, 0x9);
 
-       /* Enable HPET at VT8237R_HPET_ADDR. */
-       pci_write_config32(dev, 0x68, (VT8237R_HPET_ADDR | 0x80));
+#endif
 
        /* Power management setup */
        setup_pm(dev);
@@ -292,59 +458,93 @@ static void vt8237r_init(struct device *dev)
        rtc_init(0);
 }
 
-void vt8237r_read_resources(device_t dev)
+static void vt8237r_read_resources(device_t dev)
 {
        struct resource *res;
 
        pci_dev_read_resources(dev);
+
+       /* Fixed ACPI Base IO Base*/
+       res = new_resource(dev, 0x88);
+       res->base = VT8237R_ACPI_IO_BASE;
+       res->size = 128;
+       res->limit = 0xffffUL;
+       res->flags = IORESOURCE_IO | IORESOURCE_FIXED |
+                    IORESOURCE_STORED | IORESOURCE_ASSIGNED;
+
+       /* Fixed EISA ECLR I/O Regs     */
+       res = new_resource(dev, 3);
+       res->base = 0x4d0;
+       res->size = 2;
+       res->limit = 0xffffUL;
+       res->flags = IORESOURCE_IO | IORESOURCE_FIXED |
+                    IORESOURCE_STORED | IORESOURCE_ASSIGNED;
+
+       /* Fixed System Management Bus I/O Resource */
+       res = new_resource(dev, 0xD0);
+       res->base = VT8237R_SMBUS_IO_BASE;
+       res->size = 16;
+       res->limit = 0xffffUL;
+       res->flags = IORESOURCE_IO | IORESOURCE_FIXED |
+                    IORESOURCE_STORED | IORESOURCE_ASSIGNED;
+
        /* Fixed APIC resource */
        res = new_resource(dev, 0x44);
-       res->base = VT8237R_APIC_BASE;
+       res->base = IO_APIC_ADDR;
        res->size = 256;
-       res->limit = res->base + res->size - 1;
+       res->limit = 0xffffffffUL;
        res->align = 8;
        res->gran = 8;
        res->flags = IORESOURCE_MEM | IORESOURCE_FIXED |
                     IORESOURCE_STORED | IORESOURCE_ASSIGNED;
-}
 
-/**
- * The VT8237R is not a PCI bridge and has no resources of its own (other
- * than standard PC I/O addresses), however it does control the ISA bus
- * and so we need to manually call enable childrens resources on that bus.
- */
-void vt8237r_enable_resources(device_t dev)
-{
-       pci_dev_enable_resources(dev);
-       enable_childrens_resources(dev);
+       res = new_resource(dev, 1);
+       res->base = 0x0UL;
+       res->size = 0x1000UL;
+       res->limit = 0xffffUL;
+       res->flags = IORESOURCE_IO | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
 }
 
 static void init_keyboard(struct device *dev)
 {
        u8 regval = pci_read_config8(dev, 0x51);
        if (regval & 0x1)
-               init_pc_keyboard(0x60, 0x64, 0);
+               pc_keyboard_init(0);
 }
 
-static void southbridge_init(struct device *dev)
+static void southbridge_init_common(struct device *dev)
 {
-       vt8237r_init(dev);
+       vt8237_common_init(dev);
        pci_routing_fixup(dev);
-       setup_ioapic(VT8237R_APIC_BASE);
+       setup_ioapic(IO_APIC_ADDR, VT8237R_APIC_ID);
        setup_i8259();
        init_keyboard(dev);
 }
 
-static struct device_operations vt8237r_lpc_ops = {
-       .read_resources = vt8237r_read_resources,
-       .set_resources = pci_dev_set_resources,
-       .enable_resources = vt8237r_enable_resources,
-       .init = &southbridge_init,
-       .scan_bus = scan_static_bus,
+static const struct device_operations vt8237r_lpc_ops_s = {
+       .read_resources         = vt8237r_read_resources,
+       .set_resources          = pci_dev_set_resources,
+       .enable_resources       = pci_dev_enable_resources,
+       .init                   = vt8237s_init,
+       .scan_bus               = scan_static_bus,
+};
+
+static const struct device_operations vt8237r_lpc_ops_r = {
+       .read_resources         = vt8237r_read_resources,
+       .set_resources          = pci_dev_set_resources,
+       .enable_resources       = pci_dev_enable_resources,
+       .init                   = vt8237r_init,
+       .scan_bus               = scan_static_bus,
+};
+
+static const struct pci_driver lpc_driver_r __pci_driver = {
+       .ops    = &vt8237r_lpc_ops_r,
+       .vendor = PCI_VENDOR_ID_VIA,
+       .device = PCI_DEVICE_ID_VIA_VT8237R_LPC,
 };
 
-static struct pci_driver lpc_driver __pci_driver = {
-       .ops = &vt8237r_lpc_ops,
-       .vendor = PCI_VENDOR_ID_VIA,
-       .device = PCI_DEVICE_ID_VIA_VT8237R_LPC,
+static const struct pci_driver lpc_driver_s __pci_driver = {
+       .ops    = &vt8237r_lpc_ops_s,
+       .vendor = PCI_VENDOR_ID_VIA,
+       .device = PCI_DEVICE_ID_VIA_VT8237S_LPC,
 };