V4L/DVB (12903): DiB8000: fix channel search parameter initialization

This patch is fixing the initialization of the channel search parameters.

Signed-off-by: Olivier Grenie <olivier.grenie@dibcom.fr>
Signed-off-by: Patrick Boettcher <pboettcher@dibcom.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

diff --git a/drivers/media/dvb/frontends/dib8000.c b/drivers/media/dvb/frontends/dib8000.c
index 99ee6b0..852c790 100644 (file)
--- a/drivers/media/dvb/frontends/dib8000.c
+++ b/drivers/media/dvb/frontends/dib8000.c
@@ -22,7 +22,7 @@
 
 #define FE_CALLBACK_TIME_NEVER 0xffffffff
 
-static int debug = 0;
+static int debug;
 module_param(debug, int, 0644);
 MODULE_PARM_DESC(debug, "turn on debugging (default: 0)");
 
@@ -1671,10 +1671,6 @@ static int dib8000_autosearch_start(struct dvb_frontend *fe)
 
        int slist = 0;
 
-       state->fe.dtv_property_cache.transmission_mode = TRANSMISSION_MODE_8K;
-       state->fe.dtv_property_cache.guard_interval = GUARD_INTERVAL_1_8;
-       //state->fe.dtv_property_cache.isdbt_sb_mode                = 0;
-       //state->fe.dtv_property_cache.isdbt_partial_reception      = 0;
        state->fe.dtv_property_cache.inversion = 0;
        if (!state->fe.dtv_property_cache.isdbt_sb_mode)
                state->fe.dtv_property_cache.layer[0].segment_count = 13;
@@ -1684,6 +1680,8 @@ static int dib8000_autosearch_start(struct dvb_frontend *fe)
 
        //choose the right list, in sb, always do everything
        if (state->fe.dtv_property_cache.isdbt_sb_mode) {
+               state->fe.dtv_property_cache.transmission_mode = TRANSMISSION_MODE_8K;
+               state->fe.dtv_property_cache.guard_interval = GUARD_INTERVAL_1_8;
                slist = 7;
                dib8000_write_word(state, 0, (dib8000_read_word(state, 0) & 0x9fff) | (1 << 13));
        } else {
@@ -1691,22 +1689,21 @@ static int dib8000_autosearch_start(struct dvb_frontend *fe)
                        if (state->fe.dtv_property_cache.transmission_mode == TRANSMISSION_MODE_AUTO) {
                                slist = 7;
                                dib8000_write_word(state, 0, (dib8000_read_word(state, 0) & 0x9fff) | (1 << 13));       // P_mode = 1 to have autosearch start ok with mode2
-                       } else {
+                       } else
                                slist = 3;
-                               state->fe.dtv_property_cache.transmission_mode = state->fe.dtv_property_cache.transmission_mode;
-                       }
                } else {
                        if (state->fe.dtv_property_cache.transmission_mode == TRANSMISSION_MODE_AUTO) {
                                slist = 2;
-                               state->fe.dtv_property_cache.guard_interval = state->fe.dtv_property_cache.guard_interval;
                                dib8000_write_word(state, 0, (dib8000_read_word(state, 0) & 0x9fff) | (1 << 13));       // P_mode = 1
-                       } else {
+                       } else
                                slist = 0;
-                               state->fe.dtv_property_cache.transmission_mode = state->fe.dtv_property_cache.transmission_mode;
-                               state->fe.dtv_property_cache.guard_interval = state->fe.dtv_property_cache.guard_interval;
-                       }
                }
 
+               if (state->fe.dtv_property_cache.transmission_mode == TRANSMISSION_MODE_AUTO)
+                       state->fe.dtv_property_cache.transmission_mode = TRANSMISSION_MODE_8K;
+               if (state->fe.dtv_property_cache.guard_interval == GUARD_INTERVAL_AUTO)
+                       state->fe.dtv_property_cache.guard_interval = GUARD_INTERVAL_1_8;
+
                dprintk("using list for autosearch : %d", slist);
                dib8000_set_channel(state, (unsigned char)slist, 1);
                //dib8000_write_word(state, 0, (dib8000_read_word(state, 0) & 0x9fff) | (1 << 13));  // P_mode = 1
@@ -1733,6 +1730,7 @@ static int dib8000_autosearch_start(struct dvb_frontend *fe)
                dib8000_write_word(state, 0, (u16) ((1 << 15) | value));
                dib8000_read_word(state, 1284); // reset the INT. n_irq_pending
                dib8000_write_word(state, 0, (u16) value);
+
        }
 
        return 0;