USB: emi26.c: remove err() usage
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Apr 2012 23:53:37 +0000 (16:53 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Apr 2012 23:53:37 +0000 (16:53 -0700)
err() was a very old USB-specific macro that I thought had
gone away.  This patch removes it from being used in the
driver and uses dev_err() instead.

CC: Paul Gortmaker <paul.gortmaker@windriver.com>
CC: Andrew Morton <akpm@linux-foundation.org>
CC: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/misc/emi26.c

index da97dcec1f32c88ee95c49a097517b1e9390b41d..d65984dee751728d7cb96cb0e3aa60109429ca73 100644 (file)
@@ -78,18 +78,14 @@ static int emi26_load_firmware (struct usb_device *dev)
        const struct firmware *bitstream_fw = NULL;
        const struct firmware *firmware_fw = NULL;
        const struct ihex_binrec *rec;
-       int err;
+       int err = -ENOMEM;
        int i;
        __u32 addr;     /* Address to write */
        __u8 *buf;
 
        buf = kmalloc(FW_LOAD_SIZE, GFP_KERNEL);
-       if (!buf) {
-               dev_err(&dev->dev, "%s - error loading firmware: error = %d\n",
-                       __func__, -ENOMEM);
-               err = -ENOMEM;
+       if (!buf)
                goto wraperr;
-       }
 
        err = request_ihex_firmware(&loader_fw, "emi26/loader.fw", &dev->dev);
        if (err)
@@ -111,11 +107,8 @@ static int emi26_load_firmware (struct usb_device *dev)
 
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
-       if (err < 0) {
-               dev_err(&dev->dev,"%s - error loading firmware: error = %d\n",
-                       __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
 
        rec = (const struct ihex_binrec *)loader_fw->data;
        /* 1. We need to put the loader for the FPGA into the EZ-USB */
@@ -123,19 +116,15 @@ static int emi26_load_firmware (struct usb_device *dev)
                err = emi26_writememory(dev, be32_to_cpu(rec->addr),
                                        rec->data, be16_to_cpu(rec->len),
                                        ANCHOR_LOAD_INTERNAL);
-               if (err < 0) {
-                       err("%s - error loading firmware: error = %d", __func__, err);
+               if (err < 0)
                        goto wraperr;
-               }
                rec = ihex_next_binrec(rec);
        }
 
        /* De-assert reset (let the CPU run) */
        err = emi26_set_reset(dev,0);
-       if (err < 0) {
-               err("%s - error loading firmware: error = %d", __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
        msleep(250);    /* let device settle */
 
        /* 2. We upload the FPGA firmware into the EMI
@@ -153,18 +142,14 @@ static int emi26_load_firmware (struct usb_device *dev)
                        rec = ihex_next_binrec(rec);
                }
                err = emi26_writememory(dev, addr, buf, i, ANCHOR_LOAD_FPGA);
-               if (err < 0) {
-                       err("%s - error loading firmware: error = %d", __func__, err);
+               if (err < 0)
                        goto wraperr;
-               }
        } while (rec);
 
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
-       if (err < 0) {
-               err("%s - error loading firmware: error = %d", __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
 
        /* 3. We need to put the loader for the firmware into the EZ-USB (again...) */
        for (rec = (const struct ihex_binrec *)loader_fw->data;
@@ -172,19 +157,15 @@ static int emi26_load_firmware (struct usb_device *dev)
                err = emi26_writememory(dev, be32_to_cpu(rec->addr),
                                        rec->data, be16_to_cpu(rec->len),
                                        ANCHOR_LOAD_INTERNAL);
-               if (err < 0) {
-                       err("%s - error loading firmware: error = %d", __func__, err);
+               if (err < 0)
                        goto wraperr;
-               }
        }
        msleep(250);    /* let device settle */
 
        /* De-assert reset (let the CPU run) */
        err = emi26_set_reset(dev,0);
-       if (err < 0) {
-               err("%s - error loading firmware: error = %d", __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
 
        /* 4. We put the part of the firmware that lies in the external RAM into the EZ-USB */
 
@@ -194,19 +175,15 @@ static int emi26_load_firmware (struct usb_device *dev)
                        err = emi26_writememory(dev, be32_to_cpu(rec->addr),
                                                rec->data, be16_to_cpu(rec->len),
                                                ANCHOR_LOAD_EXTERNAL);
-                       if (err < 0) {
-                               err("%s - error loading firmware: error = %d", __func__, err);
+                       if (err < 0)
                                goto wraperr;
-                       }
                }
        }
-       
+
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
-       if (err < 0) {
-               err("%s - error loading firmware: error = %d", __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
 
        for (rec = (const struct ihex_binrec *)firmware_fw->data;
             rec; rec = ihex_next_binrec(rec)) {
@@ -214,19 +191,15 @@ static int emi26_load_firmware (struct usb_device *dev)
                        err = emi26_writememory(dev, be32_to_cpu(rec->addr),
                                                rec->data, be16_to_cpu(rec->len),
                                                ANCHOR_LOAD_INTERNAL);
-                       if (err < 0) {
-                               err("%s - error loading firmware: error = %d", __func__, err);
+                       if (err < 0)
                                goto wraperr;
-                       }
                }
        }
 
        /* De-assert reset (let the CPU run) */
        err = emi26_set_reset(dev,0);
-       if (err < 0) {
-               err("%s - error loading firmware: error = %d", __func__, err);
+       if (err < 0)
                goto wraperr;
-       }
        msleep(250);    /* let device settle */
 
        /* return 1 to fail the driver inialization
@@ -234,6 +207,10 @@ static int emi26_load_firmware (struct usb_device *dev)
        err = 1;
 
 wraperr:
+       if (err < 0)
+               dev_err(&dev->dev,"%s - error loading firmware: error = %d\n",
+                       __func__, err);
+
        release_firmware(loader_fw);
        release_firmware(bitstream_fw);
        release_firmware(firmware_fw);