summaryrefslogtreecommitdiff
path: root/drivers/media/tuners
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-12-08 10:46:59 +0100
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-12-14 16:19:04 +0100
commit7c2d8ee486b9ebd462dcd0aea1969758522f4c3a (patch)
treeffa919f38fad61de1027fa930741d3b87d60df4f /drivers/media/tuners
parent48dde945e7f856f622bd68c076f8a6b5d524d19e (diff)
media: si2157: rework the firmware load logic
Loading a firmware file should not be mandatory, as devices could work with an eeprom firmware, if available. Yet, using the eeprom firmware could lead into unpredictable results, so the best is to warn about that. Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/tuners')
-rw-r--r--drivers/media/tuners/si2157.c31
1 files changed, 18 insertions, 13 deletions
diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c
index ed28672c060d..5f4ae8593864 100644
--- a/drivers/media/tuners/si2157.c
+++ b/drivers/media/tuners/si2157.c
@@ -129,8 +129,9 @@ static int si2157_init(struct dvb_frontend *fe)
struct dtv_frontend_properties *c = &fe->dtv_property_cache;
struct i2c_client *client = fe->tuner_priv;
struct si2157_dev *dev = i2c_get_clientdata(client);
+ bool warn_firmware_not_loaded = false;
unsigned int chip_id, xtal_trim;
- unsigned int fw_required;
+ bool fw_required = true;
struct si2157_cmd cmd;
const char *fw_name;
int ret;
@@ -199,10 +200,6 @@ static int si2157_init(struct dvb_frontend *fe)
#define SI2146_A10 ('A' << 24 | 46 << 16 | '1' << 8 | '0' << 0)
#define SI2141_A10 ('A' << 24 | 41 << 16 | '1' << 8 | '0' << 0)
- /* assume firmware is required, unless verified not to be */
- /* only the SI2157_A30 has been verified not to yet */
- fw_required = true;
-
switch (chip_id) {
case SI2158_A20:
case SI2148_A20:
@@ -212,9 +209,8 @@ static int si2157_init(struct dvb_frontend *fe)
fw_name = SI2141_A10_FIRMWARE;
break;
case SI2157_A30:
- fw_name = SI2157_A30_FIRMWARE;
fw_required = false;
- break;
+ fallthrough;
case SI2177_A30:
fw_name = SI2157_A30_FIRMWARE;
break;
@@ -237,12 +233,11 @@ static int si2157_init(struct dvb_frontend *fe)
goto skip_fw_download;
ret = si2157_load_firmware(fe, fw_name);
- if (ret) {
- if (!fw_required)
- goto skip_fw_download;
-
- dev_err(&client->dev, "firmware file '%s' not found\n",
- fw_name);
+ if (fw_required && ret == -ENOENT)
+ warn_firmware_not_loaded = true;
+ else if (ret < 0) {
+ dev_err(&client->dev, "error %d when loading firmware file '%s'\n",
+ ret, fw_name);
goto err;
}
@@ -263,6 +258,11 @@ skip_fw_download:
if (ret)
goto err;
+ if (warn_firmware_not_loaded) {
+ dev_warn(&client->dev, "firmware file '%s' not found. Using firmware from eeprom.\n",
+ fw_name);
+ warn_firmware_not_loaded = false;
+ }
dev_info(&client->dev, "firmware version: %c.%c.%d\n",
cmd.args[6], cmd.args[7], cmd.args[8]);
@@ -298,6 +298,11 @@ warm:
return 0;
err:
+ if (warn_firmware_not_loaded)
+ dev_err(&client->dev,
+ "firmware file '%s' not found. Can't continue without a firmware.\n",
+ fw_name);
+
dev_dbg(&client->dev, "failed=%d\n", ret);
return ret;
}