V4L/DVB (4030): Minor code cleanup on the genpix-module
authorPatrick Boettcher <pb@linuxtv.org>
Sun, 14 May 2006 16:29:48 +0000 (13:29 -0300)
committerMauro Carvalho Chehab <mchehab@infradead.org>
Sun, 25 Jun 2006 05:00:33 +0000 (02:00 -0300)
Minor code cleanup on the genpix-module.

Signed-off-by: Patrick Boettcher <pb@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
drivers/media/dvb/dvb-usb/gp8psk.c

index d6771cbd262dc28da358ef02976eaa35b85309f8..9a98f3fdae318c98a019ce20497372ce3fb9d8e9 100644 (file)
@@ -92,36 +92,38 @@ static int gp8psk_load_bcm4500fw(struct dvb_usb_device *d)
                return ret;
        }
 
-       if (gp8psk_usb_out_op(d, LOAD_BCM4500,1,0,NULL, 0)) {
-               release_firmware(fw);
-               return -EINVAL;
-       }
+       ret = -EINVAL;
+
+       if (gp8psk_usb_out_op(d, LOAD_BCM4500,1,0,NULL, 0))
+               goto out_rel_fw;
 
        info("downloaidng bcm4500 firmware from file '%s'",bcm4500_firmware);
 
        ptr = fw->data;
-       buf = (u8 *) kmalloc(512, GFP_KERNEL | GFP_DMA);
+       buf = kmalloc(512, GFP_KERNEL | GFP_DMA);
 
        while (ptr[0] != 0xff) {
                u16 buflen = ptr[0] + 4;
                if (ptr + buflen >= fw->data + fw->size) {
                        err("failed to load bcm4500 firmware.");
-                       release_firmware(fw);
-                       kfree(buf);
-                       return -EINVAL;
+                       goto out_free;
                }
                memcpy(buf, ptr, buflen);
                if (dvb_usb_generic_write(d, buf, buflen)) {
                        err("failed to load bcm4500 firmware.");
-                       release_firmware(fw);
-                       kfree(buf);
-                       return -EIO;
+                       goto out_free;
                }
                ptr += buflen;
        }
-       release_firmware(fw);
+
+       ret = 0;
+
+out_free:
        kfree(buf);
-       return 0;
+out_rel_fw:
+       release_firmware(fw);
+
+       return ret;
 }
 
 static int gp8psk_power_ctrl(struct dvb_usb_device *d, int onoff)