Commit 1f5ef197 authored by Michael Krufky's avatar Michael Krufky Committed by Mauro Carvalho Chehab

V4L/DVB (6141): tuner: use get_rf_strength instead of get_status to determine signal strength

Signed-off-by: default avatarMichael Krufky <mkrufky@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@infradead.org>
parent 3782e050
...@@ -110,6 +110,17 @@ static void fe_standby(struct tuner *t) ...@@ -110,6 +110,17 @@ static void fe_standby(struct tuner *t)
fe_tuner_ops->sleep(&t->fe); fe_tuner_ops->sleep(&t->fe);
} }
static int fe_has_signal(struct tuner *t)
{
struct dvb_tuner_ops *fe_tuner_ops = &t->fe.ops.tuner_ops;
u16 strength;
if (fe_tuner_ops->get_rf_strength)
fe_tuner_ops->get_rf_strength(&t->fe, &strength);
return strength;
}
/* Set tuner frequency, freq in Units of 62.5kHz = 1/16MHz */ /* Set tuner frequency, freq in Units of 62.5kHz = 1/16MHz */
static void set_tv_freq(struct i2c_client *c, unsigned int freq) static void set_tv_freq(struct i2c_client *c, unsigned int freq)
{ {
...@@ -326,6 +337,7 @@ static void set_type(struct i2c_client *c, unsigned int type, ...@@ -326,6 +337,7 @@ static void set_type(struct i2c_client *c, unsigned int type,
t->ops.set_radio_freq = fe_set_freq; t->ops.set_radio_freq = fe_set_freq;
t->ops.standby = fe_standby; t->ops.standby = fe_standby;
t->ops.release = fe_release; t->ops.release = fe_release;
t->ops.has_signal = fe_has_signal;
} }
tuner_info("type set to %s\n", t->i2c.name); tuner_info("type set to %s\n", t->i2c.name);
...@@ -807,12 +819,10 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg) ...@@ -807,12 +819,10 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg)
u32 tuner_status; u32 tuner_status;
fe_tuner_ops->get_status(&t->fe, &tuner_status); fe_tuner_ops->get_status(&t->fe, &tuner_status);
if (tuner_status & TUNER_STATUS_STEREO) if (tuner_status & TUNER_STATUS_STEREO)
vt->flags |= VIDEO_TUNER_STEREO_ON; vt->flags |= VIDEO_TUNER_STEREO_ON;
else else
vt->flags &= ~VIDEO_TUNER_STEREO_ON; vt->flags &= ~VIDEO_TUNER_STEREO_ON;
vt->signal = tuner_status & TUNER_STATUS_LOCKED
? 65535 : 0;
} else { } else {
if (t->ops.is_stereo) { if (t->ops.is_stereo) {
if (t->ops.is_stereo(t)) if (t->ops.is_stereo(t))
...@@ -822,9 +832,10 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg) ...@@ -822,9 +832,10 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg)
vt->flags &= vt->flags &=
~VIDEO_TUNER_STEREO_ON; ~VIDEO_TUNER_STEREO_ON;
} }
if (t->ops.has_signal)
vt->signal = t->ops.has_signal(t);
} }
if (t->ops.has_signal)
vt->signal = t->ops.has_signal(t);
vt->flags |= VIDEO_TUNER_LOW; /* Allow freqs at 62.5 Hz */ vt->flags |= VIDEO_TUNER_LOW; /* Allow freqs at 62.5 Hz */
vt->rangelow = radio_range[0] * 16000; vt->rangelow = radio_range[0] * 16000;
...@@ -948,15 +959,14 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg) ...@@ -948,15 +959,14 @@ static int tuner_command(struct i2c_client *client, unsigned int cmd, void *arg)
fe_tuner_ops->get_status(&t->fe, &tuner_status); fe_tuner_ops->get_status(&t->fe, &tuner_status);
tuner->rxsubchans = (tuner_status & TUNER_STATUS_STEREO) ? tuner->rxsubchans = (tuner_status & TUNER_STATUS_STEREO) ?
V4L2_TUNER_SUB_STEREO : V4L2_TUNER_SUB_MONO; V4L2_TUNER_SUB_STEREO : V4L2_TUNER_SUB_MONO;
tuner->signal = tuner_status & TUNER_STATUS_LOCKED ? 65535 : 0;
} else { } else {
if (t->ops.is_stereo) { if (t->ops.is_stereo) {
tuner->rxsubchans = t->ops.is_stereo(t) ? tuner->rxsubchans = t->ops.is_stereo(t) ?
V4L2_TUNER_SUB_STEREO : V4L2_TUNER_SUB_MONO; V4L2_TUNER_SUB_STEREO : V4L2_TUNER_SUB_MONO;
} }
if (t->ops.has_signal)
tuner->signal = t->ops.has_signal(t);
} }
if (t->ops.has_signal)
tuner->signal = t->ops.has_signal(t);
tuner->capability |= tuner->capability |=
V4L2_TUNER_CAP_LOW | V4L2_TUNER_CAP_STEREO; V4L2_TUNER_CAP_LOW | V4L2_TUNER_CAP_STEREO;
tuner->audmode = t->audmode; tuner->audmode = t->audmode;
......
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