mirror of
https://github.com/rd-stuffs/msm-4.14.git
synced 2025-02-20 11:45:48 +08:00
V4L/DVB (9923): xc5000: remove init_fw option
The init_fw option was broken for the HVR-950q because we would call the reset callback inside of dvb_attach() and the callback had not been setup yet. Michael Krufky (who added the init_fw feature) says it's no longer required, so just remove the option completely. Thanks to user Zzeiss from #linuxtv chat for reporting the issue and Michael Krufky <mkrufky@linuxtv.org> for proposing the fix. Signed-off-by: Devin Heitmueller <dheitmueller@linuxtv.org> Acked-by: Michael Krufky <mkrufky@linuxtv.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
This commit is contained in:
parent
c665f4dd99
commit
d553cc958a
@ -36,10 +36,6 @@ static int debug;
|
|||||||
module_param(debug, int, 0644);
|
module_param(debug, int, 0644);
|
||||||
MODULE_PARM_DESC(debug, "Turn on/off debugging (default:off).");
|
MODULE_PARM_DESC(debug, "Turn on/off debugging (default:off).");
|
||||||
|
|
||||||
static int xc5000_load_fw_on_attach;
|
|
||||||
module_param_named(init_fw, xc5000_load_fw_on_attach, int, 0644);
|
|
||||||
MODULE_PARM_DESC(init_fw, "Load firmware during driver initialization.");
|
|
||||||
|
|
||||||
static DEFINE_MUTEX(xc5000_list_mutex);
|
static DEFINE_MUTEX(xc5000_list_mutex);
|
||||||
static LIST_HEAD(hybrid_tuner_instance_list);
|
static LIST_HEAD(hybrid_tuner_instance_list);
|
||||||
|
|
||||||
@ -1017,9 +1013,6 @@ struct dvb_frontend *xc5000_attach(struct dvb_frontend *fe,
|
|||||||
memcpy(&fe->ops.tuner_ops, &xc5000_tuner_ops,
|
memcpy(&fe->ops.tuner_ops, &xc5000_tuner_ops,
|
||||||
sizeof(struct dvb_tuner_ops));
|
sizeof(struct dvb_tuner_ops));
|
||||||
|
|
||||||
if (xc5000_load_fw_on_attach)
|
|
||||||
xc5000_init(fe);
|
|
||||||
|
|
||||||
return fe;
|
return fe;
|
||||||
fail:
|
fail:
|
||||||
mutex_unlock(&xc5000_list_mutex);
|
mutex_unlock(&xc5000_list_mutex);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user