Staging: comedi: fix comedi_parport.c checkpatch.pl issues.
authorGreg Kroah-Hartman <gregkh@suse.de>
Wed, 19 Nov 2008 18:34:00 +0000 (10:34 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 6 Jan 2009 21:52:25 +0000 (13:52 -0800)
This resolves all the resolvable checkpatch.pl issues in the
comedi_parport.c file.

Cc: David Schleef <ds@schleef.org>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/comedi/drivers/comedi_parport.c

index 9530c47806cbbd8058d0dd158721250240bd0912..ba838fffa29ec46b1660d2ec0d6e9313d63dfa36 100644 (file)
@@ -90,26 +90,26 @@ pin, which can be used to wake up tasks.
 #define PARPORT_B 1
 #define PARPORT_C 2
 
-static int parport_attach(comedi_device * dev, comedi_devconfig * it);
-static int parport_detach(comedi_device * dev);
+static int parport_attach(comedi_device *dev, comedi_devconfig *it);
+static int parport_detach(comedi_device *dev);
 static comedi_driver driver_parport = {
-      driver_name:"comedi_parport",
-      module:THIS_MODULE,
-      attach:parport_attach,
-      detach:parport_detach,
+      .driver_name =   "comedi_parport",
+      .module =                THIS_MODULE,
+      .attach =                parport_attach,
+      .detach =                parport_detach,
 };
 
 COMEDI_INITCLEANUP(driver_parport);
 
-typedef struct parport_private_struct {
+struct parport_private {
        unsigned int a_data;
        unsigned int c_data;
        int enable_irq;
-} parport_private;
-#define devpriv ((parport_private *)(dev->private))
+};
+#define devpriv ((struct parport_private *)(dev->private))
 
-static int parport_insn_a(comedi_device * dev, comedi_subdevice * s,
-       comedi_insn * insn, lsampl_t * data)
+static int parport_insn_a(comedi_device *dev, comedi_subdevice *s,
+                         comedi_insn *insn, lsampl_t *data)
 {
        if (data[0]) {
                devpriv->a_data &= ~data[0];
@@ -123,8 +123,8 @@ static int parport_insn_a(comedi_device * dev, comedi_subdevice * s,
        return 2;
 }
 
-static int parport_insn_config_a(comedi_device * dev, comedi_subdevice * s,
-       comedi_insn * insn, lsampl_t * data)
+static int parport_insn_config_a(comedi_device *dev, comedi_subdevice *s,
+                                comedi_insn *insn, lsampl_t *data)
 {
        if (data[0]) {
                s->io_bits = 0xff;
@@ -138,11 +138,12 @@ static int parport_insn_config_a(comedi_device * dev, comedi_subdevice * s,
        return 1;
 }
 
-static int parport_insn_b(comedi_device * dev, comedi_subdevice * s,
-       comedi_insn * insn, lsampl_t * data)
+static int parport_insn_b(comedi_device *dev, comedi_subdevice *s,
+                         comedi_insn *insn, lsampl_t *data)
 {
        if (data[0]) {
-               // should writes be ignored?
+               /* should writes be ignored? */
+               /* anyone??? */
        }
 
        data[1] = (inb(dev->iobase + PARPORT_B) >> 3);
@@ -150,8 +151,8 @@ static int parport_insn_b(comedi_device * dev, comedi_subdevice * s,
        return 2;
 }
 
-static int parport_insn_c(comedi_device * dev, comedi_subdevice * s,
-       comedi_insn * insn, lsampl_t * data)
+static int parport_insn_c(comedi_device *dev, comedi_subdevice *s,
+                         comedi_insn *insn, lsampl_t *data)
 {
        data[0] &= 0x0f;
        if (data[0]) {
@@ -166,8 +167,8 @@ static int parport_insn_c(comedi_device * dev, comedi_subdevice * s,
        return 2;
 }
 
-static int parport_intr_insn(comedi_device * dev, comedi_subdevice * s,
-       comedi_insn * insn, lsampl_t * data)
+static int parport_intr_insn(comedi_device *dev, comedi_subdevice *s,
+                            comedi_insn *insn, lsampl_t *data)
 {
        if (insn->n < 1)
                return -EINVAL;
@@ -176,8 +177,8 @@ static int parport_intr_insn(comedi_device * dev, comedi_subdevice * s,
        return 2;
 }
 
-static int parport_intr_cmdtest(comedi_device * dev, comedi_subdevice * s,
-       comedi_cmd * cmd)
+static int parport_intr_cmdtest(comedi_device *dev, comedi_subdevice *s,
+                               comedi_cmd *cmd)
 {
        int err = 0;
        int tmp;
@@ -251,7 +252,7 @@ static int parport_intr_cmdtest(comedi_device * dev, comedi_subdevice * s,
        return 0;
 }
 
-static int parport_intr_cmd(comedi_device * dev, comedi_subdevice * s)
+static int parport_intr_cmd(comedi_device *dev, comedi_subdevice *s)
 {
        devpriv->c_data |= 0x10;
        outb(devpriv->c_data, dev->iobase + PARPORT_C);
@@ -261,9 +262,9 @@ static int parport_intr_cmd(comedi_device * dev, comedi_subdevice * s)
        return 0;
 }
 
-static int parport_intr_cancel(comedi_device * dev, comedi_subdevice * s)
+static int parport_intr_cancel(comedi_device *dev, comedi_subdevice *s)
 {
-       printk("parport_intr_cancel()\n");
+       printk(KERN_DEBUG "parport_intr_cancel()\n");
 
        devpriv->c_data &= ~0x10;
        outb(devpriv->c_data, dev->iobase + PARPORT_C);
@@ -279,7 +280,7 @@ static irqreturn_t parport_interrupt(int irq, void *d PT_REGS_ARG)
        comedi_subdevice *s = dev->subdevices + 3;
 
        if (!devpriv->enable_irq) {
-               printk("comedi_parport: bogus irq, ignored\n");
+               printk(KERN_ERR "comedi_parport: bogus irq, ignored\n");
                return IRQ_NONE;
        }
 
@@ -290,7 +291,7 @@ static irqreturn_t parport_interrupt(int irq, void *d PT_REGS_ARG)
        return IRQ_HANDLED;
 }
 
-static int parport_attach(comedi_device * dev, comedi_devconfig * it)
+static int parport_attach(comedi_device *dev, comedi_devconfig *it)
 {
        int ret;
        unsigned int irq;
@@ -298,7 +299,7 @@ static int parport_attach(comedi_device * dev, comedi_devconfig * it)
        comedi_subdevice *s;
 
        iobase = it->options[0];
-       printk("comedi%d: parport: 0x%04lx ", dev->minor, iobase);
+       printk(KERN_INFO "comedi%d: parport: 0x%04lx ", dev->minor, iobase);
        if (!request_region(iobase, PARPORT_SIZE, "parport (comedi)")) {
                printk("I/O port conflict\n");
                return -EIO;
@@ -318,9 +319,11 @@ static int parport_attach(comedi_device * dev, comedi_devconfig * it)
        }
        dev->board_name = "parport";
 
-       if ((ret = alloc_subdevices(dev, 4)) < 0)
+       ret = alloc_subdevices(dev, 4);
+       if (ret < 0)
                return ret;
-       if ((ret = alloc_private(dev, sizeof(parport_private))) < 0)
+       ret = alloc_private(dev, sizeof(struct parport_private));
+       if (ret < 0)
                return ret;
 
        s = dev->subdevices + 0;
@@ -373,7 +376,7 @@ static int parport_attach(comedi_device * dev, comedi_devconfig * it)
        return 1;
 }
 
-static int parport_detach(comedi_device * dev)
+static int parport_detach(comedi_device *dev)
 {
        printk("comedi%d: parport: remove\n", dev->minor);