media: dvb-usb/friio, dvb-usb-v2/gl861: decompose friio and merge with gl861
authorAkihiro Tsukada <tskd08@gmail.com>
Sun, 10 Jun 2018 14:45:31 +0000 (10:45 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 30 Jul 2018 12:50:25 +0000 (08:50 -0400)
Friio device contains "gl861" bridge and "tc90522" demod,
for which the separate drivers are already in the kernel.
But friio driver was monolithic and did not use them,
practically copying those features.
This patch decomposes friio driver into sub drivers and
re-uses existing ones, thus reduces some code.

It adds some features to gl861,
to support the friio-specific init/config of the devices
and implement i2c communications to the tuner via demod
with USB vendor requests.

[mchehab+samsung@kernel.org: fix merge conflicts]
Signed-off-by: Akihiro Tsukada <tskd08@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/usb/dvb-usb-v2/Kconfig
drivers/media/usb/dvb-usb-v2/gl861.c
drivers/media/usb/dvb-usb-v2/gl861.h
drivers/media/usb/dvb-usb/Kconfig
drivers/media/usb/dvb-usb/Makefile

index 082b8d67244bf2c221b1dcdd77434382e868b01b..df4412245a8aeb8214d5d1f8b3760070e781d7ee 100644 (file)
@@ -96,10 +96,13 @@ config DVB_USB_GL861
        tristate "Genesys Logic GL861 USB2.0 support"
        depends on DVB_USB_V2
        select DVB_ZL10353 if MEDIA_SUBDRV_AUTOSELECT
+       select DVB_TC90522 if MEDIA_SUBDRV_AUTOSELECT
        select MEDIA_TUNER_QT1010 if MEDIA_SUBDRV_AUTOSELECT
+       select DVB_PLL if MEDIA_SUBDRV_AUTOSELECT
        help
          Say Y here to support the MSI Megasky 580 (55801) DVB-T USB2.0
-         receiver with USB ID 0db0:5581.
+         receiver with USB ID 0db0:5581, Friio White ISDB-T receiver
+         with USB ID 0x7a69:0001.
 
 config DVB_USB_LME2510
        tristate "LME DM04/QQBOX DVB-S USB2.0 support"
index fee4b30df77837cdb6b0bf648eac99d02c4d53f3..3338b21d8b25ca0fcb438f4d1eedce6c80c99f1a 100644 (file)
@@ -6,10 +6,14 @@
  *
  * see Documentation/media/dvb-drivers/dvb-usb.rst for more information
  */
+#include <linux/string.h>
+
 #include "gl861.h"
 
 #include "zl10353.h"
 #include "qt1010.h"
+#include "tc90522.h"
+#include "dvb-pll.h"
 
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
 
@@ -161,11 +165,478 @@ static struct dvb_usb_device_properties gl861_props = {
        }
 };
 
+
+/*
+ * For Friio
+ */
+
+struct friio_priv {
+       struct i2c_adapter *demod_sub_i2c;
+       struct i2c_client  *i2c_client_demod;
+       struct i2c_client  *i2c_client_tuner;
+       struct i2c_adapter tuner_adap;
+};
+
+struct friio_config {
+       struct i2c_board_info demod_info;
+       struct tc90522_config demod_cfg;
+
+       struct i2c_board_info tuner_info;
+       struct dvb_pll_config tuner_cfg;
+};
+
+static const struct friio_config friio_config = {
+       .demod_info = { I2C_BOARD_INFO(TC90522_I2C_DEV_TER, 0x18), },
+       .tuner_info = { I2C_BOARD_INFO("tua6034_friio", 0x60), },
+};
+
+/* For another type of I2C:
+ * message sent by a USB control-read/write transaction with data stage.
+ * Used in init/config of Friio.
+ */
+static int
+gl861_i2c_write_ex(struct dvb_usb_device *d, u8 addr, u8 *wbuf, u16 wlen)
+{
+       u8 *buf;
+       int ret;
+
+       buf = kmalloc(wlen, GFP_KERNEL);
+       if (!buf)
+               return -ENOMEM;
+
+       memcpy(buf, wbuf, wlen);
+       ret = usb_control_msg(d->udev, usb_sndctrlpipe(d->udev, 0),
+                                GL861_REQ_I2C_RAW, GL861_WRITE,
+                                addr << (8 + 1), 0x0100, buf, wlen, 2000);
+       kfree(buf);
+       return ret;
+}
+
+static int
+gl861_i2c_read_ex(struct dvb_usb_device *d, u8 addr, u8 *rbuf, u16 rlen)
+{
+       u8 *buf;
+       int ret;
+
+       buf = kmalloc(rlen, GFP_KERNEL);
+       if (!buf)
+               return -ENOMEM;
+
+       ret = usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0),
+                                GL861_REQ_I2C_READ, GL861_READ,
+                                addr << (8 + 1), 0x0100, buf, rlen, 2000);
+       if (ret > 0 && rlen > 0)
+               memcpy(buf, rbuf, rlen);
+       kfree(buf);
+       return ret;
+}
+
+/* For I2C transactions to the tuner of Friio (dvb_pll).
+ *
+ * Friio uses irregular USB encapsulation for tuner i2c transactions:
+ * write transacions are encapsulated with a different USB 'request' value.
+ *
+ * Although all transactions are sent via the demod(tc90522)
+ * and the demod provides an i2c adapter for them, it cannot be used in Friio
+ * since it assumes using the same parent adapter with the demod,
+ * which does not use the request value and uses same one for both read/write.
+ * So we define a dedicated i2c adapter here.
+ */
+
+static int
+friio_i2c_tuner_read(struct dvb_usb_device *d, struct i2c_msg *msg)
+{
+       struct friio_priv *priv;
+       u8 addr;
+
+       priv = d_to_priv(d);
+       addr = priv->i2c_client_demod->addr;
+       return gl861_i2c_read_ex(d, addr, msg->buf, msg->len);
+}
+
+static int
+friio_i2c_tuner_write(struct dvb_usb_device *d, struct i2c_msg *msg)
+{
+       u8 *buf;
+       int ret;
+       struct friio_priv *priv;
+
+       priv = d_to_priv(d);
+
+       if (msg->len < 1)
+               return -EINVAL;
+
+       buf = kmalloc(msg->len + 1, GFP_KERNEL);
+       if (!buf)
+               return -ENOMEM;
+       buf[0] = msg->addr << 1;
+       memcpy(buf + 1, msg->buf, msg->len);
+
+       ret = usb_control_msg(d->udev, usb_sndctrlpipe(d->udev, 0),
+                                GL861_REQ_I2C_RAW, GL861_WRITE,
+                                priv->i2c_client_demod->addr << (8 + 1),
+                                0xFE, buf, msg->len + 1, 2000);
+       kfree(buf);
+       return ret;
+}
+
+static int friio_tuner_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msg[],
+                               int num)
+{
+       struct dvb_usb_device *d = i2c_get_adapdata(adap);
+       int i;
+
+       if (num > 2)
+               return -EINVAL;
+
+       if (mutex_lock_interruptible(&d->i2c_mutex) < 0)
+               return -EAGAIN;
+
+       for (i = 0; i < num; i++) {
+               int ret;
+
+               if (msg[i].flags & I2C_M_RD)
+                       ret = friio_i2c_tuner_read(d, &msg[i]);
+               else
+                       ret = friio_i2c_tuner_write(d, &msg[i]);
+
+               if (ret < 0)
+                       break;
+
+               usleep_range(1000, 2000); /* avoid I2C errors */
+       }
+
+       mutex_unlock(&d->i2c_mutex);
+       return i;
+}
+
+static struct i2c_algorithm friio_tuner_i2c_algo = {
+       .master_xfer   = friio_tuner_i2c_xfer,
+       .functionality = gl861_i2c_func,
+};
+
+/* GPIO control in Friio */
+
+#define FRIIO_CTL_LNB (1 << 0)
+#define FRIIO_CTL_STROBE (1 << 1)
+#define FRIIO_CTL_CLK (1 << 2)
+#define FRIIO_CTL_LED (1 << 3)
+
+#define FRIIO_LED_RUNNING 0x6400ff64
+#define FRIIO_LED_STOPPED 0x96ff00ff
+
+/* control PIC16F676 attached to Friio */
+static int friio_ext_ctl(struct dvb_usb_device *d,
+                           u32 sat_color, int power_on)
+{
+       int i, ret;
+       struct i2c_msg msg;
+       u8 *buf;
+       u32 mask;
+       u8 power = (power_on) ? FRIIO_CTL_LNB : 0;
+
+       buf = kmalloc(2, GFP_KERNEL);
+       if (!buf)
+               return -ENOMEM;
+
+       msg.addr = 0x00;
+       msg.flags = 0;
+       msg.len = 2;
+       msg.buf = buf;
+       buf[0] = 0x00;
+
+       /* send 2bit header (&B10) */
+       buf[1] = power | FRIIO_CTL_LED | FRIIO_CTL_STROBE;
+       ret = i2c_transfer(&d->i2c_adap, &msg, 1);
+       buf[1] |= FRIIO_CTL_CLK;
+       ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+
+       buf[1] = power | FRIIO_CTL_STROBE;
+       ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+       buf[1] |= FRIIO_CTL_CLK;
+       ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+
+       /* send 32bit(satur, R, G, B) data in serial */
+       mask = 1 << 31;
+       for (i = 0; i < 32; i++) {
+               buf[1] = power | FRIIO_CTL_STROBE;
+               if (sat_color & mask)
+                       buf[1] |= FRIIO_CTL_LED;
+               ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+               buf[1] |= FRIIO_CTL_CLK;
+               ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+               mask >>= 1;
+       }
+
+       /* set the strobe off */
+       buf[1] = power;
+       ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+       buf[1] |= FRIIO_CTL_CLK;
+       ret += i2c_transfer(&d->i2c_adap, &msg, 1);
+
+       kfree(buf);
+       return (ret == 70) ? 0 : -EREMOTEIO;
+}
+
+/* init/config of gl861 for Friio */
+/* NOTE:
+ * This function cannot be moved to friio_init()/dvb_usbv2_init(),
+ * because the init defined here must be done before any activities like I2C,
+ * but friio_init() is called by dvb-usbv2 after {_frontend, _tuner}_attach(),
+ * where I2C communication is used.
+ * Thus this function is set to be called from _power_ctl().
+ *
+ * Since it will be called on the early init stage
+ * where the i2c adapter is not initialized yet,
+ * we cannot use i2c_transfer() here.
+ */
+static int friio_reset(struct dvb_usb_device *d)
+{
+       int i, ret;
+       u8 wbuf[2], rbuf[2];
+
+       static const u8 friio_init_cmds[][2] = {
+               {0x33, 0x08}, {0x37, 0x40}, {0x3a, 0x1f}, {0x3b, 0xff},
+               {0x3c, 0x1f}, {0x3d, 0xff}, {0x38, 0x00}, {0x35, 0x00},
+               {0x39, 0x00}, {0x36, 0x00},
+       };
+
+       ret = usb_set_interface(d->udev, 0, 0);
+       if (ret < 0)
+               return ret;
+
+       wbuf[0] = 0x11;
+       wbuf[1] = 0x02;
+       ret = gl861_i2c_msg(d, 0x00, wbuf, 2, NULL, 0);
+       if (ret < 0)
+               return ret;
+       usleep_range(2000, 3000);
+
+       wbuf[0] = 0x11;
+       wbuf[1] = 0x00;
+       ret = gl861_i2c_msg(d, 0x00, wbuf, 2, NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       /*
+        * Check if the dev is really a Friio White, since it might be
+        * another device, Friio Black, with the same VID/PID.
+        */
+
+       usleep_range(1000, 2000);
+       wbuf[0] = 0x03;
+       wbuf[1] = 0x80;
+       ret = gl861_i2c_write_ex(d, 0x09, wbuf, 2);
+       if (ret < 0)
+               return ret;
+
+       usleep_range(2000, 3000);
+       ret = gl861_i2c_read_ex(d, 0x09, rbuf, 2);
+       if (ret < 0)
+               return ret;
+       if (rbuf[0] != 0xff || rbuf[1] != 0xff)
+               return -ENODEV;
+
+
+       usleep_range(1000, 2000);
+       ret = gl861_i2c_write_ex(d, 0x48, wbuf, 2);
+       if (ret < 0)
+               return ret;
+
+       usleep_range(2000, 3000);
+       ret = gl861_i2c_read_ex(d, 0x48, rbuf, 2);
+       if (ret < 0)
+               return ret;
+       if (rbuf[0] != 0xff || rbuf[1] != 0xff)
+               return -ENODEV;
+
+       wbuf[0] = 0x30;
+       wbuf[1] = 0x04;
+       ret = gl861_i2c_msg(d, 0x00, wbuf, 2, NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       wbuf[0] = 0x00;
+       wbuf[1] = 0x01;
+       ret = gl861_i2c_msg(d, 0x00, wbuf, 2, NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       wbuf[0] = 0x06;
+       wbuf[1] = 0x0f;
+       ret = gl861_i2c_msg(d, 0x00, wbuf, 2, NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       for (i = 0; i < ARRAY_SIZE(friio_init_cmds); i++) {
+               ret = gl861_i2c_msg(d, 0x00, (u8 *)friio_init_cmds[i], 2,
+                                     NULL, 0);
+               if (ret < 0)
+                       return ret;
+       }
+       return 0;
+}
+
+/*
+ * DVB callbacks for Friio
+ */
+
+static int friio_power_ctrl(struct dvb_usb_device *d, int onoff)
+{
+       return onoff ? friio_reset(d) : 0;
+}
+
+static int friio_frontend_attach(struct dvb_usb_adapter *adap)
+{
+       const struct i2c_board_info *info;
+       struct dvb_usb_device *d;
+       struct tc90522_config cfg;
+       struct i2c_client *cl;
+       struct friio_priv *priv;
+
+       info = &friio_config.demod_info;
+       d = adap_to_d(adap);
+       cl = dvb_module_probe("tc90522", info->type,
+                             &d->i2c_adap, info->addr, &cfg);
+       if (!cl)
+               return -ENODEV;
+       adap->fe[0] = cfg.fe;
+
+       /* ignore cfg.tuner_i2c and create new one */
+       priv = adap_to_priv(adap);
+       priv->i2c_client_demod = cl;
+       priv->tuner_adap.algo = &friio_tuner_i2c_algo;
+       priv->tuner_adap.dev.parent = &d->udev->dev;
+       strlcpy(priv->tuner_adap.name, d->name, sizeof(priv->tuner_adap.name));
+       strlcat(priv->tuner_adap.name, "-tuner", sizeof(priv->tuner_adap.name));
+       priv->demod_sub_i2c = &priv->tuner_adap;
+       i2c_set_adapdata(&priv->tuner_adap, d);
+
+       return i2c_add_adapter(&priv->tuner_adap);
+}
+
+static int friio_frontend_detach(struct dvb_usb_adapter *adap)
+{
+       struct friio_priv *priv;
+
+       priv = adap_to_priv(adap);
+       i2c_del_adapter(&priv->tuner_adap);
+       dvb_module_release(priv->i2c_client_demod);
+       return 0;
+}
+
+static int friio_tuner_attach(struct dvb_usb_adapter *adap)
+{
+       const struct i2c_board_info *info;
+       struct dvb_pll_config cfg;
+       struct i2c_client *cl;
+       struct friio_priv *priv;
+
+       priv = adap_to_priv(adap);
+       info = &friio_config.tuner_info;
+       cfg = friio_config.tuner_cfg;
+       cfg.fe = adap->fe[0];
+
+       cl = dvb_module_probe("dvb_pll", info->type,
+                             priv->demod_sub_i2c, info->addr, &cfg);
+       if (!cl)
+               return -ENODEV;
+       priv->i2c_client_tuner = cl;
+       return 0;
+}
+
+static int friio_tuner_detach(struct dvb_usb_adapter *adap)
+{
+       struct friio_priv *priv;
+
+       priv = adap_to_priv(adap);
+       dvb_module_release(priv->i2c_client_tuner);
+       return 0;
+}
+
+static int friio_init(struct dvb_usb_device *d)
+{
+       int i;
+       int ret;
+       struct friio_priv *priv;
+
+       static const u8 demod_init[][2] = {
+               {0x01, 0x40}, {0x04, 0x38}, {0x05, 0x40}, {0x07, 0x40},
+               {0x0f, 0x4f}, {0x11, 0x21}, {0x12, 0x0b}, {0x13, 0x2f},
+               {0x14, 0x31}, {0x16, 0x02}, {0x21, 0xc4}, {0x22, 0x20},
+               {0x2c, 0x79}, {0x2d, 0x34}, {0x2f, 0x00}, {0x30, 0x28},
+               {0x31, 0x31}, {0x32, 0xdf}, {0x38, 0x01}, {0x39, 0x78},
+               {0x3b, 0x33}, {0x3c, 0x33}, {0x48, 0x90}, {0x51, 0x68},
+               {0x5e, 0x38}, {0x71, 0x00}, {0x72, 0x08}, {0x77, 0x00},
+               {0xc0, 0x21}, {0xc1, 0x10}, {0xe4, 0x1a}, {0xea, 0x1f},
+               {0x77, 0x00}, {0x71, 0x00}, {0x71, 0x00}, {0x76, 0x0c},
+       };
+
+       /* power on LNA? */
+       ret = friio_ext_ctl(d, FRIIO_LED_STOPPED, true);
+       if (ret < 0)
+               return ret;
+       msleep(20);
+
+       /* init/config demod */
+       priv = d_to_priv(d);
+       for (i = 0; i < ARRAY_SIZE(demod_init); i++) {
+               int ret;
+
+               ret = i2c_master_send(priv->i2c_client_demod, demod_init[i], 2);
+               if (ret < 0)
+                       return ret;
+       }
+       msleep(100);
+       return 0;
+}
+
+static void friio_exit(struct dvb_usb_device *d)
+{
+       friio_ext_ctl(d, FRIIO_LED_STOPPED, false);
+}
+
+static int friio_streaming_ctrl(struct dvb_frontend *fe, int onoff)
+{
+       u32 led_color;
+
+       led_color = onoff ? FRIIO_LED_RUNNING : FRIIO_LED_STOPPED;
+       return friio_ext_ctl(fe_to_d(fe), led_color, true);
+}
+
+
+static struct dvb_usb_device_properties friio_props = {
+       .driver_name = KBUILD_MODNAME,
+       .owner = THIS_MODULE,
+       .adapter_nr = adapter_nr,
+
+       .size_of_priv = sizeof(struct friio_priv),
+
+       .i2c_algo = &gl861_i2c_algo,
+       .power_ctrl = friio_power_ctrl,
+       .frontend_attach = friio_frontend_attach,
+       .frontend_detach = friio_frontend_detach,
+       .tuner_attach = friio_tuner_attach,
+       .tuner_detach = friio_tuner_detach,
+       .init = friio_init,
+       .exit = friio_exit,
+       .streaming_ctrl = friio_streaming_ctrl,
+
+       .num_adapters = 1,
+       .adapter = {
+               {
+                       .stream = DVB_USB_STREAM_BULK(0x01, 8, 16384),
+               }
+       }
+};
+
 static const struct usb_device_id gl861_id_table[] = {
        { DVB_USB_DEVICE(USB_VID_MSI, USB_PID_MSI_MEGASKY580_55801,
                &gl861_props, "MSI Mega Sky 55801 DVB-T USB2.0", NULL) },
        { DVB_USB_DEVICE(USB_VID_ALINK, USB_VID_ALINK_DTU,
                &gl861_props, "A-LINK DTU DVB-T USB2.0", NULL) },
+       { DVB_USB_DEVICE(USB_VID_774, USB_PID_FRIIO_WHITE,
+               &friio_props, "774 Friio White ISDB-T USB2.0", NULL) },
        { }
 };
 MODULE_DEVICE_TABLE(usb, gl861_id_table);
index b651b857e03408f3cedf9e281e5563355de02fe7..02c00e10748aed3aa476f78b4548f0d6f62fe66f 100644 (file)
@@ -9,5 +9,6 @@
 
 #define GL861_REQ_I2C_WRITE    0x01
 #define GL861_REQ_I2C_READ     0x02
+#define GL861_REQ_I2C_RAW      0x03
 
 #endif
index b8a1c62a06826256c981ae841e8f6d273368c85b..513df955eaa37e6c0ab742cb7a5198b8e022e4e1 100644 (file)
@@ -312,12 +312,6 @@ config DVB_USB_DTV5100
        help
          Say Y here to support the AME DTV-5100 USB2.0 DVB-T receiver.
 
-config DVB_USB_FRIIO
-       tristate "Friio ISDB-T USB2.0 Receiver support"
-       depends on DVB_USB
-       help
-         Say Y here to support the Japanese DTV receiver Friio.
-
 config DVB_USB_AZ6027
        tristate "Azurewave DVB-S/S2 USB2.0 AZ6027 support"
        depends on DVB_USB
index 9ad2618408eff4721021e66f0f56b0ad052e1b83..407d90ca8be02fef4517987927dacaa49cc0d43b 100644 (file)
@@ -71,9 +71,6 @@ obj-$(CONFIG_DVB_USB_DTV5100) += dvb-usb-dtv5100.o
 dvb-usb-cinergyT2-objs := cinergyT2-core.o cinergyT2-fe.o
 obj-$(CONFIG_DVB_USB_CINERGY_T2) += dvb-usb-cinergyT2.o
 
-dvb-usb-friio-objs := friio.o friio-fe.o
-obj-$(CONFIG_DVB_USB_FRIIO) += dvb-usb-friio.o
-
 dvb-usb-az6027-objs := az6027.o
 obj-$(CONFIG_DVB_USB_AZ6027) += dvb-usb-az6027.o