i2c bus is already needed when the frontend is probed, so init it already in 
it930x_frontend_attach.That prevents errors like
si2168: probe of 6-0067 failed with error -5

Andreas, can I have your Signed-off-by?


From: Andreas Kemnade <andr...@kemnade.info>
Signed-off-by: Jose Alberto Reguero <jose.alberto.regu...@gmail.com>

diff -upr linux/drivers/media/usb/dvb-usb-v2/af9035.c 
linux.new/drivers/media/usb/dvb-usb-v2/af9035.c
--- linux/drivers/media/usb/dvb-usb-v2/af9035.c 2018-09-12 07:40:12.000000000 
+0200
+++ linux.new/drivers/media/usb/dvb-usb-v2/af9035.c     2019-02-20 
16:45:17.467869437 +0100
@@ -1218,6 +1218,48 @@ static int it930x_frontend_attach(struct
 
        dev_dbg(&intf->dev, "adap->id=%d\n", adap->id);
 
+       /* I2C master bus 2 clock speed 300k */
+       ret = af9035_wr_reg(d, 0x00f6a7, 0x07);
+       if (ret < 0)
+               goto err;
+
+       /* I2C master bus 1,3 clock speed 300k */
+       ret = af9035_wr_reg(d, 0x00f103, 0x07);
+       if (ret < 0)
+               goto err;
+
+       /* set gpio11 low */
+       ret = af9035_wr_reg_mask(d, 0xd8d4, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
+       ret = af9035_wr_reg_mask(d, 0xd8d5, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
+       ret = af9035_wr_reg_mask(d, 0xd8d3, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
+       /* Tuner enable using gpiot2_en, gpiot2_on and gpiot2_o (reset) */
+       ret = af9035_wr_reg_mask(d, 0xd8b8, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
+       ret = af9035_wr_reg_mask(d, 0xd8b9, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
+       ret = af9035_wr_reg_mask(d, 0xd8b7, 0x00, 0x01);
+       if (ret < 0)
+               goto err;
+
+       msleep(200);
+
+       ret = af9035_wr_reg_mask(d, 0xd8b7, 0x01, 0x01);
+       if (ret < 0)
+               goto err;
+
        memset(&si2168_config, 0, sizeof(si2168_config));
        si2168_config.i2c_adapter = &adapter;
        si2168_config.fe = &adap->fe[0];
@@ -1575,48 +1617,6 @@ static int it930x_tuner_attach(struct dv
 
        dev_dbg(&intf->dev, "adap->id=%d\n", adap->id);
 
-       /* I2C master bus 2 clock speed 300k */
-       ret = af9035_wr_reg(d, 0x00f6a7, 0x07);
-       if (ret < 0)
-               goto err;
-
-       /* I2C master bus 1,3 clock speed 300k */
-       ret = af9035_wr_reg(d, 0x00f103, 0x07);
-       if (ret < 0)
-               goto err;
-
-       /* set gpio11 low */
-       ret = af9035_wr_reg_mask(d, 0xd8d4, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
-       ret = af9035_wr_reg_mask(d, 0xd8d5, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
-       ret = af9035_wr_reg_mask(d, 0xd8d3, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
-       /* Tuner enable using gpiot2_en, gpiot2_on and gpiot2_o (reset) */
-       ret = af9035_wr_reg_mask(d, 0xd8b8, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
-       ret = af9035_wr_reg_mask(d, 0xd8b9, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
-       ret = af9035_wr_reg_mask(d, 0xd8b7, 0x00, 0x01);
-       if (ret < 0)
-               goto err;
-
-       msleep(200);
-
-       ret = af9035_wr_reg_mask(d, 0xd8b7, 0x01, 0x01);
-       if (ret < 0)
-               goto err;
-
        memset(&si2157_config, 0, sizeof(si2157_config));
        si2157_config.fe = adap->fe[0];
        si2157_config.if_port = 1;

-- 
Enviado desde mi dispositivo Android con K-9 Mail. Por favor, disculpa mi 
brevedad.

Reply via email to