fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git] / drivers / parport / parport_pc.c
index 831d678..b61c17b 100644 (file)
@@ -1,9 +1,9 @@
 /* Low-level parallel-port routines for 8255-based PC-style hardware.
  * 
- * Authors: Phil Blundell <Philip.Blundell@pobox.com>
+ * Authors: Phil Blundell <philb@gnu.org>
  *          Tim Waugh <tim@cyberelk.demon.co.uk>
  *         Jose Renau <renau@acm.org>
- *          David Campbell <campbell@torque.net>
+ *          David Campbell
  *          Andrea Arcangeli
  *
  * based on work by Grant Guenther <grant@torque.net> and Phil Blundell.
@@ -42,7 +42,6 @@
  * but rather will start at port->base_hi.
  */
 
-#include <linux/config.h>
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/sched.h>
 
 #include <linux/parport.h>
 #include <linux/parport_pc.h>
+#include <linux/via.h>
 #include <asm/parport.h>
 
 #define PARPORT_PC_MAX_PORTS PARPORT_MAX
 
+#ifdef CONFIG_ISA_DMA_API
+#define HAS_DMA
+#endif
+
 /* ECR modes */
 #define ECR_SPP 00
 #define ECR_PS2 01
@@ -92,7 +96,7 @@ static struct superio_struct {        /* For Super-IO chips autodetection */
        int io;
        int irq;
        int dma;
-} superios[NR_SUPERIOS] __devinitdata = { {0,},};
+} superios[NR_SUPERIOS] = { {0,},};
 
 static int user_specified;
 #if defined(CONFIG_PARPORT_PC_SUPERIO) || \
@@ -168,8 +172,7 @@ static int change_mode(struct parport *p, int m)
                                if (time_after_eq (jiffies, expire))
                                        /* The FIFO is stuck. */
                                        return -EBUSY;
-                               __set_current_state (TASK_INTERRUPTIBLE);
-                               schedule_timeout ((HZ + 99) / 100);
+                               schedule_timeout_interruptible(msecs_to_jiffies(10));
                                if (signal_pending (current))
                                        break;
                        }
@@ -192,6 +195,7 @@ static int change_mode(struct parport *p, int m)
 
 #ifdef CONFIG_PARPORT_1284
 /* Find FIFO lossage; FIFO is reset */
+#if 0
 static int get_fifo_residue (struct parport *p)
 {
        int residue;
@@ -232,6 +236,7 @@ static int get_fifo_residue (struct parport *p)
        DPRINTK (KERN_DEBUG "*** get_fifo_residue: done residue collecting (ecr = 0x%2.2x)\n", inb (ECONTROL (p)));
        return residue;
 }
+#endif  /*  0 */
 #endif /* IEEE 1284 support */
 #endif /* FIFO support */
 
@@ -265,14 +270,14 @@ static int clear_epp_timeout(struct parport *pb)
  * of these are in parport_pc.h.
  */
 
-static irqreturn_t parport_pc_interrupt(int irq, void *dev_id, struct pt_regs *regs)
+static irqreturn_t parport_pc_interrupt(int irq, void *dev_id)
 {
-       parport_generic_irq(irq, (struct parport *) dev_id, regs);
+       parport_generic_irq(irq, (struct parport *) dev_id);
        /* FIXME! Was it really ours? */
        return IRQ_HANDLED;
 }
 
-void parport_pc_init_state(struct pardevice *dev, struct parport_state *s)
+static void parport_pc_init_state(struct pardevice *dev, struct parport_state *s)
 {
        s->u.pc.ctr = 0xc;
        if (dev->irq_func &&
@@ -284,7 +289,7 @@ void parport_pc_init_state(struct pardevice *dev, struct parport_state *s)
                             * D.Gruszka VScom */
 }
 
-void parport_pc_save_state(struct parport *p, struct parport_state *s)
+static void parport_pc_save_state(struct parport *p, struct parport_state *s)
 {
        const struct parport_pc_private *priv = p->physport->private_data;
        s->u.pc.ctr = priv->ctr;
@@ -292,7 +297,7 @@ void parport_pc_save_state(struct parport *p, struct parport_state *s)
                s->u.pc.ecr = inb (ECONTROL (p));
 }
 
-void parport_pc_restore_state(struct parport *p, struct parport_state *s)
+static void parport_pc_restore_state(struct parport *p, struct parport_state *s)
 {
        struct parport_pc_private *priv = p->physport->private_data;
        register unsigned char c = s->u.pc.ctr & priv->ctr_writable;
@@ -607,6 +612,7 @@ dump_parport_state ("leave fifo_write_block_pio", port);
        return length - left;
 }
 
+#ifdef HAS_DMA
 static size_t parport_pc_fifo_write_block_dma (struct parport *port,
                                               const void *buf, size_t length)
 {
@@ -729,11 +735,22 @@ dump_parport_state ("enter fifo_write_block_dma", port);
 dump_parport_state ("leave fifo_write_block_dma", port);
        return length - left;
 }
+#endif
+
+static inline size_t parport_pc_fifo_write_block(struct parport *port,
+                                              const void *buf, size_t length)
+{
+#ifdef HAS_DMA
+       if (port->dma != PARPORT_DMA_NONE)
+               return parport_pc_fifo_write_block_dma (port, buf, length);
+#endif
+       return parport_pc_fifo_write_block_pio (port, buf, length);
+}
 
 /* Parallel Port FIFO mode (ECP chipsets) */
-size_t parport_pc_compat_write_block_pio (struct parport *port,
-                                         const void *buf, size_t length,
-                                         int flags)
+static size_t parport_pc_compat_write_block_pio (struct parport *port,
+                                                const void *buf, size_t length,
+                                                int flags)
 {
        size_t written;
        int r;
@@ -755,10 +772,7 @@ size_t parport_pc_compat_write_block_pio (struct parport *port,
        port->physport->ieee1284.phase = IEEE1284_PH_FWD_DATA;
 
        /* Write the data to the FIFO. */
-       if (port->dma != PARPORT_DMA_NONE)
-               written = parport_pc_fifo_write_block_dma (port, buf, length);
-       else
-               written = parport_pc_fifo_write_block_pio (port, buf, length);
+       written = parport_pc_fifo_write_block(port, buf, length);
 
        /* Finish up. */
        /* For some hardware we don't want to touch the mode until
@@ -808,9 +822,9 @@ size_t parport_pc_compat_write_block_pio (struct parport *port,
 
 /* ECP */
 #ifdef CONFIG_PARPORT_1284
-size_t parport_pc_ecp_write_block_pio (struct parport *port,
-                                      const void *buf, size_t length,
-                                      int flags)
+static size_t parport_pc_ecp_write_block_pio (struct parport *port,
+                                             const void *buf, size_t length,
+                                             int flags)
 {
        size_t written;
        int r;
@@ -853,10 +867,7 @@ size_t parport_pc_ecp_write_block_pio (struct parport *port,
        port->physport->ieee1284.phase = IEEE1284_PH_FWD_DATA;
 
        /* Write the data to the FIFO. */
-       if (port->dma != PARPORT_DMA_NONE)
-               written = parport_pc_fifo_write_block_dma (port, buf, length);
-       else
-               written = parport_pc_fifo_write_block_pio (port, buf, length);
+       written = parport_pc_fifo_write_block(port, buf, length);
 
        /* Finish up. */
        /* For some hardware we don't want to touch the mode until
@@ -923,8 +934,10 @@ size_t parport_pc_ecp_write_block_pio (struct parport *port,
        return written;
 }
 
-size_t parport_pc_ecp_read_block_pio (struct parport *port,
-                                     void *buf, size_t length, int flags)
+#if 0
+static size_t parport_pc_ecp_read_block_pio (struct parport *port,
+                                            void *buf, size_t length,
+                                            int flags)
 {
        size_t left = length;
        size_t fifofull;
@@ -1142,7 +1155,7 @@ out_no_data:
 dump_parport_state ("fwd idle", port);
        return length - left;
 }
-
+#endif  /*  0  */
 #endif /* IEEE 1284 support */
 #endif /* Allowed to use FIFO/DMA */
 
@@ -1155,7 +1168,7 @@ dump_parport_state ("fwd idle", port);
 
 /* GCC is not inlining extern inline function later overwriten to non-inline,
    so we use outlined_ variants here.  */
-struct parport_operations parport_pc_ops = 
+static const struct parport_operations parport_pc_ops =
 {
        .write_data     = parport_pc_write_data,
        .read_data      = parport_pc_read_data,
@@ -1197,10 +1210,11 @@ struct parport_operations parport_pc_ops =
 static void __devinit show_parconfig_smsc37c669(int io, int key)
 {
        int cr1,cr4,cra,cr23,cr26,cr27,i=0;
-       static const char *modes[]={ "SPP and Bidirectional (PS/2)",    
-                                    "EPP and SPP",
-                                    "ECP",
-                                    "ECP and EPP" };
+       static const char *const modes[]={
+               "SPP and Bidirectional (PS/2)",
+               "EPP and SPP",
+               "ECP",
+               "ECP and EPP" };
 
        outb(key,io);
        outb(key,io);
@@ -1274,7 +1288,7 @@ static void __devinit show_parconfig_smsc37c669(int io, int key)
 static void __devinit show_parconfig_winbond(int io, int key)
 {
        int cr30,cr60,cr61,cr70,cr74,crf0,i=0;
-       static const char *modes[] = {
+       static const char *const modes[] = {
                "Standard (SPP) and Bidirectional(PS/2)", /* 0 */
                "EPP-1.9 and SPP",
                "ECP",
@@ -1283,7 +1297,9 @@ static void __devinit show_parconfig_winbond(int io, int key)
                "EPP-1.7 and SPP",              /* 5 */
                "undefined!",
                "ECP and EPP-1.7" };
-       static char *irqtypes[] = { "pulsed low, high-Z", "follows nACK" };
+       static char *const irqtypes[] = {
+               "pulsed low, high-Z",
+               "follows nACK" };
                
        /* The registers are called compatible-PnP because the
            register layout is modelled after ISA-PnP, the access
@@ -1403,6 +1419,9 @@ static void __devinit winbond_check(int io, int key)
 {
        int devid,devrev,oldid,x_devid,x_devrev,x_oldid;
 
+       if (!request_region(io, 3, __FUNCTION__))
+               return;
+
        /* First probe without key */
        outb(0x20,io);
        x_devid=inb(io+1);
@@ -1423,15 +1442,20 @@ static void __devinit winbond_check(int io, int key)
        outb(0xaa,io);    /* Magic Seal */
 
        if ((x_devid == devid) && (x_devrev == devrev) && (x_oldid == oldid))
-               return; /* protection against false positives */
+               goto out; /* protection against false positives */
 
        decode_winbond(io,key,devid,devrev,oldid);
+out:
+       release_region(io, 3);
 }
 
 static void __devinit winbond_check2(int io,int key)
 {
         int devid,devrev,oldid,x_devid,x_devrev,x_oldid;
 
+       if (!request_region(io, 3, __FUNCTION__))
+               return;
+
        /* First probe without the key */
        outb(0x20,io+2);
        x_devid=inb(io+2);
@@ -1451,15 +1475,20 @@ static void __devinit winbond_check2(int io,int key)
         outb(0xaa,io);    /* Magic Seal */
 
        if ((x_devid == devid) && (x_devrev == devrev) && (x_oldid == oldid))
-               return; /* protection against false positives */
+               goto out; /* protection against false positives */
 
-        decode_winbond(io,key,devid,devrev,oldid);
+       decode_winbond(io,key,devid,devrev,oldid);
+out:
+       release_region(io, 3);
 }
 
 static void __devinit smsc_check(int io, int key)
 {
         int id,rev,oldid,oldrev,x_id,x_rev,x_oldid,x_oldrev;
 
+       if (!request_region(io, 3, __FUNCTION__))
+               return;
+
        /* First probe without the key */
        outb(0x0d,io);
        x_oldid=inb(io+1);
@@ -1485,9 +1514,11 @@ static void __devinit smsc_check(int io, int key)
 
        if ((x_id == id) && (x_oldrev == oldrev) &&
            (x_oldid == oldid) && (x_rev == rev))
-               return; /* protection against false positives */
+               goto out; /* protection against false positives */
 
         decode_smsc(io,key,oldid,oldrev);
+out:
+       release_region(io, 3);
 }
 
 
@@ -1525,7 +1556,7 @@ static int __devinit get_superio_dma (struct parport *p)
        return PARPORT_DMA_NONE;
 }
 
-static int __devinit get_superio_irq (struct parport *p)
+static int get_superio_irq (struct parport *p)
 {
        int i=0;
         while( (superios[i].io != p->base) && (i<NR_SUPERIOS))
@@ -1547,7 +1578,7 @@ static int __devinit get_superio_irq (struct parport *p)
  *                         this shall always be the case!)
  *
  */
-static int __devinit parport_SPP_supported(struct parport *pb)
+static int parport_SPP_supported(struct parport *pb)
 {
        unsigned char r, w;
 
@@ -1628,7 +1659,7 @@ static int __devinit parport_SPP_supported(struct parport *pb)
  * two bits of ECR aren't writable, so we check by writing ECR and
  * reading it back to see if it's what we expect.
  */
-static int __devinit parport_ECR_present(struct parport *pb)
+static int parport_ECR_present(struct parport *pb)
 {
        struct parport_pc_private *priv = pb->private_data;
        unsigned char r = 0xc;
@@ -1680,7 +1711,7 @@ static int __devinit parport_ECR_present(struct parport *pb)
  * be misdetected here is rather academic. 
  */
 
-static int __devinit parport_PS2_supported(struct parport *pb)
+static int parport_PS2_supported(struct parport *pb)
 {
        int ok = 0;
   
@@ -1836,7 +1867,7 @@ static int __devinit parport_ECP_supported(struct parport *pb)
 }
 #endif
 
-static int __devinit parport_ECPPS2_supported(struct parport *pb)
+static int parport_ECPPS2_supported(struct parport *pb)
 {
        const struct parport_pc_private *priv = pb->private_data;
        int result;
@@ -1854,7 +1885,7 @@ static int __devinit parport_ECPPS2_supported(struct parport *pb)
 
 /* EPP mode detection  */
 
-static int __devinit parport_EPP_supported(struct parport *pb)
+static int parport_EPP_supported(struct parport *pb)
 {
        const struct parport_pc_private *priv = pb->private_data;
 
@@ -1899,7 +1930,7 @@ static int __devinit parport_EPP_supported(struct parport *pb)
        return 1;
 }
 
-static int __devinit parport_ECPEPP_supported(struct parport *pb)
+static int parport_ECPEPP_supported(struct parport *pb)
 {
        struct parport_pc_private *priv = pb->private_data;
        int result;
@@ -1944,7 +1975,7 @@ static int __devinit parport_ECPPS2_supported(struct parport *pb){return 0;}
 /* --- IRQ detection -------------------------------------- */
 
 /* Only if supports ECP mode */
-static int __devinit programmable_irq_support(struct parport *pb)
+static int programmable_irq_support(struct parport *pb)
 {
        int irq, intrLine;
        unsigned char oecr = inb (ECONTROL (pb));
@@ -1961,7 +1992,7 @@ static int __devinit programmable_irq_support(struct parport *pb)
        return irq;
 }
 
-static int __devinit irq_probe_ECP(struct parport *pb)
+static int irq_probe_ECP(struct parport *pb)
 {
        int i;
        unsigned long irqs;
@@ -1989,7 +2020,7 @@ static int __devinit irq_probe_ECP(struct parport *pb)
  * This detection seems that only works in National Semiconductors
  * This doesn't work in SMC, LGS, and Winbond 
  */
-static int __devinit irq_probe_EPP(struct parport *pb)
+static int irq_probe_EPP(struct parport *pb)
 {
 #ifndef ADVANCED_DETECT
        return PARPORT_IRQ_NONE;
@@ -2028,7 +2059,7 @@ static int __devinit irq_probe_EPP(struct parport *pb)
 #endif /* Advanced detection */
 }
 
-static int __devinit irq_probe_SPP(struct parport *pb)
+static int irq_probe_SPP(struct parport *pb)
 {
        /* Don't even try to do this. */
        return PARPORT_IRQ_NONE;
@@ -2041,7 +2072,7 @@ static int __devinit irq_probe_SPP(struct parport *pb)
  * When ECP is available we can autoprobe for IRQs.
  * NOTE: If we can autoprobe it, we can register the IRQ.
  */
-static int __devinit parport_irq_probe(struct parport *pb)
+static int parport_irq_probe(struct parport *pb)
 {
        struct parport_pc_private *priv = pb->private_data;
 
@@ -2110,7 +2141,7 @@ static int __devinit parport_dma_probe (struct parport *p)
 /* --- Initialisation code -------------------------------- */
 
 static LIST_HEAD(ports_list);
-static spinlock_t ports_lock = SPIN_LOCK_UNLOCKED;
+static DEFINE_SPINLOCK(ports_lock);
 
 struct parport *parport_pc_probe_port (unsigned long int base,
                                       unsigned long int base_hi,
@@ -2147,7 +2178,7 @@ struct parport *parport_pc_probe_port (unsigned long int base,
        priv->ctr_writable = ~0x10;
        priv->ecr = 0;
        priv->fifo_depth = 0;
-       priv->dma_buf = 0;
+       priv->dma_buf = NULL;
        priv->dma_handle = 0;
        priv->dev = dev;
        INIT_LIST_HEAD(&priv->list);
@@ -2265,6 +2296,7 @@ struct parport *parport_pc_probe_port (unsigned long int base,
                }
 
 #ifdef CONFIG_PARPORT_PC_FIFO
+#ifdef HAS_DMA
                if (p->dma != PARPORT_DMA_NONE) {
                        if (request_dma (p->dma, p->name)) {
                                printk (KERN_WARNING "%s: dma %d in use, "
@@ -2286,7 +2318,8 @@ struct parport *parport_pc_probe_port (unsigned long int base,
                                }
                        }
                }
-#endif /* CONFIG_PARPORT_PC_FIFO */
+#endif
+#endif
        }
 
        /* Done probing.  Now put the port into a sensible start-up state. */
@@ -2337,8 +2370,10 @@ void parport_pc_unregister_port (struct parport *p)
        spin_lock(&ports_lock);
        list_del_init(&priv->list);
        spin_unlock(&ports_lock);
+#if defined(CONFIG_PARPORT_PC_FIFO) && defined(HAS_DMA)
        if (p->dma != PARPORT_DMA_NONE)
                free_dma(p->dma);
+#endif
        if (p->irq != PARPORT_IRQ_NONE)
                free_irq(p->irq, p);
        release_region(p->base, 3);
@@ -2346,12 +2381,12 @@ void parport_pc_unregister_port (struct parport *p)
                release_region(p->base + 3, p->size - 3);
        if (p->modes & PARPORT_MODE_ECP)
                release_region(p->base_hi, 3);
-#ifdef CONFIG_PARPORT_PC_FIFO
+#if defined(CONFIG_PARPORT_PC_FIFO) && defined(HAS_DMA)
        if (priv->dma_buf)
                pci_free_consistent(priv->dev, PAGE_SIZE,
                                    priv->dma_buf,
                                    priv->dma_handle);
-#endif /* CONFIG_PARPORT_PC_FIFO */
+#endif
        kfree (p->private_data);
        parport_put_port(p);
        kfree (ops); /* hope no-one cached it */
@@ -2363,7 +2398,8 @@ EXPORT_SYMBOL (parport_pc_unregister_port);
 
 /* ITE support maintained by Rich Liu <richliu@poorman.org> */
 static int __devinit sio_ite_8872_probe (struct pci_dev *pdev, int autoirq,
-                                        int autodma)
+                                        int autodma,
+                                        const struct parport_pc_via_data *via)
 {
        short inta_addr[6] = { 0x2A0, 0x2C0, 0x220, 0x240, 0x1E0 };
        struct resource *base_res;
@@ -2466,71 +2502,162 @@ static int __devinit sio_ite_8872_probe (struct pci_dev *pdev, int autoirq,
        return 0;
 }
 
-/* Via support maintained by Jeff Garzik <jgarzik@pobox.com> */
-static int __devinit sio_via_686a_probe (struct pci_dev *pdev, int autoirq,
-                                        int autodma)
+/* VIA 8231 support by Pavel Fedin <sonic_amiga@rambler.ru>
+   based on VIA 686a support code by Jeff Garzik <jgarzik@pobox.com> */
+static int __devinitdata parport_init_mode = 0;
+
+/* Data for two known VIA chips */
+static struct parport_pc_via_data via_686a_data __devinitdata = {
+       0x51,
+       0x50,
+       0x85,
+       0x02,
+       0xE2,
+       0xF0,
+       0xE6
+};
+static struct parport_pc_via_data via_8231_data __devinitdata = {
+       0x45,
+       0x44,
+       0x50,
+       0x04,
+       0xF2,
+       0xFA,
+       0xF6
+};
+
+static int __devinit sio_via_probe (struct pci_dev *pdev, int autoirq,
+                                   int autodma,
+                                   const struct parport_pc_via_data *via)
 {
-       u8 tmp;
+       u8 tmp, tmp2, siofunc;
+       u8 ppcontrol = 0;
        int dma, irq;
-       unsigned port1, port2, have_eppecp;
+       unsigned port1, port2;
+       unsigned have_epp = 0;
 
+       printk(KERN_DEBUG "parport_pc: VIA 686A/8231 detected\n");
+
+       switch(parport_init_mode)
+       {
+       case 1:
+           printk(KERN_DEBUG "parport_pc: setting SPP mode\n");
+           siofunc = VIA_FUNCTION_PARPORT_SPP;
+           break;
+       case 2:
+           printk(KERN_DEBUG "parport_pc: setting PS/2 mode\n");
+           siofunc = VIA_FUNCTION_PARPORT_SPP;
+           ppcontrol = VIA_PARPORT_BIDIR;
+           break;
+       case 3:
+           printk(KERN_DEBUG "parport_pc: setting EPP mode\n");
+           siofunc = VIA_FUNCTION_PARPORT_EPP;
+           ppcontrol = VIA_PARPORT_BIDIR;
+           have_epp = 1;
+           break;
+       case 4:
+           printk(KERN_DEBUG "parport_pc: setting ECP mode\n");
+           siofunc = VIA_FUNCTION_PARPORT_ECP;
+           ppcontrol = VIA_PARPORT_BIDIR;
+           break;
+       case 5:
+           printk(KERN_DEBUG "parport_pc: setting EPP+ECP mode\n");
+           siofunc = VIA_FUNCTION_PARPORT_ECP;
+           ppcontrol = VIA_PARPORT_BIDIR|VIA_PARPORT_ECPEPP;
+           have_epp = 1;
+           break;
+        default:
+           printk(KERN_DEBUG "parport_pc: probing current configuration\n");
+           siofunc = VIA_FUNCTION_PROBE;
+           break;
+       }
        /*
-        * unlock super i/o configuration, set 0x85_1
-        */
-       pci_read_config_byte (pdev, 0x85, &tmp);
-       tmp |= (1 << 1);
-       pci_write_config_byte (pdev, 0x85, tmp);
-       
-       /* 
-        * Super I/O configuration, index port == 3f0h, data port == 3f1h
+        * unlock super i/o configuration
         */
+       pci_read_config_byte(pdev, via->via_pci_superio_config_reg, &tmp);
+       tmp |= via->via_pci_superio_config_data;
+       pci_write_config_byte(pdev, via->via_pci_superio_config_reg, tmp);
+
+       /* Bits 1-0: Parallel Port Mode / Enable */
+       outb(via->viacfg_function, VIA_CONFIG_INDEX);
+       tmp = inb (VIA_CONFIG_DATA);
+       /* Bit 5: EPP+ECP enable; bit 7: PS/2 bidirectional port enable */
+       outb(via->viacfg_parport_control, VIA_CONFIG_INDEX);
+       tmp2 = inb (VIA_CONFIG_DATA);
+       if (siofunc == VIA_FUNCTION_PROBE)
+       {
+           siofunc = tmp & VIA_FUNCTION_PARPORT_DISABLE;
+           ppcontrol = tmp2;
+       }
+       else
+       {
+           tmp &= ~VIA_FUNCTION_PARPORT_DISABLE;
+           tmp |= siofunc;
+           outb(via->viacfg_function, VIA_CONFIG_INDEX);
+           outb(tmp, VIA_CONFIG_DATA);
+           tmp2 &= ~(VIA_PARPORT_BIDIR|VIA_PARPORT_ECPEPP);
+           tmp2 |= ppcontrol;
+           outb(via->viacfg_parport_control, VIA_CONFIG_INDEX);
+           outb(tmp2, VIA_CONFIG_DATA);
+       }
        
-       /* 0xE2_1-0: Parallel Port Mode / Enable */
-       outb (0xE2, 0x3F0);
-       tmp = inb (0x3F1);
+       /* Parallel Port I/O Base Address, bits 9-2 */
+       outb(via->viacfg_parport_base, VIA_CONFIG_INDEX);
+       port1 = inb(VIA_CONFIG_DATA) << 2;
        
-       if ((tmp & 0x03) == 0x03) {
-               printk (KERN_INFO "parport_pc: Via 686A parallel port disabled in BIOS\n");
+       printk (KERN_DEBUG "parport_pc: Current parallel port base: 0x%X\n",port1);
+       if ((port1 == 0x3BC) && have_epp)
+       {
+           outb(via->viacfg_parport_base, VIA_CONFIG_INDEX);
+           outb((0x378 >> 2), VIA_CONFIG_DATA);
+           printk(KERN_DEBUG "parport_pc: Parallel port base changed to 0x378\n");
+           port1 = 0x378;
+       }
+
+       /*
+        * lock super i/o configuration
+        */
+       pci_read_config_byte(pdev, via->via_pci_superio_config_reg, &tmp);
+       tmp &= ~via->via_pci_superio_config_data;
+       pci_write_config_byte(pdev, via->via_pci_superio_config_reg, tmp);
+
+       if (siofunc == VIA_FUNCTION_PARPORT_DISABLE) {
+               printk(KERN_INFO "parport_pc: VIA parallel port disabled in BIOS\n");
                return 0;
        }
        
-       /* 0xE6: Parallel Port I/O Base Address, bits 9-2 */
-       outb (0xE6, 0x3F0);
-       port1 = inb (0x3F1) << 2;
-       
+       /* Bits 7-4: PnP Routing for Parallel Port IRQ */
+       pci_read_config_byte(pdev, via->via_pci_parport_irq_reg, &tmp);
+       irq = ((tmp & VIA_IRQCONTROL_PARALLEL) >> 4);
+
+       if (siofunc == VIA_FUNCTION_PARPORT_ECP)
+       {
+           /* Bits 3-2: PnP Routing for Parallel Port DMA */
+           pci_read_config_byte(pdev, via->via_pci_parport_dma_reg, &tmp);
+           dma = ((tmp & VIA_DMACONTROL_PARALLEL) >> 2);
+       }
+       else
+           /* if ECP not enabled, DMA is not enabled, assumed bogus 'dma' value */
+           dma = PARPORT_DMA_NONE;
+
+       /* Let the user (or defaults) steer us away from interrupts and DMA */
+       if (autoirq == PARPORT_IRQ_NONE) {
+           irq = PARPORT_IRQ_NONE;
+           dma = PARPORT_DMA_NONE;
+       }
+       if (autodma == PARPORT_DMA_NONE)
+           dma = PARPORT_DMA_NONE;
+
        switch (port1) {
        case 0x3bc: port2 = 0x7bc; break;
        case 0x378: port2 = 0x778; break;
        case 0x278: port2 = 0x678; break;
        default:
-               printk (KERN_INFO "parport_pc: Weird Via 686A parport base 0x%X, ignoring\n",
+               printk(KERN_INFO "parport_pc: Weird VIA parport base 0x%X, ignoring\n",
                        port1);
                return 0;
        }
 
-       /* 0xF0_5: EPP+ECP enable */
-       outb (0xF0, 0x3F0);
-       have_eppecp = (inb (0x3F1) & (1 << 5));
-       
-       /*
-        * lock super i/o configuration, clear 0x85_1
-        */
-       pci_read_config_byte (pdev, 0x85, &tmp);
-       tmp &= ~(1 << 1);
-       pci_write_config_byte (pdev, 0x85, tmp);
-
-       /*
-        * Get DMA and IRQ from PCI->ISA bridge PCI config registers
-        */
-
-       /* 0x50_3-2: PnP Routing for Parallel Port DRQ */
-       pci_read_config_byte (pdev, 0x50, &tmp);
-       dma = ((tmp >> 2) & 0x03);
-       
-       /* 0x51_7-4: PnP Routing for Parallel Port IRQ */
-       pci_read_config_byte (pdev, 0x51, &tmp);
-       irq = ((tmp >> 4) & 0x0F);
-
        /* filter bogus IRQs */
        switch (irq) {
        case 0:
@@ -2544,22 +2671,10 @@ static int __devinit sio_via_686a_probe (struct pci_dev *pdev, int autoirq,
                break;
        }
 
-       /* if ECP not enabled, DMA is not enabled, assumed bogus 'dma' value */
-       if (!have_eppecp)
-               dma = PARPORT_DMA_NONE;
-
-       /* Let the user (or defaults) steer us away from interrupts and DMA */
-       if (autoirq != PARPORT_IRQ_AUTO) {
-               irq = PARPORT_IRQ_NONE;
-               dma = PARPORT_DMA_NONE;
-       }
-       if (autodma != PARPORT_DMA_AUTO)
-               dma = PARPORT_DMA_NONE;
-
        /* finally, do the probe with values obtained */
        if (parport_pc_probe_port (port1, port2, irq, dma, NULL)) {
                printk (KERN_INFO
-                       "parport_pc: Via 686A parallel port: io=0x%X", port1);
+                       "parport_pc: VIA parallel port: io=0x%X", port1);
                if (irq != PARPORT_IRQ_NONE)
                        printk (", irq=%d", irq);
                if (dma != PARPORT_DMA_NONE)
@@ -2568,7 +2683,7 @@ static int __devinit sio_via_686a_probe (struct pci_dev *pdev, int autoirq,
                return 1;
        }
        
-       printk (KERN_WARNING "parport_pc: Strange, can't probe Via 686A parallel port: io=0x%X, irq=%d, dma=%d\n",
+       printk(KERN_WARNING "parport_pc: Strange, can't probe VIA parallel port: io=0x%X, irq=%d, dma=%d\n",
                port1, irq, dma);
        return 0;
 }
@@ -2576,19 +2691,22 @@ static int __devinit sio_via_686a_probe (struct pci_dev *pdev, int autoirq,
 
 enum parport_pc_sio_types {
        sio_via_686a = 0,       /* Via VT82C686A motherboard Super I/O */
+       sio_via_8231,           /* Via VT8231 south bridge integrated Super IO */
        sio_ite_8872,
        last_sio
 };
 
 /* each element directly indexed from enum list, above */
 static struct parport_pc_superio {
-       int (*probe) (struct pci_dev *pdev, int autoirq, int autodma);
+       int (*probe) (struct pci_dev *pdev, int autoirq, int autodma,
+                     const struct parport_pc_via_data *via);
+       const struct parport_pc_via_data *via;
 } parport_pc_superio_info[] __devinitdata = {
-       { sio_via_686a_probe, },
-       { sio_ite_8872_probe, },
+       { sio_via_probe, &via_686a_data, },
+       { sio_via_probe, &via_8231_data, },
+       { sio_ite_8872_probe, NULL, },
 };
 
-
 enum parport_pc_pci_cards {
        siig_1p_10x = last_sio,
        siig_2p_10x,
@@ -2625,13 +2743,20 @@ enum parport_pc_pci_cards {
        syba_2p_epp,
        syba_1p_ecp,
        titan_010l,
+       titan_1284p1,
        titan_1284p2,
        avlab_1p,
        avlab_2p,
+       oxsemi_952,
        oxsemi_954,
        oxsemi_840,
        aks_0100,
        mobility_pp,
+       netmos_9705,
+       netmos_9715,
+       netmos_9755,
+       netmos_9805,
+       netmos_9815,
 };
 
 
@@ -2654,7 +2779,7 @@ static struct parport_pc_pci {
        /* If set, this is called after probing for ports.  If 'failed'
         * is non-zero we couldn't use any of the ports. */
        void (*postinit_hook) (struct pci_dev *pdev, int failed);
-} cards[] __devinitdata = {
+} cards[] = {
        /* siig_1p_10x */               { 1, { { 2, 3 }, } },
        /* siig_2p_10x */               { 2, { { 2, 3 }, { 4, 5 }, } },
        /* siig_1p_20x */               { 1, { { 0, 1 }, } },
@@ -2692,20 +2817,28 @@ static struct parport_pc_pci {
        /* syba_2p_epp AP138B */        { 2, { { 0, 0x078 }, { 0, 0x178 }, } },
        /* syba_1p_ecp W83787 */        { 1, { { 0, 0x078 }, } },
        /* titan_010l */                { 1, { { 3, -1 }, } },
+       /* titan_1284p1 */              { 1, { { 0, 1 }, } },
        /* titan_1284p2 */              { 2, { { 0, 1 }, { 2, 3 }, } },
        /* avlab_1p             */      { 1, { { 0, 1}, } },
        /* avlab_2p             */      { 2, { { 0, 1}, { 2, 3 },} },
        /* The Oxford Semi cards are unusual: 954 doesn't support ECP,
         * and 840 locks up if you write 1 to bit 2! */
+       /* oxsemi_952 */                { 1, { { 0, 1 }, } },
        /* oxsemi_954 */                { 1, { { 0, -1 }, } },
        /* oxsemi_840 */                { 1, { { 0, -1 }, } },
-       /* aks_0100 */                  { 1, { { 0, 1 }, } },
+       /* aks_0100 */                  { 1, { { 0, -1 }, } },
        /* mobility_pp */               { 1, { { 0, 1 }, } },
+       /* netmos_9705 */               { 1, { { 0, -1 }, } }, /* untested */
+        /* netmos_9715 */               { 2, { { 0, 1 }, { 2, 3 },} }, /* untested */
+        /* netmos_9755 */               { 2, { { 0, 1 }, { 2, 3 },} }, /* untested */
+       /* netmos_9805 */               { 1, { { 0, -1 }, } }, /* untested */
+       /* netmos_9815 */               { 2, { { 0, -1 }, { 2, -1 }, } }, /* untested */
 };
 
-static struct pci_device_id parport_pc_pci_tbl[] = {
+static const struct pci_device_id parport_pc_pci_tbl[] = {
        /* Super-IO onboard chips */
        { 0x1106, 0x0686, PCI_ANY_ID, PCI_ANY_ID, 0, 0, sio_via_686a },
+       { 0x1106, 0x8231, PCI_ANY_ID, PCI_ANY_ID, 0, 0, sio_via_8231 },
        { PCI_VENDOR_ID_ITE, PCI_DEVICE_ID_ITE_8872,
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, sio_ite_8872 },
 
@@ -2759,24 +2892,44 @@ static struct pci_device_id parport_pc_pci_tbl[] = {
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, syba_1p_ecp },
        { PCI_VENDOR_ID_TITAN, PCI_DEVICE_ID_TITAN_010L,
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, titan_010l },
+       { 0x9710, 0x9805, 0x1000, 0x0010, 0, 0, titan_1284p1 },
        { 0x9710, 0x9815, 0x1000, 0x0020, 0, 0, titan_1284p2 },
        /* PCI_VENDOR_ID_AVLAB/Intek21 has another bunch of cards ...*/
        { 0x14db, 0x2120, PCI_ANY_ID, PCI_ANY_ID, 0, 0, avlab_1p}, /* AFAVLAB_TK9902 */
        { 0x14db, 0x2121, PCI_ANY_ID, PCI_ANY_ID, 0, 0, avlab_2p},
+       { PCI_VENDOR_ID_OXSEMI, PCI_DEVICE_ID_OXSEMI_16PCI952PP,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, oxsemi_952 },
        { PCI_VENDOR_ID_OXSEMI, PCI_DEVICE_ID_OXSEMI_16PCI954PP,
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, oxsemi_954 },
        { PCI_VENDOR_ID_OXSEMI, PCI_DEVICE_ID_OXSEMI_12PCI840,
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, oxsemi_840 },
        { PCI_VENDOR_ID_AKS, PCI_DEVICE_ID_AKS_ALADDINCARD,
          PCI_ANY_ID, PCI_ANY_ID, 0, 0, aks_0100 },
+       /* NetMos communication controllers */
+       { PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9705,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9705 },
+       { PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9715,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9715 },
+       { PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9755,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9755 },
+       { PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9805,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9805 },
+       { PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9815,
+         PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9815 },
        { 0, } /* terminate list */
 };
 MODULE_DEVICE_TABLE(pci,parport_pc_pci_tbl);
 
+struct pci_parport_data {
+       int num;
+       struct parport *ports[2];
+};
+
 static int parport_pc_pci_probe (struct pci_dev *dev,
                                           const struct pci_device_id *id)
 {
        int err, count, n, i = id->driver_data;
+       struct pci_parport_data *data;
 
        if (i < last_sio)
                /* This is an onboard Super-IO and has already been probed */
@@ -2788,9 +2941,15 @@ static int parport_pc_pci_probe (struct pci_dev *dev,
        if ((err = pci_enable_device (dev)) != 0)
                return err;
 
+       data = kmalloc(sizeof(struct pci_parport_data), GFP_KERNEL);
+       if (!data)
+               return -ENOMEM;
+
        if (cards[i].preinit_hook &&
-           cards[i].preinit_hook (dev, PARPORT_IRQ_NONE, PARPORT_DMA_NONE))
+           cards[i].preinit_hook (dev, PARPORT_IRQ_NONE, PARPORT_DMA_NONE)) {
+               kfree(data);
                return -ENODEV;
+       }
 
        for (n = 0; n < cards[i].numports; n++) {
                int lo = cards[i].addr[n].lo;
@@ -2809,21 +2968,48 @@ static int parport_pc_pci_probe (struct pci_dev *dev,
                        "I/O at %#lx(%#lx)\n",
                        parport_pc_pci_tbl[i + last_sio].vendor,
                        parport_pc_pci_tbl[i + last_sio].device, io_lo, io_hi);
-               if (parport_pc_probe_port (io_lo, io_hi, PARPORT_IRQ_NONE,
-                                          PARPORT_DMA_NONE, dev))
+               data->ports[count] =
+                       parport_pc_probe_port (io_lo, io_hi, PARPORT_IRQ_NONE,
+                                              PARPORT_DMA_NONE, dev);
+               if (data->ports[count])
                        count++;
        }
 
+       data->num = count;
+
        if (cards[i].postinit_hook)
                cards[i].postinit_hook (dev, count == 0);
 
-       return count == 0 ? -ENODEV : 0;
+       if (count) {
+               pci_set_drvdata(dev, data);
+               return 0;
+       }
+
+       kfree(data);
+
+       return -ENODEV;
+}
+
+static void __devexit parport_pc_pci_remove(struct pci_dev *dev)
+{
+       struct pci_parport_data *data = pci_get_drvdata(dev);
+       int i;
+
+       pci_set_drvdata(dev, NULL);
+
+       if (data) {
+               for (i = data->num - 1; i >= 0; i--)
+                       parport_pc_unregister_port(data->ports[i]);
+
+               kfree(data);
+       }
 }
 
 static struct pci_driver parport_pc_pci_driver = {
        .name           = "parport_pc",
        .id_table       = parport_pc_pci_tbl,
        .probe          = parport_pc_pci_probe,
+       .remove         = __devexit_p(parport_pc_pci_remove),
 };
 
 static int __init parport_pc_init_superio (int autoirq, int autodma)
@@ -2832,13 +3018,13 @@ static int __init parport_pc_init_superio (int autoirq, int autodma)
        struct pci_dev *pdev = NULL;
        int ret = 0;
 
-       while ((pdev = pci_find_device(PCI_ANY_ID, PCI_ANY_ID, pdev)) != NULL) {
-               id = pci_match_device (parport_pc_pci_tbl, pdev);
+       for_each_pci_dev(pdev) {
+               id = pci_match_id(parport_pc_pci_tbl, pdev);
                if (id == NULL || id->driver_data >= last_sio)
                        continue;
 
                if (parport_pc_superio_info[id->driver_data].probe
-                       (pdev, autoirq, autodma)) {
+                       (pdev, autoirq, autodma,parport_pc_superio_info[id->driver_data].via)) {
                        ret++;
                }
        }
@@ -2918,7 +3104,7 @@ static struct pnp_driver parport_pc_pnp_driver = {
 
 
 /* This is called by parport_pc_find_nonpci_ports (in asm/parport.h) */
-static int __init __attribute__((unused))
+static int __devinit __attribute__((unused))
 parport_pc_find_isa_ports (int autoirq, int autodma)
 {
        int count = 0;
@@ -2943,9 +3129,9 @@ parport_pc_find_isa_ports (int autoirq, int autodma)
  * autoirq is PARPORT_IRQ_NONE, PARPORT_IRQ_AUTO, or PARPORT_IRQ_PROBEONLY
  * autodma is PARPORT_DMA_NONE or PARPORT_DMA_AUTO
  */
-static int __init parport_pc_find_ports (int autoirq, int autodma)
+static void __init parport_pc_find_ports (int autoirq, int autodma)
 {
-       int count = 0, r;
+       int count = 0, err;
 
 #ifdef CONFIG_PARPORT_PC_SUPERIO
        detect_and_report_winbond ();
@@ -2957,23 +3143,17 @@ static int __init parport_pc_find_ports (int autoirq, int autodma)
 
        /* PnP ports, skip detection if SuperIO already found them */
        if (!count) {
-               r = pnp_register_driver (&parport_pc_pnp_driver);
-               if (r >= 0) {
+               err = pnp_register_driver (&parport_pc_pnp_driver);
+               if (!err)
                        pnp_registered_parport = 1;
-                       count += r;
-               }
        }
 
        /* ISA ports and whatever (see asm/parport.h). */
-       count += parport_pc_find_nonpci_ports (autoirq, autodma);
+       parport_pc_find_nonpci_ports (autoirq, autodma);
 
-       r = pci_register_driver (&parport_pc_pci_driver);
-       if (r >= 0) {
+       err = pci_register_driver (&parport_pc_pci_driver);
+       if (!err)
                pci_registered_parport = 1;
-               count += r;
-       }
-
-       return count;
 }
 
 /*
@@ -3024,22 +3204,46 @@ static int __init parport_parse_dma(const char *dmastr, int *val)
                                     PARPORT_DMA_NONE, PARPORT_DMA_NOFIFO);
 }
 
+#ifdef CONFIG_PCI
+static int __init parport_init_mode_setup(char *str)
+{
+       printk(KERN_DEBUG "parport_pc.c: Specified parameter parport_init_mode=%s\n", str);
+
+       if (!strcmp (str, "spp"))
+               parport_init_mode=1;
+       if (!strcmp (str, "ps2"))
+               parport_init_mode=2;
+       if (!strcmp (str, "epp"))
+               parport_init_mode=3;
+       if (!strcmp (str, "ecp"))
+               parport_init_mode=4;
+       if (!strcmp (str, "ecpepp"))
+               parport_init_mode=5;
+       return 1;
+}
+#endif
+
 #ifdef MODULE
 static const char *irq[PARPORT_PC_MAX_PORTS];
 static const char *dma[PARPORT_PC_MAX_PORTS];
 
 MODULE_PARM_DESC(io, "Base I/O address (SPP regs)");
-MODULE_PARM(io, "1-" __MODULE_STRING(PARPORT_PC_MAX_PORTS) "i");
+module_param_array(io, int, NULL, 0);
 MODULE_PARM_DESC(io_hi, "Base I/O address (ECR)");
-MODULE_PARM(io_hi, "1-" __MODULE_STRING(PARPORT_PC_MAX_PORTS) "i");
+module_param_array(io_hi, int, NULL, 0);
 MODULE_PARM_DESC(irq, "IRQ line");
-MODULE_PARM(irq, "1-" __MODULE_STRING(PARPORT_PC_MAX_PORTS) "s");
+module_param_array(irq, charp, NULL, 0);
 MODULE_PARM_DESC(dma, "DMA channel");
-MODULE_PARM(dma, "1-" __MODULE_STRING(PARPORT_PC_MAX_PORTS) "s");
+module_param_array(dma, charp, NULL, 0);
 #if defined(CONFIG_PARPORT_PC_SUPERIO) || \
        (defined(CONFIG_PARPORT_1284) && defined(CONFIG_PARPORT_PC_FIFO))
 MODULE_PARM_DESC(verbose_probing, "Log chit-chat during initialisation");
-MODULE_PARM(verbose_probing, "i");
+module_param(verbose_probing, int, 0644);
+#endif
+#ifdef CONFIG_PCI
+static char *init_mode;
+MODULE_PARM_DESC(init_mode, "Initialise mode for VIA VT8231 port (spp, ps2, epp, ecp or ecpepp)");
+module_param(init_mode, charp, 0);
 #endif
 
 static int __init parse_parport_params(void)
@@ -3047,6 +3251,11 @@ static int __init parse_parport_params(void)
        unsigned int i;
        int val;
 
+#ifdef CONFIG_PCI
+       if (init_mode)
+               parport_init_mode_setup(init_mode);
+#endif
+
        for (i = 0; i < PARPORT_PC_MAX_PORTS && io[i]; i++) {
                if (parport_parse_irq(irq[i], &val))
                        return 1;
@@ -3154,14 +3363,21 @@ static int __init parse_parport_params(void)
 }
 
 __setup ("parport=", parport_setup);
+
+/*
+ * Acceptable parameters:
+ *
+ * parport_init_mode=[spp|ps2|epp|ecp|ecpepp]
+ */
+#ifdef CONFIG_PCI
+__setup("parport_init_mode=",parport_init_mode_setup);
+#endif
 #endif
 
 /* "Parser" ends here */
 
 static int __init parport_pc_init(void)
 {
-       int count = 0;
-
        if (parse_parport_params())
                return -EINVAL;
 
@@ -3174,12 +3390,11 @@ static int __init parport_pc_init(void)
                                break;
                        if ((io_hi[i]) == PARPORT_IOHI_AUTO)
                               io_hi[i] = 0x400 + io[i];
-                       if (parport_pc_probe_port(io[i], io_hi[i],
-                                                 irqval[i], dmaval[i], NULL))
-                               count++;
+                       parport_pc_probe_port(io[i], io_hi[i],
+                                                 irqval[i], dmaval[i], NULL);
                }
        } else
-               count += parport_pc_find_ports (irqval[0], dmaval[0]);
+               parport_pc_find_ports (irqval[0], dmaval[0]);
 
        return 0;
 }