aboutsummaryrefslogtreecommitdiff
path: root/drivers/media/tuners
diff options
context:
space:
mode:
authorGravatar Antti Palosaari <crope@iki.fi> 2014-12-06 14:51:17 -0300
committerGravatar Mauro Carvalho Chehab <mchehab@osg.samsung.com> 2015-01-29 18:48:39 -0200
commit78dab40a51fc4370744ba38b5d8b9c3294c8c136 (patch)
tree8d28cfa2e87044f9f7a0cf2eb7b1aa5f61752a34 /drivers/media/tuners
parent[media] si2157: carry pointer to client instead of state in tuner_priv (diff)
downloadlinux-78dab40a51fc4370744ba38b5d8b9c3294c8c136.tar.gz
linux-78dab40a51fc4370744ba38b5d8b9c3294c8c136.tar.bz2
linux-78dab40a51fc4370744ba38b5d8b9c3294c8c136.zip
[media] si2157: change firmware download error handling
Rename firmare download error path goto label. Remove firmware NULL set as NULL value is not needed anymore, due to recent change which started using goto labels for firmware error handling. Cc: Olli Salonen <olli.salonen@iki.fi> Signed-off-by: Antti Palosaari <crope@iki.fi> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/tuners')
-rw-r--r--drivers/media/tuners/si2157.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c
index 88afb2adfdd1..6174c8eaa8b4 100644
--- a/drivers/media/tuners/si2157.c
+++ b/drivers/media/tuners/si2157.c
@@ -81,7 +81,7 @@ static int si2157_init(struct dvb_frontend *fe)
struct si2157_dev *dev = i2c_get_clientdata(client);
int ret, len, remaining;
struct si2157_cmd cmd;
- const struct firmware *fw = NULL;
+ const struct firmware *fw;
u8 *fw_file;
unsigned int chip_id;
@@ -154,7 +154,7 @@ static int si2157_init(struct dvb_frontend *fe)
dev_err(&client->dev, "firmware file '%s' is invalid\n",
fw_file);
ret = -EINVAL;
- goto fw_release_exit;
+ goto err_release_firmware;
}
dev_info(&client->dev, "downloading firmware from file '%s'\n",
@@ -169,12 +169,11 @@ static int si2157_init(struct dvb_frontend *fe)
if (ret) {
dev_err(&client->dev, "firmware download failed %d\n",
ret);
- goto fw_release_exit;
+ goto err_release_firmware;
}
}
release_firmware(fw);
- fw = NULL;
skip_fw_download:
/* reboot the tuner with new firmware? */
@@ -191,7 +190,7 @@ warm:
dev->active = true;
return 0;
-fw_release_exit:
+err_release_firmware:
release_firmware(fw);
err:
dev_dbg(&client->dev, "failed=%d\n", ret);