summaryrefslogtreecommitdiff
path: root/drivers/media/v4l2-core
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@redhat.com>2013-03-25 09:55:58 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2013-03-25 15:10:42 -0300
commit6f8ca0b541c8bb542edb02dad68bd723625132e7 (patch)
tree4c57b5ab07d79da0355bf4d35d205f38d8557ad7 /drivers/media/v4l2-core
parent383730c36419b85233fe2b5081253a2181160d17 (diff)
[media] tuner-core: Remove the now uneeded checks at fe_has_signal/get_afc
Now that those functions are only used when the corresponding function calls are defined, we don't need to check if those function calls are present at the structure before using it. Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/v4l2-core')
-rw-r--r--drivers/media/v4l2-core/tuner-core.c16
1 files changed, 6 insertions, 10 deletions
diff --git a/drivers/media/v4l2-core/tuner-core.c b/drivers/media/v4l2-core/tuner-core.c
index 5e18f440180..f1e8b402b7c 100644
--- a/drivers/media/v4l2-core/tuner-core.c
+++ b/drivers/media/v4l2-core/tuner-core.c
@@ -222,8 +222,7 @@ static int fe_has_signal(struct dvb_frontend *fe)
{
u16 strength = 0;
- if (fe->ops.tuner_ops.get_rf_strength)
- fe->ops.tuner_ops.get_rf_strength(fe, &strength);
+ fe->ops.tuner_ops.get_rf_strength(fe, &strength);
return strength;
}
@@ -232,8 +231,7 @@ static int fe_get_afc(struct dvb_frontend *fe)
{
s32 afc = 0;
- if (fe->ops.tuner_ops.get_afc)
- fe->ops.tuner_ops.get_afc(fe, &afc);
+ fe->ops.tuner_ops.get_afc(fe, &afc);
return afc;
}
@@ -256,8 +254,6 @@ static void tuner_status(struct dvb_frontend *fe);
static const struct analog_demod_ops tuner_analog_ops = {
.set_params = fe_set_params,
.standby = fe_standby,
- .has_signal = fe_has_signal,
- .get_afc = fe_get_afc,
.set_config = fe_set_config,
.tuner_status = tuner_status
};
@@ -453,10 +449,10 @@ static void set_type(struct i2c_client *c, unsigned int type,
memcpy(analog_ops, &tuner_analog_ops,
sizeof(struct analog_demod_ops));
- if (fe_tuner_ops->get_rf_strength == NULL)
- analog_ops->has_signal = NULL;
- if (fe_tuner_ops->get_afc == NULL)
- analog_ops->get_afc = NULL;
+ if (fe_tuner_ops->get_rf_strength)
+ analog_ops->has_signal = fe_has_signal;
+ if (fe_tuner_ops->get_afc)
+ analog_ops->get_afc = fe_get_afc;
} else {
t->name = analog_ops->info.name;