Staging: comedi: gsc_hpdi: style fixes printk
authorDirk Hohndel <hohndel@infradead.org>
Wed, 21 Oct 2009 08:05:42 +0000 (01:05 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Dec 2009 20:23:02 +0000 (12:23 -0800)
Style cleanup in staging based on Greg's tutorial / checkpatch
Add printk KERN_ facility level (KERN_WARNING seemed appropriate for all)

Signed-off-by: Dirk Hohndel <hohndel@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/comedi/drivers/gsc_hpdi.c

index 0bb30162e92cae70f6355b9953275a4308563a78..396fa89a5a6e2c0779d041cc857e7a355c690043 100644 (file)
@@ -110,7 +110,8 @@ enum hpdi_registers {
 int command_channel_valid(unsigned int channel)
 {
        if (channel == 0 || channel > 6) {
-               printk("gsc_hpdi: bug! invalid cable command channel\n");
+               printk(KERN_WARNING
+                      "gsc_hpdi: bug! invalid cable command channel\n");
                return 0;
        }
        return 1;
@@ -560,7 +561,7 @@ static int hpdi_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        int i;
        int retval;
 
-       printk("comedi%d: gsc_hpdi\n", dev->minor);
+       printk(KERN_WARNING "comedi%d: gsc_hpdi\n", dev->minor);
 
        if (alloc_private(dev, sizeof(struct hpdi_private)) < 0)
                return -ENOMEM;
@@ -588,11 +589,12 @@ static int hpdi_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                } while (pcidev != NULL);
        }
        if (dev->board_ptr == NULL) {
-               printk("gsc_hpdi: no hpdi card found\n");
+               printk(KERN_WARNING "gsc_hpdi: no hpdi card found\n");
                return -EIO;
        }
 
-       printk("gsc_hpdi: found %s on bus %i, slot %i\n", board(dev)->name,
+       printk(KERN_WARNING
+              "gsc_hpdi: found %s on bus %i, slot %i\n", board(dev)->name,
               pcidev->bus->number, PCI_SLOT(pcidev->devfn));
 
        if (comedi_pci_enable(pcidev, driver_hpdi.driver_name)) {
@@ -618,7 +620,7 @@ static int hpdi_attach(struct comedi_device *dev, struct comedi_devconfig *it)
            ioremap(priv(dev)->hpdi_phys_iobase,
                    pci_resource_len(pcidev, HPDI_BADDRINDEX));
        if (!priv(dev)->plx9080_iobase || !priv(dev)->hpdi_iobase) {
-               printk(" failed to remap io memory\n");
+               printk(KERN_WARNING " failed to remap io memory\n");
                return -ENOMEM;
        }
 
@@ -630,12 +632,13 @@ static int hpdi_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        /*  get irq */
        if (request_irq(pcidev->irq, handle_interrupt, IRQF_SHARED,
                        driver_hpdi.driver_name, dev)) {
-               printk(" unable to allocate irq %u\n", pcidev->irq);
+               printk(KERN_WARNING
+                      " unable to allocate irq %u\n", pcidev->irq);
                return -EINVAL;
        }
        dev->irq = pcidev->irq;
 
-       printk(" irq %u\n", dev->irq);
+       printk(KERN_WARNING " irq %u\n", dev->irq);
 
        /*  alocate pci dma buffers */
        for (i = 0; i < NUM_DMA_BUFFERS; i++) {
@@ -653,7 +656,8 @@ static int hpdi_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                                                   &priv(dev)->
                                                   dma_desc_phys_addr);
        if (priv(dev)->dma_desc_phys_addr & 0xf) {
-               printk(" dma descriptors not quad-word aligned (bug)\n");
+               printk(KERN_WARNING
+                      " dma descriptors not quad-word aligned (bug)\n");
                return -EIO;
        }
 
@@ -672,7 +676,7 @@ static int hpdi_detach(struct comedi_device *dev)
 {
        unsigned int i;
 
-       printk("comedi%d: gsc_hpdi: remove\n", dev->minor);
+       printk(KERN_WARNING "comedi%d: gsc_hpdi: remove\n", dev->minor);
 
        if (dev->irq)
                free_irq(dev->irq, dev);