summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2014-12-01 18:55:29 -0300
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2014-12-04 16:00:17 -0200
commitb59113845f76b8b21946563ee538a580fc972c6c (patch)
treebcbfe0eb0f780bb9ff9ba56f0a10cf1dfef16d7d
parente8379ecff31c0a172cb2ccc7cf28c24b45f144cf (diff)
[media] mn88473: Remove uneeded check before release_firmware()
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>
-rw-r--r--drivers/staging/media/mn88473/mn88473.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/media/mn88473/mn88473.c b/drivers/staging/media/mn88473/mn88473.c
index 165933540b3..52180bbc361 100644
--- a/drivers/staging/media/mn88473/mn88473.c
+++ b/drivers/staging/media/mn88473/mn88473.c
@@ -262,8 +262,7 @@ static int mn88473_init(struct dvb_frontend *fe)
return 0;
err:
- if (fw)
- release_firmware(fw);
+ release_firmware(fw);
dev_dbg(&client->dev, "failed=%d\n", ret);
return ret;