diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-01 15:12:56 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-04-01 06:22:27 -0300 |
| commit | 68feaa6b8bc537de1d02732fb1b74fe5433ceff1 (patch) | |
| tree | e614fa0ea6049bdd01b28d61a36ff0c4f073a6ee /drivers/staging/media/mn88472 | |
| parent | 18acc6ace529f331cf6a780ac0d68216df47ca27 (diff) | |
| download | linux-68feaa6b8bc537de1d02732fb1b74fe5433ceff1.tar.bz2 | |
[media] mn88472: Deletion of an unnecessary check before the function call "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>
Diffstat (limited to 'drivers/staging/media/mn88472')
| -rw-r--r-- | drivers/staging/media/mn88472/mn88472.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/media/mn88472/mn88472.c b/drivers/staging/media/mn88472/mn88472.c index 6eebe564e557..ce14cb287594 100644 --- a/drivers/staging/media/mn88472/mn88472.c +++ b/drivers/staging/media/mn88472/mn88472.c @@ -299,8 +299,7 @@ static int mn88472_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; |