V4L/DVB (8735): dtv5100: replace dummy frontend by zl10353
authorAntoine Jacquet <royale@zerezo.com>
Mon, 11 Aug 2008 22:21:33 +0000 (19:21 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 12 Oct 2008 11:36:51 +0000 (09:36 -0200)
Remove the dtv5100-fe.c dummy frontend and replace it by the real frontend for the chipset.

Signed-off-by: Antoine Jacquet <royale@zerezo.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/dvb-usb/Makefile
drivers/media/dvb/dvb-usb/dtv5100-fe.c [deleted file]
drivers/media/dvb/dvb-usb/dtv5100.c
drivers/media/dvb/dvb-usb/dtv5100.h

index 5e170fb..116544f 100644 (file)
@@ -67,7 +67,7 @@ obj-$(CONFIG_DVB_USB_ANYSEE) += dvb-usb-anysee.o
 dvb-usb-dw2102-objs = dw2102.o
 obj-$(CONFIG_DVB_USB_DW2102) += dvb-usb-dw2102.o
 
-dvb-usb-dtv5100-objs = dtv5100.o dtv5100-fe.o
+dvb-usb-dtv5100-objs = dtv5100.o
 obj-$(CONFIG_DVB_USB_DTV5100) += dvb-usb-dtv5100.o
 
 EXTRA_CFLAGS += -Idrivers/media/dvb/dvb-core/ -Idrivers/media/dvb/frontends/
diff --git a/drivers/media/dvb/dvb-usb/dtv5100-fe.c b/drivers/media/dvb/dvb-usb/dtv5100-fe.c
deleted file mode 100644 (file)
index 08fe33b..0000000
+++ /dev/null
@@ -1,147 +0,0 @@
-/*
- * DVB USB Linux driver for AME DTV-5100 USB2.0 DVB-T
- *
- * Copyright (C) 2008  Antoine Jacquet <royale@zerezo.com>
- * http://royale.zerezo.com/dtv5100/
- *
- * Inspired by dvb_dummy_fe.c
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
- */
-
-#include "dvb-usb.h"
-#include "qt1010_priv.h"
-
-struct dtv5100_fe_state {
-       struct dvb_frontend frontend;
-};
-
-static int dtv5100_fe_read_status(struct dvb_frontend* fe, fe_status_t* status)
-{
-       *status = FE_HAS_SIGNAL
-               | FE_HAS_CARRIER
-               | FE_HAS_VITERBI
-               | FE_HAS_SYNC
-               | FE_HAS_LOCK;
-
-       return 0;
-}
-
-static int dtv5100_fe_read_ber(struct dvb_frontend* fe, u32* ber)
-{
-       *ber = 0;
-       return 0;
-}
-
-static int dtv5100_fe_read_signal_strength(struct dvb_frontend* fe, u16* strength)
-{
-       *strength = 0;
-       return 0;
-}
-
-static int dtv5100_fe_read_snr(struct dvb_frontend* fe, u16* snr)
-{
-       *snr = 0;
-       return 0;
-}
-
-static int dtv5100_fe_read_ucblocks(struct dvb_frontend* fe, u32* ucblocks)
-{
-       *ucblocks = 0;
-       return 0;
-}
-
-static int dtv5100_fe_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_parameters *p)
-{
-       return 0;
-}
-
-static int dtv5100_fe_set_frontend(struct dvb_frontend* fe, struct dvb_frontend_parameters *p)
-{
-       if (fe->ops.tuner_ops.set_params) {
-               fe->ops.tuner_ops.set_params(fe, p);
-               if (fe->ops.i2c_gate_ctrl) fe->ops.i2c_gate_ctrl(fe, 0);
-       }
-
-       return 0;
-}
-
-static int dtv5100_fe_sleep(struct dvb_frontend* fe)
-{
-       return 0;
-}
-
-static int dtv5100_fe_init(struct dvb_frontend* fe)
-{
-       return 0;
-}
-
-static void dtv5100_fe_release(struct dvb_frontend* fe)
-{
-       struct dtv5100_fe_state* state = fe->demodulator_priv;
-       kfree(state);
-}
-
-static struct dvb_frontend_ops dtv5100_fe_ops;
-
-struct dvb_frontend* dtv5100_fe_attach(void)
-{
-       struct dtv5100_fe_state* state = NULL;
-
-       /* allocate memory for the internal state */
-       state = kmalloc(sizeof(struct dtv5100_fe_state), GFP_KERNEL);
-       if (state == NULL) goto error;
-
-       /* create dvb_frontend */
-       memcpy(&state->frontend.ops, &dtv5100_fe_ops, sizeof(struct dvb_frontend_ops));
-       state->frontend.demodulator_priv = state;
-       return &state->frontend;
-
-error:
-       kfree(state);
-       return NULL;
-}
-
-static struct dvb_frontend_ops dtv5100_fe_ops = {
-
-       .info = {
-               .name                   = "Dummy DVB-T",
-               .type                   = FE_OFDM,
-               .frequency_min          = QT1010_MIN_FREQ,
-               .frequency_max          = QT1010_MAX_FREQ,
-               .frequency_stepsize     = QT1010_STEP,
-               .caps = FE_CAN_FEC_1_2 | FE_CAN_FEC_2_3 | FE_CAN_FEC_3_4 |
-                               FE_CAN_FEC_4_5 | FE_CAN_FEC_5_6 | FE_CAN_FEC_6_7 |
-                               FE_CAN_FEC_7_8 | FE_CAN_FEC_8_9 | FE_CAN_FEC_AUTO |
-                               FE_CAN_QAM_16 | FE_CAN_QAM_64 | FE_CAN_QAM_AUTO |
-                               FE_CAN_TRANSMISSION_MODE_AUTO |
-                               FE_CAN_GUARD_INTERVAL_AUTO |
-                               FE_CAN_HIERARCHY_AUTO,
-       },
-
-       .release = dtv5100_fe_release,
-
-       .init = dtv5100_fe_init,
-       .sleep = dtv5100_fe_sleep,
-
-       .set_frontend = dtv5100_fe_set_frontend,
-       .get_frontend = dtv5100_fe_get_frontend,
-
-       .read_status = dtv5100_fe_read_status,
-       .read_ber = dtv5100_fe_read_ber,
-       .read_signal_strength = dtv5100_fe_read_signal_strength,
-       .read_snr = dtv5100_fe_read_snr,
-       .read_ucblocks = dtv5100_fe_read_ucblocks,
-};
index f89d017..7168a49 100644 (file)
@@ -22,6 +22,7 @@
  */
 
 #include "dtv5100.h"
+#include "zl10353.h"
 #include "qt1010.h"
 
 /* debug */
@@ -30,25 +31,44 @@ module_param_named(debug, dvb_usb_dtv5100_debug, int, 0644);
 MODULE_PARM_DESC(debug, "set debugging level" DVB_USB_DEBUG_STATUS);
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
 
-static int dtv5100_read_reg(struct dvb_usb_device *d, u8 addr, u8 reg, u8 *value)
+static int dtv5100_i2c_msg(struct dvb_usb_device *d, u8 addr,
+                         u8 *wbuf, u16 wlen, u8 *rbuf, u16 rlen)
 {
-       return usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0),
-                              DTV5100_I2C_READ, USB_TYPE_VENDOR|USB_DIR_IN,
-                              0, (addr << 8) + reg, value, 1,
-                              DTV5100_USB_TIMEOUT);
-}
+       u8 request;
+       u8 type;
+       u16 value;
+       u16 index;
+
+       switch (wlen) {
+       case 1:
+               /* write { reg }, read { value } */
+               request = (addr == DTV5100_DEMOD_ADDR ? DTV5100_DEMOD_READ :
+                                                       DTV5100_TUNER_READ);
+               type = USB_TYPE_VENDOR|USB_DIR_IN;
+               value = 0;
+               break;
+       case 2:
+               /* write { reg, value } */
+               request = (addr == DTV5100_DEMOD_ADDR ? DTV5100_DEMOD_WRITE :
+                                                       DTV5100_TUNER_WRITE);
+               type = USB_TYPE_VENDOR|USB_DIR_OUT;
+               value = wbuf[1];
+               break;
+       default:
+               warn("wlen = %x, aborting.", wlen);
+               return -EINVAL;
+       }
+       index = (addr << 8) + wbuf[0];
 
-static int dtv5100_write_reg(struct dvb_usb_device *d, u8 addr, u8 reg, u8 value)
-{
-       return usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0),
-                              DTV5100_I2C_WRITE, USB_TYPE_VENDOR|USB_DIR_OUT,
-                              value, (addr << 8) + reg, NULL, 0,
+       msleep(1); /* avoid I2C errors */
+       return usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0), request,
+                              type, value, index, rbuf, rlen,
                               DTV5100_USB_TIMEOUT);
 }
 
 /* I2C */
 static int dtv5100_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msg[],
-                           int num)
+                          int num)
 {
        struct dvb_usb_device *d = i2c_get_adapdata(adap);
        int i;
@@ -60,25 +80,16 @@ static int dtv5100_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msg[],
                return -EAGAIN;
 
        for (i = 0; i < num; i++) {
+               /* write/read request */
                if (i+1 < num && (msg[i+1].flags & I2C_M_RD)) {
-                       /* write/read request, 2 messages
-                        *      msg = {
-                        *              { reg },
-                        *              { val },
-                        *      }
-                        */
-                       if (dtv5100_read_reg(d, msg[i].addr, msg[i].buf[0], msg[i+1].buf) < 0)
+                       if (dtv5100_i2c_msg(d, msg[i].addr, msg[i].buf,
+                                           msg[i].len, msg[i+1].buf,
+                                           msg[i+1].len) < 0)
                                break;
                        i++;
-               } else {
-                       /* write request, 1 message
-                        *      msg = {
-                        *              { reg, val },
-                        *      }
-                        */
-                       if (dtv5100_write_reg(d, msg[i].addr, msg[i].buf[0], msg[i].buf[1]) < 0)
+               } else if (dtv5100_i2c_msg(d, msg[i].addr, msg[i].buf,
+                                          msg[i].len, NULL, 0) < 0)
                                break;
-               }
        }
 
        mutex_unlock(&d->i2c_mutex);
@@ -96,14 +107,27 @@ static struct i2c_algorithm dtv5100_i2c_algo = {
 };
 
 /* Callbacks for DVB USB */
+static struct zl10353_config dtv5100_zl10353_config = {
+       .demod_address = DTV5100_DEMOD_ADDR,
+       .no_tuner = 1,
+       .parallel_ts = 1,
+};
+
 static int dtv5100_frontend_attach(struct dvb_usb_adapter *adap)
 {
-       adap->fe = dtv5100_fe_attach();
+       adap->fe = dvb_attach(zl10353_attach, &dtv5100_zl10353_config,
+               &adap->dev->i2c_adap);
+       if (adap->fe == NULL)
+               return -EIO;
+
+       /* disable i2c gate, or it won't work... is this safe? */
+       adap->fe->ops.i2c_gate_ctrl = NULL;
+
        return 0;
 }
 
 static struct qt1010_config dtv5100_qt1010_config = {
-       .i2c_address = 0xc4
+       .i2c_address = DTV5100_TUNER_ADDR
 };
 
 static int dtv5100_tuner_attach(struct dvb_usb_adapter *adap)
@@ -122,12 +146,7 @@ static int dtv5100_probe(struct usb_interface *intf,
        int i, ret;
        struct usb_device *udev = interface_to_usbdev(intf);
 
-       ret = dvb_usb_device_init(intf, &dtv5100_properties,
-                                 THIS_MODULE, NULL, adapter_nr);
-       if (ret)
-               return ret;
-
-       /* initialize frontend & non qt1010 part? */
+       /* initialize non qt1010/zl10353 part? */
        for (i = 0; dtv5100_init[i].request; i++)
        {
                ret = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0),
@@ -140,6 +159,11 @@ static int dtv5100_probe(struct usb_interface *intf,
                        return ret;
        }
 
+       ret = dvb_usb_device_init(intf, &dtv5100_properties,
+                                 THIS_MODULE, NULL, adapter_nr);
+       if (ret)
+               return ret;
+
        return 0;
 }
 
index 1aadb86..e36bf2d 100644 (file)
 #include "dvb-usb.h"
 
 #define DTV5100_USB_TIMEOUT 500
-#define DTV5100_I2C_WRITE 0xc7
-#define DTV5100_I2C_READ  0xc8
+
+#define DTV5100_DEMOD_ADDR     0x00
+#define DTV5100_DEMOD_WRITE    0xc0
+#define DTV5100_DEMOD_READ     0xc1
+
+#define DTV5100_TUNER_ADDR     0xc4
+#define DTV5100_TUNER_WRITE    0xc7
+#define DTV5100_TUNER_READ     0xc8
 
 #define DRIVER_AUTHOR "Antoine Jacquet, http://royale.zerezo.com/"
 #define DRIVER_DESC "AME DTV-5100 USB2.0 DVB-T"
@@ -39,87 +45,6 @@ static struct {
 } dtv5100_init[] = {
        { 0x000000c5, 0x00000000, 0x00000001 },
        { 0x000000c5, 0x00000001, 0x00000001 },
-       { 0x000000c0, 0x0000000b, 0x00000050 },
-       { 0x000000c0, 0x00000044, 0x00000051 },
-       { 0x000000c0, 0x00000046, 0x00000052 },
-       { 0x000000c0, 0x00000015, 0x00000053 },
-       { 0x000000c0, 0x0000000f, 0x00000054 },
-       { 0x000000c0, 0x00000080, 0x00000055 },
-       { 0x000000c0, 0x00000001, 0x000000ea },
-       { 0x000000c0, 0x00000000, 0x000000ea },
-       { 0x000000c0, 0x00000075, 0x0000005c },
-       { 0x000000c0, 0x000000a1, 0x0000009c },
-       { 0x000000c0, 0x00000000, 0x0000008e },
-       { 0x000000c0, 0x00000000, 0x00000090 },
-       { 0x000000c0, 0x000000ff, 0x00000091 },
-       { 0x000000c0, 0x000000ff, 0x00000092 },
-       { 0x000000c0, 0x00000000, 0x00000093 },
-       { 0x000000c0, 0x000000ff, 0x00000094 },
-       { 0x000000c0, 0x00000000, 0x00000058 },
-       { 0x000000c0, 0x0000003f, 0x00000095 },
-       { 0x000000c0, 0x0000003f, 0x00000096 },
-       { 0x000000c0, 0x0000000c, 0x0000005a },
-       { 0x000000c0, 0x0000002b, 0x00000056 },
-       { 0x000000c0, 0x00000017, 0x0000005f },
-       { 0x000000c0, 0x00000040, 0x0000005e },
-       { 0x000000c0, 0x00000036, 0x00000064 },
-       { 0x000000c0, 0x00000067, 0x00000065 },
-       { 0x000000c0, 0x000000e5, 0x00000066 },
-       { 0x000000c0, 0x00000073, 0x000000cc },
-       { 0x000000c0, 0x000000cd, 0x0000006c },
-       { 0x000000c0, 0x0000007e, 0x0000006d },
-       { 0x000000c0, 0x00000001, 0x00000071 },
-       { 0x000000c0, 0x00000001, 0x00000070 },
-       /**/
-       { 0x000000c7, 0x00000080, 0x0000c401 },
-       { 0x000000c7, 0x0000003f, 0x0000c402 },
-       { 0x000000c7, 0x00000034, 0x0000c405 }, // 2
-       { 0x000000c7, 0x00000044, 0x0000c406 },
-       { 0x000000c7, 0x00000038, 0x0000c407 }, // 4
-       { 0x000000c7, 0x00000008, 0x0000c408 },
-       { 0x000000c7, 0x0000001c, 0x0000c409 }, // 6
-       { 0x000000c7, 0x0000000d, 0x0000c40a }, // 7
-       { 0x000000c7, 0x00000045, 0x0000c40b }, // 8
-       { 0x000000c7, 0x000000e1, 0x0000c40c },
-       { 0x000000c7, 0x00000078, 0x0000c41a }, // 10
-       { 0x000000c7, 0x00000000, 0x0000c41b },
-       { 0x000000c7, 0x00000089, 0x0000c41c },
-       { 0x000000c7, 0x000000fd, 0x0000c411 }, // 13
-       { 0x000000c7, 0x00000095, 0x0000c412 }, // 14
-       { 0x000000c7, 0x000000d6, 0x0000c422 }, // 15
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c7, 0x000000d0, 0x0000c41e },
-       { 0x000000c8, 0x00000000, 0x0000c422 },
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c8, 0x00000000, 0x0000c405 },
-       { 0x000000c8, 0x00000000, 0x0000c422 },
-       { 0x000000c7, 0x000000d0, 0x0000c423 },
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c7, 0x000000e0, 0x0000c41e },
-       { 0x000000c8, 0x00000000, 0x0000c423 },
-       { 0x000000c8, 0x00000000, 0x0000c423 },
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c7, 0x000000d0, 0x0000c424 },
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c7, 0x000000f0, 0x0000c41e },
-       { 0x000000c8, 0x00000000, 0x0000c424 },
-       { 0x000000c7, 0x00000000, 0x0000c41e },
-       { 0x000000c7, 0x0000007f, 0x0000c414 },
-       { 0x000000c7, 0x0000007f, 0x0000c415 },
-       { 0x000000c7, 0x00000030, 0x0000c405 },
-       { 0x000000c7, 0x00000000, 0x0000c406 },
-       { 0x000000c7, 0x0000001f, 0x0000c415 },
-       { 0x000000c7, 0x000000ff, 0x0000c416 },
-       { 0x000000c7, 0x000000ff, 0x0000c418 },
-       { 0x000000c7, 0x00000051, 0x0000c41f }, // here
-       { 0x000000c7, 0x00000016, 0x0000c420 }, // here
-       { 0x000000c7, 0x00000053, 0x0000c421 },
-       { 0x000000c7, 0x000000c1, 0x0000c425 }, // here
-       { 0x000000c7, 0x00000014, 0x0000c426 }, // here
-       { 0x000000c7, 0x00000082, 0x0000c400 },
-       { 0x000000c7, 0x00000000, 0x0000c402 },
-       { 0x000000c7, 0x00000000, 0x0000c401 },
-       /**/
        { }             /* Terminating entry */
 };