Commit 92b75ab0 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

V4L/DVB (7605): tuner-xc3028: Avoids too much firmware reloads

xc3028_sleep function is being used with a different meaning. This
should be called only before doing S1/S3 sleep.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@infradead.org>
parent 102a0b08
...@@ -235,6 +235,7 @@ static v4l2_std_id parse_audio_std_option(void) ...@@ -235,6 +235,7 @@ static v4l2_std_id parse_audio_std_option(void)
static void free_firmware(struct xc2028_data *priv) static void free_firmware(struct xc2028_data *priv)
{ {
int i; int i;
tuner_dbg("%s called\n", __func__);
if (!priv->firm) if (!priv->firm)
return; return;
...@@ -1050,27 +1051,6 @@ static int xc2028_set_params(struct dvb_frontend *fe, ...@@ -1050,27 +1051,6 @@ static int xc2028_set_params(struct dvb_frontend *fe,
T_DIGITAL_TV, type, 0, demod); T_DIGITAL_TV, type, 0, demod);
} }
static int xc2028_sleep(struct dvb_frontend *fe)
{
struct xc2028_data *priv = fe->tuner_priv;
int rc = 0;
tuner_dbg("%s called\n", __func__);
mutex_lock(&priv->lock);
if (priv->firm_version < 0x0202)
rc = send_seq(priv, {0x00, 0x08, 0x00, 0x00});
else
rc = send_seq(priv, {0x80, 0x08, 0x00, 0x00});
priv->cur_fw.type = 0; /* need firmware reload */
mutex_unlock(&priv->lock);
return rc;
}
static int xc2028_dvb_release(struct dvb_frontend *fe) static int xc2028_dvb_release(struct dvb_frontend *fe)
{ {
...@@ -1118,21 +1098,21 @@ static int xc2028_set_config(struct dvb_frontend *fe, void *priv_cfg) ...@@ -1118,21 +1098,21 @@ static int xc2028_set_config(struct dvb_frontend *fe, void *priv_cfg)
mutex_lock(&priv->lock); mutex_lock(&priv->lock);
kfree(priv->ctrl.fname);
free_firmware(priv);
memcpy(&priv->ctrl, p, sizeof(priv->ctrl)); memcpy(&priv->ctrl, p, sizeof(priv->ctrl));
priv->ctrl.fname = NULL; if (priv->ctrl.max_len < 9)
priv->ctrl.max_len = 13;
if (p->fname) { if (p->fname) {
if (priv->ctrl.fname && strcmp(p->fname, priv->ctrl.fname)) {
kfree(priv->ctrl.fname);
free_firmware(priv);
}
priv->ctrl.fname = kstrdup(p->fname, GFP_KERNEL); priv->ctrl.fname = kstrdup(p->fname, GFP_KERNEL);
if (priv->ctrl.fname == NULL) if (priv->ctrl.fname == NULL)
rc = -ENOMEM; rc = -ENOMEM;
} }
if (priv->ctrl.max_len < 9)
priv->ctrl.max_len = 13;
mutex_unlock(&priv->lock); mutex_unlock(&priv->lock);
return rc; return rc;
...@@ -1152,8 +1132,6 @@ static const struct dvb_tuner_ops xc2028_dvb_tuner_ops = { ...@@ -1152,8 +1132,6 @@ static const struct dvb_tuner_ops xc2028_dvb_tuner_ops = {
.get_frequency = xc2028_get_frequency, .get_frequency = xc2028_get_frequency,
.get_rf_strength = xc2028_signal, .get_rf_strength = xc2028_signal,
.set_params = xc2028_set_params, .set_params = xc2028_set_params,
.sleep = xc2028_sleep,
}; };
struct dvb_frontend *xc2028_attach(struct dvb_frontend *fe, struct dvb_frontend *xc2028_attach(struct dvb_frontend *fe,
......
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