staging: comedi: remove unnecessary dev->board_name initialization
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 8 Apr 2013 17:57:35 +0000 (10:57 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Apr 2013 18:44:07 +0000 (11:44 -0700)
The dev->board_name is now initialized by the comedi core before calling
the(*attach) or (*auto_attach) function in a driver. As long as the driver
does no additional probing, it's no longer necessary initialize the board_name.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
51 files changed:
drivers/staging/comedi/drivers/8255.c
drivers/staging/comedi/drivers/addi_apci_1032.c
drivers/staging/comedi/drivers/addi_apci_1710.c
drivers/staging/comedi/drivers/addi_apci_2032.c
drivers/staging/comedi/drivers/addi_apci_2200.c
drivers/staging/comedi/drivers/addi_apci_3501.c
drivers/staging/comedi/drivers/adl_pci8164.c
drivers/staging/comedi/drivers/adl_pci9111.c
drivers/staging/comedi/drivers/adq12b.c
drivers/staging/comedi/drivers/adv_pci1723.c
drivers/staging/comedi/drivers/adv_pci1724.c
drivers/staging/comedi/drivers/aio_iiro_16.c
drivers/staging/comedi/drivers/c6xdigio.c
drivers/staging/comedi/drivers/cb_pcimdas.c
drivers/staging/comedi/drivers/cb_pcimdda.c
drivers/staging/comedi/drivers/comedi_parport.c
drivers/staging/comedi/drivers/comedi_test.c
drivers/staging/comedi/drivers/contec_pci_dio.c
drivers/staging/comedi/drivers/das16m1.c
drivers/staging/comedi/drivers/das6402.c
drivers/staging/comedi/drivers/dmm32at.c
drivers/staging/comedi/drivers/dt2814.c
drivers/staging/comedi/drivers/dt2815.c
drivers/staging/comedi/drivers/dt2817.c
drivers/staging/comedi/drivers/dt9812.c
drivers/staging/comedi/drivers/dyna_pci10xx.c
drivers/staging/comedi/drivers/fl512.c
drivers/staging/comedi/drivers/icp_multi.c
drivers/staging/comedi/drivers/ii_pci20kc.c
drivers/staging/comedi/drivers/jr3_pci.c
drivers/staging/comedi/drivers/ke_counter.c
drivers/staging/comedi/drivers/mpc624.c
drivers/staging/comedi/drivers/multiq3.c
drivers/staging/comedi/drivers/ni_daq_700.c
drivers/staging/comedi/drivers/ni_daq_dio24.c
drivers/staging/comedi/drivers/pcl725.c
drivers/staging/comedi/drivers/pcm3724.c
drivers/staging/comedi/drivers/pcm3730.c
drivers/staging/comedi/drivers/pcmda12.c
drivers/staging/comedi/drivers/pcmmio.c
drivers/staging/comedi/drivers/quatech_daqp_cs.c
drivers/staging/comedi/drivers/rti802.c
drivers/staging/comedi/drivers/s526.c
drivers/staging/comedi/drivers/s626.c
drivers/staging/comedi/drivers/serial2002.c
drivers/staging/comedi/drivers/skel.c
drivers/staging/comedi/drivers/ssv_dnp.c
drivers/staging/comedi/drivers/unioxx5.c
drivers/staging/comedi/drivers/usbdux.c
drivers/staging/comedi/drivers/usbduxfast.c
drivers/staging/comedi/drivers/usbduxsigma.c

index c7aa41a..0237187 100644 (file)
@@ -354,8 +354,6 @@ static int dev_8255_attach(struct comedi_device *dev,
        unsigned long iobase;
        int i;
 
-       dev->board_name = "8255";
-
        for (i = 0; i < COMEDI_NDEVCONFOPTS; i++) {
                iobase = it->options[i];
                if (!iobase)
index 3d32448..3d4878f 100644 (file)
@@ -296,8 +296,6 @@ static int apci1032_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 3640c50..acde462 100644 (file)
@@ -35,8 +35,6 @@ static int apci1710_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 59e092e..e5fa20a 100644 (file)
@@ -287,8 +287,6 @@ static int apci2032_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_pci_enable(dev);
        if (ret)
                return ret;
index 0953f65..8cad37c 100644 (file)
@@ -88,8 +88,6 @@ static int apci2200_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_pci_enable(dev);
        if (ret)
                return ret;
index 75a36e3..a0cf6ec 100644 (file)
@@ -339,8 +339,6 @@ static int apci3501_auto_attach(struct comedi_device *dev,
        int ao_n_chan;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 469a51d..b3ec60a 100644 (file)
@@ -78,8 +78,6 @@ static int adl_pci8164_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_pci_enable(dev);
        if (ret)
                return ret;
index 9c27e98..99bce7c 100644 (file)
@@ -865,8 +865,6 @@ static int pci9111_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        dev_private = kzalloc(sizeof(*dev_private), GFP_KERNEL);
        if (!dev_private)
                return -ENOMEM;
index f7950df..dd31597 100644 (file)
@@ -217,8 +217,6 @@ static int adq12b_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        int unipolar, differential;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        unipolar = it->options[1];
        differential = it->options[2];
index 9e81e58..ccc114d 100644 (file)
@@ -242,8 +242,6 @@ static int pci1723_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index a33929e..e60f125 100644 (file)
@@ -358,8 +358,6 @@ static int adv_pci1724_auto_attach(struct comedi_device *dev,
                devpriv->gain_value[i] = -1;
        }
 
-       dev->board_name = dev->driver->driver_name;
-
        retval = comedi_pci_enable(dev);
        if (retval)
                return retval;
index 64c1ae5..29aba2b 100644 (file)
@@ -83,8 +83,6 @@ static int aio_iiro_16_attach(struct comedi_device *dev,
 
        printk(KERN_INFO "comedi%d: aio_iiro_16: ", dev->minor);
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
 
        if (!request_region(iobase, AIO_IIRO_16_SIZE, dev->board_name)) {
index 070037c..816b879 100644 (file)
@@ -431,7 +431,6 @@ static int c6xdigio_attach(struct comedi_device *dev,
                return -EIO;
        }
        dev->iobase = iobase;
-       dev->board_name = "c6xdigio";
 
        result = comedi_alloc_subdevices(dev, 2);
        if (result)
index f6d99a3..29813c9 100644 (file)
@@ -215,8 +215,6 @@ static int cb_pcimdas_auto_attach(struct comedi_device *dev,
        unsigned long iobase_8255;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index d00f7f6..f7d1136 100644 (file)
@@ -161,8 +161,6 @@ static int cb_pcimdda_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 76d59dc..e392f68 100644 (file)
@@ -266,8 +266,6 @@ static int parport_attach(struct comedi_device *dev,
        unsigned long iobase;
        struct comedi_subdevice *s;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        if (!request_region(iobase, PARPORT_SIZE, dev->board_name)) {
                dev_err(dev->class_dev, "I/O port conflict\n");
index 270fea5..c1d8e86 100644 (file)
@@ -384,8 +384,6 @@ static int waveform_attach(struct comedi_device *dev,
        int i;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index da0be62..f2230bf 100644 (file)
@@ -77,8 +77,6 @@ static int contec_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_pci_enable(dev);
        if (ret)
                return ret;
index 7ba8fc7..3c37a66 100644 (file)
@@ -573,8 +573,6 @@ static int das16m1_attach(struct comedi_device *dev,
        unsigned int irq;
        unsigned long iobase;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
 
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
index 2efddb8..c2f20c2 100644 (file)
@@ -288,8 +288,6 @@ static int das6402_attach(struct comedi_device *dev,
        int ret;
        struct comedi_subdevice *s;
 
-       dev->board_name = "das6402";
-
        iobase = it->options[0];
        if (iobase == 0)
                iobase = 0x300;
index 9e21241..ca07e54 100644 (file)
@@ -698,8 +698,6 @@ static int dmm32at_attach(struct comedi_device *dev,
        unsigned long iobase;
        unsigned int irq;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        irq = it->options[1];
 
index e520dba..e4d629e 100644 (file)
@@ -255,7 +255,6 @@ static int dt2814_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                return -EIO;
        }
        dev->iobase = iobase;
-       dev->board_name = "dt2814";
 
        outb(0, dev->iobase + DT2814_CSR);
        udelay(100);
index 10828fc..ad18b1a 100644 (file)
@@ -171,7 +171,6 @@ static int dt2815_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        }
 
        dev->iobase = iobase;
-       dev->board_name = "dt2815";
 
        ret = comedi_alloc_subdevices(dev, 1);
        if (ret)
index 502e42e..0dd6dd0 100644 (file)
@@ -135,7 +135,6 @@ static int dt2817_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                return -EIO;
        }
        dev->iobase = iobase;
-       dev->board_name = "dt2817";
 
        ret = comedi_alloc_subdevices(dev, 1);
        if (ret)
index 9ca3488..81eb5ed 100644 (file)
@@ -1012,8 +1012,6 @@ static int dt9812_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = "dt9812";
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 17f9ec2..93ec8e4 100644 (file)
@@ -187,8 +187,6 @@ static int dyna_pci10xx_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 019c96e..07f9d67 100644 (file)
@@ -125,7 +125,6 @@ static int fl512_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                return -EIO;
        }
        dev->iobase = iobase;
-       dev->board_name = "fl512";
 
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
index f29a797..2bd916e 100644 (file)
@@ -503,8 +503,6 @@ static int icp_multi_auto_attach(struct comedi_device *dev,
        resource_size_t iobase;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 93584e2..03719f2 100644 (file)
@@ -212,7 +212,6 @@ static int pci20xxx_attach(struct comedi_device *dev,
        dev->private = devpriv;
 
        devpriv->ioaddr = (void __iomem *)(unsigned long)it->options[0];
-       dev->board_name = "pci20kc";
 
        /* Check PCI-20001 C-2A Carrier Board ID */
        if ((readb(devpriv->ioaddr) & PCI20000_ID) != PCI20000_ID) {
index f21ebb5..653c5c3 100644 (file)
@@ -700,7 +700,6 @@ static int jr3_pci_auto_attach(struct comedi_device *dev,
                return -EINVAL;
                break;
        }
-       dev->board_name = "jr3_pci";
 
        result = comedi_pci_enable(dev);
        if (result)
index bca29e5..e0e6475 100644 (file)
@@ -96,8 +96,6 @@ static int cnt_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_pci_enable(dev);
        if (ret)
                return ret;
index 67dc5ad..3b7a53f 100644 (file)
@@ -296,7 +296,6 @@ static int mpc624_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        }
 
        dev->iobase = iobase;
-       dev->board_name = "mpc624";
 
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
index 1f5f402..fe94a62 100644 (file)
@@ -252,7 +252,6 @@ static int multiq3_attach(struct comedi_device *dev,
                        dev->minor, irq);
        else
                printk(KERN_WARNING "comedi%d: no irq\n", dev->minor);
-       dev->board_name = "multiq3";
 
        result = comedi_alloc_subdevices(dev, 5);
        if (result)
index 9cc6092..d067ef7 100644 (file)
@@ -202,8 +202,6 @@ static int daq700_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        link->config_flags |= CONF_AUTO_SET_IO;
        ret = comedi_pcmcia_enable(dev, NULL);
        if (ret)
index e1cc9d0..c17ffe3 100644 (file)
@@ -52,8 +52,6 @@ static int dio24_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        link->config_flags |= CONF_AUTO_SET_IO;
        ret = comedi_pcmcia_enable(dev, NULL);
        if (ret)
index 21fbc1a..3cd5e7f 100644 (file)
@@ -54,7 +54,6 @@ static int pcl725_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                printk("I/O port conflict\n");
                return -EIO;
        }
-       dev->board_name = "pcl725";
        dev->iobase = iobase;
        dev->irq = 0;
 
index 5f062df..d78aeeb 100644 (file)
@@ -230,8 +230,6 @@ static int pcm3724_attach(struct comedi_device *dev,
        unsigned int iorange;
        int ret, i;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        iorange = PCM3724_SIZE;
 
index 067f14d..d049194 100644 (file)
@@ -64,7 +64,6 @@ static int pcm3730_attach(struct comedi_device *dev,
                return -EIO;
        }
        dev->iobase = iobase;
-       dev->board_name = "pcm3730";
        dev->iobase = dev->iobase;
        dev->irq = 0;
 
index 13f79f4..64b76e4 100644 (file)
@@ -168,8 +168,6 @@ static int pcmda12_attach(struct comedi_device *dev,
        }
        dev->iobase = iobase;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 5fa1fe0..a342e37 100644 (file)
@@ -1038,8 +1038,6 @@ static int pcmmio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        unsigned int irq[MAX_ASICS];
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        irq[0] = it->options[1];
 
index 911eb6b..e092ce8 100644 (file)
@@ -715,8 +715,6 @@ static int daqp_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 2185ca1..1440162 100644 (file)
@@ -103,8 +103,6 @@ static int rti802_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        }
        dev->iobase = iobase;
 
-       dev->board_name = "rti802";
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 39232b3..b490b47 100644 (file)
@@ -555,8 +555,6 @@ static int s526_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        int iobase;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        iobase = it->options[0];
        if (!iobase || !request_region(iobase, S526_IOSIZE, dev->board_name)) {
                comedi_error(dev, "I/O port conflict");
index 02f9406..b6c7cb9 100644 (file)
@@ -2590,8 +2590,6 @@ static int s626_auto_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 21e3cf1..8900086 100644 (file)
@@ -724,8 +724,6 @@ static int serial2002_attach(struct comedi_device *dev,
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
        if (!devpriv)
                return -ENOMEM;
index 6fb7d5d..6dc77fc 100644 (file)
@@ -481,10 +481,12 @@ static int skel_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 
        thisboard = comedi_board(dev);
 
-/*
- * Initialize dev->board_name.
- */
-       dev->board_name = thisboard->name;
+       /*
+        * The dev->board_name is initialized by the comedi core before
+        * calling the (*attach) function. It can be optionally set by
+        * the driver if additional probing has been done.
+        */
+       /* dev->board_name = thisboard->name; */
 
        /* Allocate the private data */
        devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL);
index afa4016..a76df09 100644 (file)
@@ -167,8 +167,6 @@ static int dnp_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        struct comedi_subdevice *s;
        int ret;
 
-       dev->board_name = dev->driver->driver_name;
-
        ret = comedi_alloc_subdevices(dev, 1);
        if (ret)
                return ret;
index 74b974b..fdeec4b 100644 (file)
@@ -442,7 +442,6 @@ static int unioxx5_attach(struct comedi_device *dev,
 
        iobase = it->options[0];
 
-       dev->board_name = DRIVER_NAME;
        dev->iobase = iobase;
        iobase += UNIOXX5_SUBDEV_BASE;
 
index 148ecfc..6f5da67 100644 (file)
@@ -2207,8 +2207,6 @@ static int usbdux_attach_common(struct comedi_device *dev,
        /* pointer back to the corresponding comedi device */
        udev->comedidev = dev;
 
-       dev->board_name = "usbdux";
-
        /* set number of subdevices */
        if (udev->high_speed) {
                /* with pwm */
index 1ba0e3d..7f95af3 100644 (file)
@@ -1387,7 +1387,7 @@ static int usbduxfast_attach_common(struct comedi_device *dev,
        down(&udfs->sem);
        /* pointer back to the corresponding comedi device */
        udfs->comedidev = dev;
-       dev->board_name = "usbduxfast";
+
        ret = comedi_alloc_subdevices(dev, 1);
        if (ret) {
                up(&udfs->sem);
index 1c2bfb4..d3bc1b9 100644 (file)
@@ -2205,7 +2205,7 @@ static int usbduxsigma_attach_common(struct comedi_device *dev,
        down(&uds->sem);
        /* pointer back to the corresponding comedi device */
        uds->comedidev = dev;
-       dev->board_name = "usbduxsigma";
+
        /* set number of subdevices */
        if (uds->high_speed)
                n_subdevs = 4;  /* with pwm */