Commit dc89cfcf authored by Markus Elfring's avatar Markus Elfring Committed by 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: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 99d2fae9
...@@ -196,8 +196,7 @@ static int si2157_init(struct dvb_frontend *fe) ...@@ -196,8 +196,7 @@ static int si2157_init(struct dvb_frontend *fe)
return 0; return 0;
err: err:
if (fw) release_firmware(fw);
release_firmware(fw);
dev_dbg(&s->client->dev, "failed=%d\n", ret); dev_dbg(&s->client->dev, "failed=%d\n", ret);
return ret; return ret;
......
...@@ -1336,8 +1336,7 @@ static int xc5000_release(struct dvb_frontend *fe) ...@@ -1336,8 +1336,7 @@ static int xc5000_release(struct dvb_frontend *fe)
if (priv) { if (priv) {
cancel_delayed_work(&priv->timer_sleep); cancel_delayed_work(&priv->timer_sleep);
if (priv->firmware) release_firmware(priv->firmware);
release_firmware(priv->firmware);
hybrid_tuner_release_state(priv); hybrid_tuner_release_state(priv);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment