[PARISC] Misc. janitorial work
[pandora-kernel.git] / drivers / parisc / superio.c
index d14888e..828eb45 100644 (file)
@@ -12,6 +12,7 @@
  *      (C) Copyright 2001 John Marvin <jsm fc hp com>
  *      (C) Copyright 2003 Grant Grundler <grundler parisc-linux org>
  *     (C) Copyright 2005 Kyle McMartin <kyle@parisc-linux.org>
+ *     (C) Copyright 2006 Helge Deller <deller@gmx.de>
  *
  *     This program is free software; you can redistribute it and/or
  *     modify it under the terms of the GNU General Public License as
@@ -89,6 +90,9 @@ static struct superio_device sio_dev;
 #define DBG_INIT(x...)
 #endif
 
+#define SUPERIO        "SuperIO"
+#define PFX    SUPERIO ": "
+
 static irqreturn_t
 superio_interrupt(int parent_irq, void *devp, struct pt_regs *regs)
 {
@@ -117,7 +121,7 @@ superio_interrupt(int parent_irq, void *devp, struct pt_regs *regs)
        local_irq = results & 0x0f;
 
        if (local_irq == 2 || local_irq > 7) {
-               printk(KERN_ERR "SuperIO: slave interrupted!\n");
+               printk(KERN_ERR PFX "slave interrupted!\n");
                return IRQ_HANDLED;
        }
 
@@ -128,7 +132,7 @@ superio_interrupt(int parent_irq, void *devp, struct pt_regs *regs)
                outb(OCW3_ISR,IC_PIC1+0);
                results = inb(IC_PIC1+0);
                if ((results & 0x80) == 0) { /* if ISR7 not set: spurious */
-                       printk(KERN_WARNING "SuperIO: spurious interrupt!\n");
+                       printk(KERN_WARNING PFX "spurious interrupt!\n");
                        return IRQ_HANDLED;
                }
        }
@@ -151,11 +155,11 @@ superio_init(struct pci_dev *pcidev)
        struct pci_dev *pdev = sio->lio_pdev;
        u16 word;
 
-        if (sio->suckyio_irq_enabled)                                       
+       if (sio->suckyio_irq_enabled)
                return;
 
-       if (!pdev) BUG();
-       if (!sio->usb_pdev) BUG();
+       BUG_ON(!pdev);
+       BUG_ON(!sio->usb_pdev);
 
        /* use the IRQ iosapic found for USB INT D... */
        pdev->irq = sio->usb_pdev->irq;
@@ -163,34 +167,34 @@ superio_init(struct pci_dev *pcidev)
        /* ...then properly fixup the USB to point at suckyio PIC */
        sio->usb_pdev->irq = superio_fixup_irq(sio->usb_pdev);
 
-       printk(KERN_INFO "SuperIO: Found NS87560 Legacy I/O device at %s (IRQ %i) \n",
+       printk(KERN_INFO PFX "Found NS87560 Legacy I/O device at %s (IRQ %i) \n",
               pci_name(pdev), pdev->irq);
 
        pci_read_config_dword (pdev, SIO_SP1BAR, &sio->sp1_base);
        sio->sp1_base &= ~1;
-       printk (KERN_INFO "SuperIO: Serial port 1 at 0x%x\n", sio->sp1_base);
+       printk(KERN_INFO PFX "Serial port 1 at 0x%x\n", sio->sp1_base);
 
        pci_read_config_dword (pdev, SIO_SP2BAR, &sio->sp2_base);
        sio->sp2_base &= ~1;
-       printk (KERN_INFO "SuperIO: Serial port 2 at 0x%x\n", sio->sp2_base);
+       printk(KERN_INFO PFX "Serial port 2 at 0x%x\n", sio->sp2_base);
 
        pci_read_config_dword (pdev, SIO_PPBAR, &sio->pp_base);
        sio->pp_base &= ~1;
-       printk (KERN_INFO "SuperIO: Parallel port at 0x%x\n", sio->pp_base);
+       printk(KERN_INFO PFX "Parallel port at 0x%x\n", sio->pp_base);
 
        pci_read_config_dword (pdev, SIO_FDCBAR, &sio->fdc_base);
        sio->fdc_base &= ~1;
-       printk (KERN_INFO "SuperIO: Floppy controller at 0x%x\n", sio->fdc_base);
+       printk(KERN_INFO PFX "Floppy controller at 0x%x\n", sio->fdc_base);
        pci_read_config_dword (pdev, SIO_ACPIBAR, &sio->acpi_base);
        sio->acpi_base &= ~1;
-       printk (KERN_INFO "SuperIO: ACPI at 0x%x\n", sio->acpi_base);
+       printk(KERN_INFO PFX "ACPI at 0x%x\n", sio->acpi_base);
 
        request_region (IC_PIC1, 0x1f, "pic1");
        request_region (IC_PIC2, 0x1f, "pic2");
        request_region (sio->acpi_base, 0x1f, "acpi");
 
        /* Enable the legacy I/O function */
-        pci_read_config_word (pdev, PCI_COMMAND, &word);
+       pci_read_config_word (pdev, PCI_COMMAND, &word);
        word |= PCI_COMMAND_SERR | PCI_COMMAND_PARITY | PCI_COMMAND_IO;
        pci_write_config_word (pdev, PCI_COMMAND, word);
 
@@ -263,14 +267,14 @@ superio_init(struct pci_dev *pcidev)
        /* Setup USB power regulation */
        outb(1, sio->acpi_base + USB_REG_CR);
        if (inb(sio->acpi_base + USB_REG_CR) & 1)
-               printk(KERN_INFO "SuperIO: USB regulator enabled\n");
+               printk(KERN_INFO PFX "USB regulator enabled\n");
        else
-               printk(KERN_ERR "USB regulator not initialized!\n");
+               printk(KERN_ERR PFX "USB regulator not initialized!\n");
 
        if (request_irq(pdev->irq, superio_interrupt, SA_INTERRUPT,
-                       "SuperIO", (void *)sio)) {
+                       SUPERIO, (void *)sio)) {
 
-               printk(KERN_ERR "SuperIO: could not get irq\n");
+               printk(KERN_ERR PFX "could not get irq\n");
                BUG();
                return;
        }
@@ -284,7 +288,7 @@ static void superio_disable_irq(unsigned int irq)
        u8 r8;
 
        if ((irq < 1) || (irq == 2) || (irq > 7)) {
-               printk(KERN_ERR "SuperIO: Illegal irq number.\n");
+               printk(KERN_ERR PFX "Illegal irq number.\n");
                BUG();
                return;
        }
@@ -301,7 +305,7 @@ static void superio_enable_irq(unsigned int irq)
        u8 r8;
 
        if ((irq < 1) || (irq == 2) || (irq > 7)) {
-               printk(KERN_ERR "SuperIO: Illegal irq number (%d).\n", irq);
+               printk(KERN_ERR PFX "Illegal irq number (%d).\n", irq);
                BUG();
                return;
        }
@@ -319,7 +323,7 @@ static unsigned int superio_startup_irq(unsigned int irq)
 }
 
 static struct hw_interrupt_type superio_interrupt_type = {
-       .typename =     "SuperIO",
+       .typename =     SUPERIO,
        .startup =      superio_startup_irq,
        .shutdown =     superio_disable_irq,
        .enable =       superio_enable_irq,
@@ -385,45 +389,36 @@ int superio_fixup_irq(struct pci_dev *pcidev)
        return local_irq;
 }
 
-static struct uart_port serial[] = {
-       {
-               .iotype         = UPIO_PORT,
-               .line           = 0,
-               .type           = PORT_16550A,
-               .uartclk        = 115200*16,
-               .fifosize       = 16,
-       },
-       {
-               .iotype         = UPIO_PORT,
-               .line           = 1,
-               .type           = PORT_16550A,
-               .uartclk        = 115200*16,
-               .fifosize       = 16,
-       }
-};
-
 static void __devinit superio_serial_init(void)
 {
 #ifdef CONFIG_SERIAL_8250
        int retval;
-        
-       serial[0].iobase = sio_dev.sp1_base;
-       serial[0].irq = SP1_IRQ;
-       spin_lock_init(&serial[0].lock);
-
-       retval = early_serial_setup(&serial[0]);
+       struct uart_port serial_port;
+
+       memset(&serial_port, 0, sizeof(serial_port));
+       serial_port.iotype      = UPIO_PORT;
+       serial_port.type        = PORT_16550A;
+       serial_port.uartclk     = 115200*16;
+       serial_port.fifosize    = 16;
+       spin_lock_init(&serial_port.lock);
+
+       /* serial port #1 */
+       serial_port.iobase      = sio_dev.sp1_base;
+       serial_port.irq         = SP1_IRQ;
+       serial_port.line        = 0;
+       retval = early_serial_setup(&serial_port);
        if (retval < 0) {
-               printk(KERN_WARNING "SuperIO: Register Serial #0 failed.\n");
+               printk(KERN_WARNING PFX "Register Serial #0 failed.\n");
                return;
        }
 
-       serial[1].iobase = sio_dev.sp2_base;
-       serial[1].irq = SP2_IRQ;
-       spin_lock_init(&serial[1].lock);
-       retval = early_serial_setup(&serial[1]);
-
+       /* serial port #2 */
+       serial_port.iobase      = sio_dev.sp2_base;
+       serial_port.irq         = SP2_IRQ;
+       serial_port.line        = 1;
+       retval = early_serial_setup(&serial_port);
        if (retval < 0)
-               printk(KERN_WARNING "SuperIO: Register Serial #1 failed.\n");
+               printk(KERN_WARNING PFX "Register Serial #1 failed.\n");
 #endif /* CONFIG_SERIAL_8250 */
 }
 
@@ -437,7 +432,7 @@ static void __devinit superio_parport_init(void)
                        PARPORT_DMA_NONE /* dma */,
                        NULL /*struct pci_dev* */) )
 
-               printk(KERN_WARNING "SuperIO: Probing parallel port failed.\n");
+               printk(KERN_WARNING PFX "Probing parallel port failed.\n");
 #endif /* CONFIG_PARPORT_PC */
 }
 
@@ -471,8 +466,7 @@ superio_probe(struct pci_dev *dev, const struct pci_device_id *id)
                dev->subsystem_vendor, dev->subsystem_device,
                dev->class);
 
-       if (!sio->suckyio_irq_enabled)
-               BUG(); /* Enabled by PCI_FIXUP_FINAL */
+       BUG_ON(!sio->suckyio_irq_enabled);      /* Enabled by PCI_FIXUP_FINAL */
 
        if (dev->device == PCI_DEVICE_ID_NS_87560_LIO) {        /* Function 1 */
                superio_parport_init();
@@ -499,7 +493,7 @@ static struct pci_device_id superio_tbl[] = {
 };
 
 static struct pci_driver superio_driver = {
-       .name =         "SuperIO",
+       .name =         SUPERIO,
        .id_table =     superio_tbl,
        .probe =        superio_probe,
 };