Commit c32b2816 authored by Antti Palosaari's avatar Antti Palosaari Committed by Mauro Carvalho Chehab

[media] si2168: change firmware variable name and type

Rename firmware variable from fw_file to fw_name and change its type
from u8 to const char as request_firmware() input defines.
Signed-off-by: default avatarAntti Palosaari <crope@iki.fi>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 9b7839c3
...@@ -347,7 +347,7 @@ static int si2168_init(struct dvb_frontend *fe) ...@@ -347,7 +347,7 @@ static int si2168_init(struct dvb_frontend *fe)
struct si2168_dev *dev = i2c_get_clientdata(client); struct si2168_dev *dev = i2c_get_clientdata(client);
int ret, len, remaining; int ret, len, remaining;
const struct firmware *fw; const struct firmware *fw;
u8 *fw_file; const char *fw_name;
struct si2168_cmd cmd; struct si2168_cmd cmd;
unsigned int chip_id; unsigned int chip_id;
...@@ -405,13 +405,13 @@ static int si2168_init(struct dvb_frontend *fe) ...@@ -405,13 +405,13 @@ static int si2168_init(struct dvb_frontend *fe)
switch (chip_id) { switch (chip_id) {
case SI2168_A20: case SI2168_A20:
fw_file = SI2168_A20_FIRMWARE; fw_name = SI2168_A20_FIRMWARE;
break; break;
case SI2168_A30: case SI2168_A30:
fw_file = SI2168_A30_FIRMWARE; fw_name = SI2168_A30_FIRMWARE;
break; break;
case SI2168_B40: case SI2168_B40:
fw_file = SI2168_B40_FIRMWARE; fw_name = SI2168_B40_FIRMWARE;
break; break;
default: default:
dev_err(&client->dev, "unknown chip version Si21%d-%c%c%c\n", dev_err(&client->dev, "unknown chip version Si21%d-%c%c%c\n",
...@@ -425,12 +425,12 @@ static int si2168_init(struct dvb_frontend *fe) ...@@ -425,12 +425,12 @@ static int si2168_init(struct dvb_frontend *fe)
cmd.args[2], cmd.args[1], cmd.args[3], cmd.args[4]); cmd.args[2], cmd.args[1], cmd.args[3], cmd.args[4]);
/* request the firmware, this will block and timeout */ /* request the firmware, this will block and timeout */
ret = request_firmware(&fw, fw_file, &client->dev); ret = request_firmware(&fw, fw_name, &client->dev);
if (ret) { if (ret) {
/* fallback mechanism to handle old name for Si2168 B40 fw */ /* fallback mechanism to handle old name for Si2168 B40 fw */
if (chip_id == SI2168_B40) { if (chip_id == SI2168_B40) {
fw_file = SI2168_B40_FIRMWARE_FALLBACK; fw_name = SI2168_B40_FIRMWARE_FALLBACK;
ret = request_firmware(&fw, fw_file, &client->dev); ret = request_firmware(&fw, fw_name, &client->dev);
} }
if (ret == 0) { if (ret == 0) {
...@@ -440,13 +440,13 @@ static int si2168_init(struct dvb_frontend *fe) ...@@ -440,13 +440,13 @@ static int si2168_init(struct dvb_frontend *fe)
} else { } else {
dev_err(&client->dev, dev_err(&client->dev,
"firmware file '%s' not found\n", "firmware file '%s' not found\n",
fw_file); fw_name);
goto err_release_firmware; goto err_release_firmware;
} }
} }
dev_info(&client->dev, "downloading firmware from file '%s'\n", dev_info(&client->dev, "downloading firmware from file '%s'\n",
fw_file); fw_name);
if ((fw->size % 17 == 0) && (fw->data[0] > 5)) { if ((fw->size % 17 == 0) && (fw->data[0] > 5)) {
/* firmware is in the new format */ /* firmware is in the new format */
......
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