staging: comedi: daqboard2000: use request_firmware()
authorH Hartley Sweeten <hartleys@visionengravers.com>
Wed, 19 Sep 2012 01:46:57 +0000 (18:46 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 19 Sep 2012 08:41:43 +0000 (09:41 +0100)
This driver requires loading a firmware file for the cpld. This
is currently being done by passing the firmware data using the
COMEDI_DEVCONFIG ioctl through the attach() hook in the driver.
This does not work for auto-configured PCI devices due to the
firmware loading options not being set in the comedi_devconfig
parameter passed to the driver.

Change the driver so it gets the firmware using request_firmware()
and ignore any firmware options passed in the comedi_devconfig.

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/daqboard2000.c

index 0daa83db726aa6c81e1d114c5c627f6be0efdb51..d85f58e10bbe84719f5d78ce13b9f69f0e58bf26 100644 (file)
@@ -117,9 +117,12 @@ Configuration options:
 
 #include <linux/delay.h>
 #include <linux/interrupt.h>
+#include <linux/firmware.h>
 
 #include "8255.h"
 
+#define DAQBOARD2000_FIRMWARE          "daqboard2000_firmware.bin"
+
 #define PCI_VENDOR_ID_IOTECH           0x1616
 
 #define DAQBOARD2000_SUBSYSTEM_IDS2    0x0002  /* Daqboard/2000 - 2 Dacs */
@@ -547,14 +550,14 @@ static int daqboard2000_writeCPLD(struct comedi_device *dev, int data)
 }
 
 static int initialize_daqboard2000(struct comedi_device *dev,
-                                  unsigned char *cpld_array, int len)
+                                  const u8 *cpld_array, size_t len)
 {
        struct daqboard2000_private *devpriv = dev->private;
        int result = -EIO;
        /* Read the serial EEPROM control register */
        int secr;
        int retry;
-       int i;
+       size_t i;
 
        /* Check to make sure the serial eeprom is present on the board */
        secr = readl(devpriv->plx + 0x6c);
@@ -604,6 +607,22 @@ static int initialize_daqboard2000(struct comedi_device *dev,
        return result;
 }
 
+static int daqboard2000_upload_firmware(struct comedi_device *dev)
+{
+       struct pci_dev *pcidev = comedi_to_pci_dev(dev);
+       const struct firmware *fw;
+       int ret;
+
+       ret = request_firmware(&fw, DAQBOARD2000_FIRMWARE, &pcidev->dev);
+       if (ret)
+               return ret;
+
+       ret = initialize_daqboard2000(dev, fw->data, fw->size);
+       release_firmware(fw);
+
+       return ret;
+}
+
 static void daqboard2000_adcStopDmaTransfer(struct comedi_device *dev)
 {
 /*  printk("Implement: daqboard2000_adcStopDmaTransfer\n");*/
@@ -747,8 +766,6 @@ static int daqboard2000_attach(struct comedi_device *dev,
        struct pci_dev *pcidev;
        struct comedi_subdevice *s;
        resource_size_t pci_base;
-       void *aux_data;
-       unsigned int aux_len;
        int result;
 
        result = alloc_private(dev, sizeof(*devpriv));
@@ -790,18 +807,10 @@ static int daqboard2000_attach(struct comedi_device *dev,
           printk("Interrupt before is: %x\n", interrupt);
         */
 
-       aux_data = comedi_aux_data(it->options, 0);
-       aux_len = it->options[COMEDI_DEVCONF_AUX_DATA_LENGTH];
-
-       if (aux_data && aux_len) {
-               result = initialize_daqboard2000(dev, aux_data, aux_len);
-       } else {
-               dev_dbg(dev->class_dev,
-                       "no FPGA initialization code, aborting\n");
-               result = -EIO;
-       }
+       result = daqboard2000_upload_firmware(dev);
        if (result < 0)
-               goto out;
+               return result;
+
        daqboard2000_initializeAdc(dev);
        daqboard2000_initializeDac(dev);
        /*
@@ -835,7 +844,6 @@ static int daqboard2000_attach(struct comedi_device *dev,
        result = subdev_8255_init(dev, s, daqboard2000_8255_cb,
                        (unsigned long)(devpriv->daq + dioP2ExpansionIO8Bit));
 
-out:
        return result;
 }
 
@@ -896,3 +904,4 @@ module_comedi_pci_driver(daqboard2000_driver, daqboard2000_pci_driver);
 MODULE_AUTHOR("Comedi http://www.comedi.org");
 MODULE_DESCRIPTION("Comedi low-level driver");
 MODULE_LICENSE("GPL");
+MODULE_FIRMWARE(DAQBOARD2000_FIRMWARE);