V4L/DVB (4322): Fix dvb-pll autoprobing
authorAndrew de Quincey <adq_dvb@lidskialf.net>
Tue, 18 Jul 2006 19:37:13 +0000 (16:37 -0300)
committerMauro Carvalho Chehab <mchehab@infradead.org>
Sat, 29 Jul 2006 20:22:26 +0000 (17:22 -0300)
Trent Piepho pointed out that the pll test i2c transmission is slightly
wrong; it was transmitting a zero length message, and then reading from the
PLL. This was wrong; it should only be transmitting a single read i2c message.

Signed-off-by: Andrew de Quincey <adq_dvb@lidskialf.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
drivers/media/dvb/frontends/dvb-pll.c

index 4c8a506479b4f1eb7dc11a8eb5be306b2f13559b..2be33f27c69fbfb7944140220eb34df815f15034 100644 (file)
@@ -614,8 +614,7 @@ static struct dvb_tuner_ops dvb_pll_tuner_ops = {
 int dvb_pll_attach(struct dvb_frontend *fe, int pll_addr, struct i2c_adapter *i2c, struct dvb_pll_desc *desc)
 {
        u8 b1 [] = { 0 };
-       struct i2c_msg msg [] = { { .addr = pll_addr, .flags = 0, .buf = NULL, .len = 0 },
-                                 { .addr = pll_addr, .flags = I2C_M_RD, .buf = b1, .len = 1 } };
+       struct i2c_msg msg = { .addr = pll_addr, .flags = I2C_M_RD, .buf = b1, .len = 1 };
        struct dvb_pll_priv *priv = NULL;
        int ret;
 
@@ -623,8 +622,8 @@ int dvb_pll_attach(struct dvb_frontend *fe, int pll_addr, struct i2c_adapter *i2
                if (fe->ops.i2c_gate_ctrl)
                        fe->ops.i2c_gate_ctrl(fe, 1);
 
-               ret = i2c_transfer (i2c, msg, 2);
-               if (ret != 2)
+               ret = i2c_transfer (i2c, &msg, 1);
+               if (ret != 1)
                        return -1;
                if (fe->ops.i2c_gate_ctrl)
                             fe->ops.i2c_gate_ctrl(fe, 0);