staging: comedi: jr3_pci: tidy up jr3_pci_open()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Fri, 7 Mar 2014 20:41:02 +0000 (13:41 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Mar 2014 19:38:53 +0000 (12:38 -0700)
For aesthetics, rename the subdevice private data pointer from 'p' to
'spriv' and add a local variable for the comedi_subdevice pointer.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/jr3_pci.c

index ceaf417d3e2333b3a7e8aa80f7d146145ff17099..83f2f8b9026aa9d3f63169eef367ea2a306a1c09 100644 (file)
@@ -286,18 +286,18 @@ static int jr3_pci_ai_insn_read(struct comedi_device *dev,
 
 static int jr3_pci_open(struct comedi_device *dev)
 {
-       int i;
        struct jr3_pci_dev_private *devpriv = dev->private;
+       struct jr3_pci_subdev_private *spriv;
+       struct comedi_subdevice *s;
+       int i;
 
        dev_dbg(dev->class_dev, "jr3_pci_open\n");
        for (i = 0; i < devpriv->n_channels; i++) {
-               struct jr3_pci_subdev_private *p;
-
-               p = dev->subdevices[i].private;
-               if (p) {
-                       dev_dbg(dev->class_dev, "serial: %p %d (%d)\n", p,
-                               p->serial_no, p->channel_no);
-               }
+               s = &dev->subdevices[i];
+               spriv = s->private;
+               if (spriv)
+                       dev_dbg(dev->class_dev, "serial: %p %d (%d)\n",
+                               spriv, spriv->serial_no, spriv->channel_no);
        }
        return 0;
 }