提交 dc89cfcf 编写于 作者: M Markus Elfring 提交者: Mauro Carvalho Chehab

[media] tuners: remove uneeded checks 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: NMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: NMauro Carvalho Chehab <mchehab@osg.samsung.com>
上级 99d2fae9
......@@ -196,8 +196,7 @@ static int si2157_init(struct dvb_frontend *fe)
return 0;
err:
if (fw)
release_firmware(fw);
release_firmware(fw);
dev_dbg(&s->client->dev, "failed=%d\n", ret);
return ret;
......
......@@ -1336,8 +1336,7 @@ static int xc5000_release(struct dvb_frontend *fe)
if (priv) {
cancel_delayed_work(&priv->timer_sleep);
if (priv->firmware)
release_firmware(priv->firmware);
release_firmware(priv->firmware);
hybrid_tuner_release_state(priv);
}
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册