media: dvb_frontend: ensure that the step is ok for both FE and tuner
authorMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Thu, 5 Jul 2018 22:59:37 +0000 (18:59 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Thu, 2 Aug 2018 22:11:46 +0000 (18:11 -0400)
The frequency step should take into account the tuner step,
as, if tuner step is bigger than frontend step, the zigzag
algorithm won't be doing the right thing, as it will be
tuning multiple times at the same frequency.

Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/dvb-core/dvb_frontend.c

index fe0fae4..c4e7ebf 100644 (file)
@@ -939,7 +939,10 @@ static void dvb_frontend_get_frequency_limits(struct dvb_frontend *fe,
 static u32 dvb_frontend_get_stepsize(struct dvb_frontend *fe)
 {
        struct dtv_frontend_properties *c = &fe->dtv_property_cache;
-       u32 step = fe->ops.info.frequency_stepsize_hz;
+       u32 fe_step = fe->ops.info.frequency_stepsize_hz;
+       u32 tuner_step = fe->ops.tuner_ops.info.frequency_step_hz;
+       u32 step = max(fe_step, tuner_step);
+
        switch (c->delivery_system) {
        case SYS_DVBS:
        case SYS_DVBS2: