[media] dvb-frontends/cxd2841er: optionally tune earlier in set_frontend()
authorDaniel Scheller <d.scheller@gmx.net>
Sun, 9 Apr 2017 19:38:20 +0000 (16:38 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 20 Jun 2017 12:54:17 +0000 (09:54 -0300)
When AUTO_IFHZ is set and the tuner is supposed to provide proper IF speed
values, it should be possible to have the tuner setup take place before
the demod is configured, else the demod might be configured with either
wrong (old), or even no values at all, which obviously will cause issues.
To set this behaviour in the most flexible way, this is done with a
separate flag instead of making this depend on AUTO_IFHZ.

It should be evaluated if tuning shouldn't take place earlier in all cases
and hardware constellations.

Signed-off-by: Daniel Scheller <d.scheller@gmx.net>
Acked-by: Abylay Ospan <aospan@netup.ru>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/dvb-frontends/cxd2841er.c
drivers/media/dvb-frontends/cxd2841er.h

index 3644a32772323cd0a2902c9f30bd706de7c58cf7..61f74fea1aa6d8e4a001a9e37b67117804ca159b 100644 (file)
@@ -3306,6 +3306,10 @@ static int cxd2841er_set_frontend_s(struct dvb_frontend *fe)
                __func__,
                (p->delivery_system == SYS_DVBS ? "DVB-S" : "DVB-S2"),
                 p->frequency, symbol_rate, priv->xtal);
+
+       if (priv->flags & CXD2841ER_EARLY_TUNE)
+               cxd2841er_tuner_set(fe);
+
        switch (priv->state) {
        case STATE_SLEEP_S:
                ret = cxd2841er_sleep_s_to_active_s(
@@ -3325,7 +3329,8 @@ static int cxd2841er_set_frontend_s(struct dvb_frontend *fe)
                goto done;
        }
 
-       cxd2841er_tuner_set(fe);
+       if (!(priv->flags & CXD2841ER_EARLY_TUNE))
+               cxd2841er_tuner_set(fe);
 
        cxd2841er_tune_done(priv);
        timeout = ((3000000 + (symbol_rate - 1)) / symbol_rate) + 150;
@@ -3365,6 +3370,10 @@ static int cxd2841er_set_frontend_tc(struct dvb_frontend *fe)
 
        dev_dbg(&priv->i2c->dev, "%s() delivery_system=%d bandwidth_hz=%d\n",
                 __func__, p->delivery_system, p->bandwidth_hz);
+
+       if (priv->flags & CXD2841ER_EARLY_TUNE)
+               cxd2841er_tuner_set(fe);
+
        if (p->delivery_system == SYS_DVBT) {
                priv->system = SYS_DVBT;
                switch (priv->state) {
@@ -3447,7 +3456,8 @@ static int cxd2841er_set_frontend_tc(struct dvb_frontend *fe)
        if (ret)
                goto done;
 
-       cxd2841er_tuner_set(fe);
+       if (!(priv->flags & CXD2841ER_EARLY_TUNE))
+               cxd2841er_tuner_set(fe);
 
        cxd2841er_tune_done(priv);
        timeout = 2500;
index 90ced9733ab9a43daa78420939f28c2e37817b18..061e5519ffc92d44a2c60ee7fa17e51c157c3e72 100644 (file)
@@ -28,6 +28,7 @@
 #define CXD2841ER_AUTO_IFHZ    2       /* bit 1 */
 #define CXD2841ER_TS_SERIAL    4       /* bit 2 */
 #define CXD2841ER_ASCOT                8       /* bit 3 */
+#define CXD2841ER_EARLY_TUNE   16      /* bit 4 */
 
 enum cxd2841er_xtal {
        SONY_XTAL_20500, /* 20.5 MHz */