staging: comedi: jr3_pci: tidy up comedi_load_firmware()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Fri, 17 May 2013 18:16:39 +0000 (11:16 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 17 May 2013 19:09:49 +0000 (12:09 -0700)
Refactor the function to remove some unnecessary indents and make
it a bit more concise.

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 42e78d1e2ae92c781ce3d1a94c32c5274c66ff1f..9b61b557010f57774b247eeacd35ee97bcb90885 100644 (file)
@@ -95,28 +95,28 @@ struct jr3_pci_subdev_private {
 /* Hotplug firmware loading stuff */
 static int comedi_load_firmware(struct comedi_device *dev, const char *name,
                                int (*cb)(struct comedi_device *dev,
-                                       const u8 *data, size_t size))
+                                         const u8 *data, size_t size))
 {
        struct pci_dev *pcidev = comedi_to_pci_dev(dev);
-       int result = 0;
        const struct firmware *fw;
        char *firmware_path;
+       int ret;
+
+       if (!cb)
+               return -EINVAL;
 
        firmware_path = kasprintf(GFP_KERNEL, "comedi/%s", name);
-       if (!firmware_path) {
-               result = -ENOMEM;
-       } else {
-               result = request_firmware(&fw, firmware_path, &pcidev->dev);
-               if (result == 0) {
-                       if (!cb)
-                               result = -EINVAL;
-                       else
-                               result = cb(dev, fw->data, fw->size);
-                       release_firmware(fw);
-               }
-               kfree(firmware_path);
+       if (!firmware_path)
+               return -ENOMEM;
+
+       ret = request_firmware(&fw, firmware_path, &pcidev->dev);
+       if (ret == 0) {
+               ret = cb(dev, fw->data, fw->size);
+               release_firmware(fw);
        }
-       return result;
+       kfree(firmware_path);
+
+       return ret;
 }
 
 static struct poll_delay_t poll_delay_min_max(int min, int max)