[media] si2157: 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 is. Signed-off-by: Antti Palosaari <crope@iki.fi> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
This commit is contained in:
parent
6c3725332d
commit
08272db62a
|
@ -82,7 +82,7 @@ static int si2157_init(struct dvb_frontend *fe)
|
||||||
int ret, len, remaining;
|
int ret, len, remaining;
|
||||||
struct si2157_cmd cmd;
|
struct si2157_cmd cmd;
|
||||||
const struct firmware *fw;
|
const struct firmware *fw;
|
||||||
u8 *fw_file;
|
const char *fw_name;
|
||||||
unsigned int chip_id;
|
unsigned int chip_id;
|
||||||
|
|
||||||
dev_dbg(&client->dev, "\n");
|
dev_dbg(&client->dev, "\n");
|
||||||
|
@ -123,12 +123,12 @@ static int si2157_init(struct dvb_frontend *fe)
|
||||||
switch (chip_id) {
|
switch (chip_id) {
|
||||||
case SI2158_A20:
|
case SI2158_A20:
|
||||||
case SI2148_A20:
|
case SI2148_A20:
|
||||||
fw_file = SI2158_A20_FIRMWARE;
|
fw_name = SI2158_A20_FIRMWARE;
|
||||||
break;
|
break;
|
||||||
case SI2157_A30:
|
case SI2157_A30:
|
||||||
case SI2147_A30:
|
case SI2147_A30:
|
||||||
case SI2146_A10:
|
case SI2146_A10:
|
||||||
fw_file = NULL;
|
fw_name = NULL;
|
||||||
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",
|
||||||
|
@ -141,27 +141,27 @@ static int si2157_init(struct dvb_frontend *fe)
|
||||||
dev_info(&client->dev, "found a 'Silicon Labs Si21%d-%c%c%c'\n",
|
dev_info(&client->dev, "found a 'Silicon Labs Si21%d-%c%c%c'\n",
|
||||||
cmd.args[2], cmd.args[1], cmd.args[3], cmd.args[4]);
|
cmd.args[2], cmd.args[1], cmd.args[3], cmd.args[4]);
|
||||||
|
|
||||||
if (fw_file == NULL)
|
if (fw_name == NULL)
|
||||||
goto skip_fw_download;
|
goto skip_fw_download;
|
||||||
|
|
||||||
/* 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) {
|
||||||
dev_err(&client->dev, "firmware file '%s' not found\n",
|
dev_err(&client->dev, "firmware file '%s' not found\n",
|
||||||
fw_file);
|
fw_name);
|
||||||
goto err;
|
goto err;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* firmware should be n chunks of 17 bytes */
|
/* firmware should be n chunks of 17 bytes */
|
||||||
if (fw->size % 17 != 0) {
|
if (fw->size % 17 != 0) {
|
||||||
dev_err(&client->dev, "firmware file '%s' is invalid\n",
|
dev_err(&client->dev, "firmware file '%s' is invalid\n",
|
||||||
fw_file);
|
fw_name);
|
||||||
ret = -EINVAL;
|
ret = -EINVAL;
|
||||||
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);
|
||||||
|
|
||||||
for (remaining = fw->size; remaining > 0; remaining -= 17) {
|
for (remaining = fw->size; remaining > 0; remaining -= 17) {
|
||||||
len = fw->data[fw->size - remaining];
|
len = fw->data[fw->size - remaining];
|
||||||
|
|
Loading…
Reference in New Issue