[media] saa7164: Remove pointless conditional and save a few bytes in saa7164_downloa...
authorJesper Juhl <jj@chaosbits.net>
Sat, 18 Dec 2010 22:05:02 +0000 (19:05 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 29 Dec 2010 10:17:04 +0000 (08:17 -0200)
Hi,

release_firmware() just does nothing if passed a NULL pointer. So there's
no reason to test before the call in
saa7164-fw.c::saa7164_downloadfirmware().

Removing the pointless conditional also saves a few bytes.
before:
   text    data     bss     dec     hex filename
   7943     112    2144   10199    27d7 drivers/media/video/saa7164/saa7164-fw.o
after:
   text    data     bss     dec     hex filename
   7931     112    2136   10179    27c3 drivers/media/video/saa7164/saa7164-fw.o

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Cc: Steven Toth <stoth@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/saa7164/saa7164-fw.c

index 22b02323dca9c4714dd5bade1b7a4d6485421ba7..ebed6f786a23750a69d17df841924563b66a9123 100644 (file)
@@ -608,8 +608,6 @@ int saa7164_downloadfirmware(struct saa7164_dev *dev)
        ret = 0;
 
 out:
-       if (fw)
-               release_firmware(fw);
-
+       release_firmware(fw);
        return ret;
 }