[media] DVB-frontends: Deletion of unnecessary checks before the function call "relea...
authorMarkus Elfring <elfring@users.sourceforge.net>
Wed, 19 Nov 2014 21:27:24 +0000 (18:27 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 25 Nov 2014 11:52:44 +0000 (09:52 -0200)
GIT_AUTHOR_DATE=1416472432
The release_firmware() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/dvb-frontends/drx39xyj/drxj.c
drivers/media/dvb-frontends/drxk_hard.c
drivers/media/dvb-frontends/m88ds3103.c
drivers/media/dvb-frontends/si2168.c

index 5ec221f..2bfa7a4 100644 (file)
@@ -12255,8 +12255,7 @@ static void drx39xxj_release(struct dvb_frontend *fe)
        kfree(demod->my_ext_attr);
        kfree(demod->my_common_attr);
        kfree(demod->my_i2c_dev_addr);
-       if (demod->firmware)
-               release_firmware(demod->firmware);
+       release_firmware(demod->firmware);
        kfree(demod);
        kfree(state);
 }
Simple merge