[linux-dvb] [PATCH] DViCO FusionHDTV3 Gold - add dvb support

Michael Krufky mkrufky at m1k.net
Thu Jun 16 03:07:18 CEST 2005


Mac Michaels wrote:

>This patch was sent to both v4l2 and dvb lists because it affects files in both video and dvb media subdirectories. If it needs to be made against some other source tree, please tell me how to get that source tree.
>
Mac-

It took me over an hour to patch this code against video4linux cvs 
(while eating chinese food and being bombarded by instant-messages).

I made a few minor changes to it, in order to fit in with the rest of 
the code in cvs.  Please patch this against current video4linux cvs 
yourself and tell me if it still works correctly for you.  After you 
tell me that it works, I'd like to adapt this to include support for my 
DViCO FusionHDTV3 Gold-T card (card #28), which uses a Thomson DDT 7611 
tuner, as opposed to your card (#17) which uses the microtune tuner.

There currently is some dvb code in video4linux cvs, which will 
eventually be moved into the DVB cvs.  I don't know what is involved in 
this, but I'd like to commit this to cvs before that move is made, this 
way all of the DVB code can me moved together.

ALSO, if you plan on sending in more patches, this is how you should do it:

- Patches should be against current video4linux cvs, in patch -p1 form.

Here is how I make my patches (this isn't necessarily the ONLY way, but 
I find it to be easiest.)

cvs -d :pserver:anonymous at cvs.linuxtv.org:/cvs/video4linux login
cvs -d :pserver:anonymous at cvs.linuxtv.org:/cvs/video4linux co video4linux

cp video4linux video4linux.orig -r

cd video4linux
(make your changes within the video4linux directory)
cd ..

diff -upN video4linux.orig video4linux > name_of_your_patch.patch

Then, edit the patchfile in nano or emacs or whatever editor you like, 
and remove the lines saying "Common subdirectories: yadda yadda yadda"

If you follow the guidelines above, people will be more likely to try 
out your patch, because they will be able to apply it cleanly against 
video4linux cvs by doing the following: (assuming that the patchfile and 
video4linux source are both located in /usr/src)

cd /usr/src/video4linux
patch -p1 < ../name_of_your_patch.patch

And remember, if you cant get your patch to apply, nobody else will either.

Once again, I greatly thank you for your time spent on the LGDT3302 
driver ... I truly cannot wait untill I can get this code working with 
my board.

-- 
Michael Krufky

-------------- next part --------------
diff -upN video4linux.orig/Makefile video4linux/Makefile
--- video4linux.orig/Makefile	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/Makefile	2005-06-15 19:50:27.000000000 +0000
@@ -39,7 +39,7 @@ obj-$(CONFIG_VIDEO_IR)		+= ir-common.o
 obj-$(CONFIG_VIDEO_TUNER)	+= tuner.o tda9887.o
 obj-$(CONFIG_VIDEO_TVAUDIO)	+= msp3400.o tvaudio.o tvmixer.o
 
-obj-$(CONFIG_VIDEO_CX88_DVB)	+= video-buf-dvb.o cx88-dvb.o cx22702.o dvb-pll.o or51132.o
+obj-$(CONFIG_VIDEO_CX88_DVB)	+= video-buf-dvb.o cx88-dvb.o cx22702.o dvb-pll.o or51132.o lgdt3302.o
 obj-$(CONFIG_VIDEO_SAA7134_DVB) += video-buf-dvb.o saa7134-dvb.o mt352.o
 
 # 2.6-only stuff
diff -upN video4linux.orig/cx88-cards.c video4linux/cx88-cards.c
--- video4linux.orig/cx88-cards.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/cx88-cards.c	2005-06-15 20:06:46.000000000 +0000
@@ -482,6 +482,7 @@ struct cx88_board cx88_boards[] = {
 			.vmux   = 2,
 			.gpio0	= 0x0f00,
 		}},
+		.dvb            = 1,
 	},
         [CX88_BOARD_HAUPPAUGE_DVB_T1] = {
                 .name           = "Hauppauge Nova-T DVB-T",
diff -upN video4linux.orig/cx88-dvb.c video4linux/cx88-dvb.c
--- video4linux.orig/cx88-dvb.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/cx88-dvb.c	2005-06-15 20:17:23.000000000 +0000
@@ -30,9 +30,10 @@
 #include <linux/file.h>
 #include <linux/suspend.h>
 
-/* those two frontends need merging via linuxtv cvs ... */
+/* these three frontends need merging via linuxtv cvs ... */
 #define HAVE_CX22702 1
 #define HAVE_OR51132 1
+#define HAVE_LGDT3302 1
 
 #include "cx88.h"
 #include "dvb-pll.h"
@@ -44,6 +45,9 @@
 #if HAVE_OR51132
 # include "or51132.h"
 #endif
+#if HAVE_LGDT3302
+#include "lgdt3302.h"
+#endif
 
 MODULE_DESCRIPTION("driver for cx2388x based DVB cards");
 MODULE_AUTHOR("Chris Pascoe <c.pascoe at itee.uq.edu.au>");
@@ -199,6 +203,23 @@ static struct or51132_config pchdtv_hd30
 };
 #endif
 
+static int lgdt3302_set_ts_param(struct dvb_frontend* fe, int is_punctured)
+{
+	struct cx8802_dev *dev= fe->dvb->priv;
+	if (is_punctured)
+		dev->ts_gen_cntrl |= 0x04;
+	else
+		dev->ts_gen_cntrl &= ~0x04;
+	return 0;
+}
+
+struct lgdt3302_config fusionhdtv_3_gold = {
+	.demod_address    = 0x0e,
+	.pll_address      = 0x61,
+	.pll_desc         = &dvb_pll_microtune_4042,
+	.set_ts_params    = lgdt3302_set_ts_param,
+};
+
 static int dvb_register(struct cx8802_dev *dev)
 {
 	/* init struct videobuf_dvb */
@@ -242,6 +263,22 @@ static int dvb_register(struct cx8802_de
 						 &dev->core->i2c_adap);
 		break;
 #endif
+#if HAVE_LGDT3302
+	case CX88_BOARD_DVICO_FUSIONHDTV_3_GOLD:
+		dev->ts_gen_cntrl = 0x08;
+		{
+		/* Do a hardware reset of chip before using it. */
+		struct cx88_core *core = dev->core;
+
+		cx_clear(MO_GP0_IO, 1);
+		mdelay(100);
+		cx_set(MO_GP0_IO, 9); // ANT connector too FIXME
+		mdelay(200);
+		dev->dvb.frontend = lgdt3302_attach(&fusionhdtv_3_gold,
+						    &dev->core->i2c_adap);
+		}
+		break;
+#endif
 	default:
 		printk("%s: The frontend of your DVB/ATSC card isn't supported yet\n",
 		       dev->core->name);
diff -upN video4linux.orig/cx88-i2c.c video4linux/cx88-i2c.c
--- video4linux.orig/cx88-i2c.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/cx88-i2c.c	2005-06-15 20:20:21.000000000 +0000
@@ -157,6 +157,7 @@ static struct i2c_client cx8800_i2c_clie
 };
 
 static char *i2c_devs[128] = {
+	[ 0x1c >> 1 ] = "lgdt3302",
 	[ 0x86 >> 1 ] = "tda9887/cx22702",
 	[ 0xa0 >> 1 ] = "eeprom",
 	[ 0xc0 >> 1 ] = "tuner (analog)",
diff -upN video4linux.orig/cx88-mpeg.c video4linux/cx88-mpeg.c
--- video4linux.orig/cx88-mpeg.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/cx88-mpeg.c	2005-06-15 20:26:01.000000000 +0000
@@ -70,11 +70,16 @@ static int cx8802_start_dma(struct cx880
 
 	if (cx88_boards[core->board].dvb) {
 		/* negedge driven & software reset */
-		cx_write(TS_GEN_CNTRL, 0x40);
+		cx_write(TS_GEN_CNTRL, 0x0040 | dev->ts_gen_cntrl);
 		udelay(100);
 		cx_write(MO_PINMUX_IO, 0x00);
-		cx_write(TS_HW_SOP_CNTRL,47<<16|188<<4|0x00);
-		cx_write(TS_SOP_STAT,0x00);
+		if (core->board == CX88_BOARD_DVICO_FUSIONHDTV_3_GOLD) {
+			cx_write(TS_HW_SOP_CNTRL,0x47<<16 | 188<<4 | 0x00);
+			cx_write(TS_SOP_STAT, 0<<16 | 0<<14 | 1<<13 | 0<<12);
+		} else {
+			cx_write(TS_HW_SOP_CNTRL,47<<16|188<<4|0x00);
+			cx_write(TS_SOP_STAT,0x00);
+		}
 		cx_write(TS_GEN_CNTRL, dev->ts_gen_cntrl);
 		udelay(100);
 	}
diff -upN video4linux.orig/dvb-pll.c video4linux/dvb-pll.c
--- video4linux.orig/dvb-pll.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/dvb-pll.c	2005-06-15 19:53:59.000000000 +0000
@@ -95,6 +95,19 @@ struct dvb_pll_desc dvb_pll_lg_z201 = {
 };
 EXPORT_SYMBOL(dvb_pll_lg_z201);
 
+struct dvb_pll_desc dvb_pll_microtune_4042 = {
+	.name  = "Microtune 4042 FI5",
+	.min   =  57000000,
+	.max   = 858000000,
+	.count = 3,
+	.entries = {
+		{ 162000000, 44000000, 62500, 0x8e, 0xa1 },
+		{ 457000000, 44000000, 62500, 0x8e, 0x91 },
+		{ 999999999, 44000000, 62500, 0x8e, 0x31 },
+		},
+};
+EXPORT_SYMBOL(dvb_pll_microtune_4042);
+
 struct dvb_pll_desc dvb_pll_unknown_1 = {
 	.name  = "unknown 1", /* used by dntv live dvb-t */
 	.min   = 174000000,
diff -upN video4linux.orig/dvb-pll.h video4linux/dvb-pll.h
--- video4linux.orig/dvb-pll.h	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/dvb-pll.h	2005-06-15 19:54:58.000000000 +0000
@@ -21,6 +21,7 @@ extern struct dvb_pll_desc dvb_pll_thoms
 extern struct dvb_pll_desc dvb_pll_thomson_dtt759x;
 extern struct dvb_pll_desc dvb_pll_thomson_dtt7610;
 extern struct dvb_pll_desc dvb_pll_lg_z201;
+extern struct dvb_pll_desc dvb_pll_microtune_4042;
 extern struct dvb_pll_desc dvb_pll_unknown_1;
 
 int dvb_pll_configure(struct dvb_pll_desc *desc, u8 *buf,
diff -upN video4linux.orig/lgdt3302.c video4linux/lgdt3302.c
--- video4linux.orig/lgdt3302.c	1970-01-01 00:00:00.000000000 +0000
+++ video4linux/lgdt3302.c	2005-06-15 20:02:03.000000000 +0000
@@ -0,0 +1,607 @@
+/*
+ *    Support for LGDT3302 (DViCO FustionHDTV 3 Gold) - VSB/QAM
+ *
+ *    Copyright (C) 2005 Wilson Michaels <wilsonmichaels at earthlink.net>
+ *
+ *    Based on code from  Kirk Lapray <kirk_lapray at bigfoot.com>
+ *                           Copyright (C) 2005
+ *
+ *    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., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+/*                 
+ *                      NOTES ABOUT THIS DRIVER
+ *
+ * This driver supports DViCO FusionHDTV 3 Gold under Linux 2.6.11.
+ *
+ * SNR returns a signed integer of the SNR in db (Max 45 db) (Typical 20 db). 
+ * Note: negative values mean noise is greater than signal (unusable signal).
+ * 
+ * TODO:
+ * BER and signal strength always return 0.
+ * Convert to Linux 2.6.12 driver standards & merge into kernel.
+ * 
+ */
+
+
+#include <linux/version.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/moduleparam.h>
+#include <linux/init.h>
+#include <linux/delay.h>
+#include <asm/byteorder.h>
+
+#include "dvb_frontend.h"
+#include "dvb-pll.h"
+#include "lgdt3302_priv.h"
+#include "lgdt3302.h"
+
+static int debug = 0;
+module_param(debug, int, 0644);
+MODULE_PARM_DESC(debug,"Turn on/off lgdt3302 frontend debugging (default:off).");
+#define dprintk(args...) \
+do { \
+if (debug) printk(KERN_DEBUG "lgdt3302: " args); \
+} while (0)
+
+
+struct lgdt3302_state
+{
+  struct i2c_adapter* i2c;
+  struct dvb_frontend_ops ops;
+
+  /* Configuration settings */
+  const struct lgdt3302_config* config;
+
+  struct dvb_frontend frontend;
+  
+  /* Demodulator private data */
+  fe_modulation_t current_modulation;
+
+  /* Tuner private data */
+  u32 current_frequency;
+};
+
+static int i2c_writebytes (struct lgdt3302_state* state,
+			   u8 addr, /* demod_address or pll_address */
+			   u8 *buf, /* data bytes to send */
+			   int len  /* number of bytes to send */ )
+{
+  if (addr == state->config->pll_address) {
+    struct i2c_msg msg =
+      { .addr = addr, .flags = 0, .buf = buf,  .len = len };
+    int err;
+
+    if ((err = i2c_transfer(state->i2c, &msg, 1)) != 1) {
+      printk(KERN_WARNING "lgdt3302: %s error (addr %02x <- %02x, err == %i)\n", __FUNCTION__, addr, buf[0], err);
+      return -EREMOTEIO;
+    }
+  } else {
+    u8 tmp[] = { buf[0], buf[1] };
+    struct i2c_msg msg =
+      { .addr = addr, .flags = 0, .buf = tmp,  .len = 2 };
+    int err;
+    int i;
+
+    for (i=1; i<len; i++) {
+      tmp[1] = buf[i];
+      if ((err = i2c_transfer(state->i2c, &msg, 1)) != 1) {
+	printk(KERN_WARNING "lgdt3302: %s error (addr %02x <- %02x, err == %i)\n", __FUNCTION__, addr, buf[0], err);
+	return -EREMOTEIO;
+      }
+      tmp[0]++;
+    }
+  }
+
+  return 0;
+}
+static int i2c_readbytes (struct lgdt3302_state* state,
+			  u8 addr, /* demod_address or pll_address */
+			  u8 *buf, /* holds data bytes read */
+			  int len  /* number of bytes to read */ )
+{
+  struct i2c_msg msg =
+    { .addr = addr, .flags = I2C_M_RD, .buf = buf,  .len = len };
+  int err;
+
+  if ((err = i2c_transfer(state->i2c, &msg, 1)) != 1) {
+    printk(KERN_WARNING "lgdt3302: %s error (addr %02x, err == %i)\n", __FUNCTION__, addr, err);
+    return -EREMOTEIO;
+  }
+
+  return 0;
+}
+
+/*
+ * This routine writes the register (reg) to the demod bus 
+ * then reads the data returned for (len) bytes.
+ */
+static u8 i2c_selectreadbytes (struct lgdt3302_state* state,
+			       enum I2C_REG reg, u8* buf, int len)
+{
+  u8 wr [] = { reg };
+  struct i2c_msg msg [] = {
+    { .addr = state->config->demod_address,
+      .flags = 0, .buf = wr,  .len = 1 },
+    { .addr = state->config->demod_address,
+      .flags = I2C_M_RD, .buf = buf, .len = len },
+  };
+  int ret;
+
+  ret = i2c_transfer(state->i2c, msg, 2);
+  if (ret != 2) {
+    printk(KERN_WARNING "lgdt3302: %s: addr 0x%02x select 0x%02x error (ret == %i)\n", __FUNCTION__, state->config->demod_address, reg, ret);
+  } else {
+    ret = 0;
+  }
+  return ret;
+}
+
+/* Software reset */
+int lgdt3302_SwReset(struct lgdt3302_state* state)
+{
+  u8 ret;
+  u8 reset[] = {
+    IRQ_MASK,
+    0x00 // bit 6 is active low software reset
+         // bits 5-0 are 1 to mask interrupts
+  };
+
+  ret = i2c_writebytes(state, 
+		       state->config->demod_address,
+		       reset, sizeof(reset));
+  if (ret == 0) {
+    /* spec says reset takes 100 ns why wait */
+    //mdelay(100);        // keep low for 100mS
+    reset[1] = 0x7f;// force reset high (inactive)
+    // and unmask interrupts
+    ret = i2c_writebytes(state,
+			 state->config->demod_address,
+			 reset, sizeof(reset));
+  }
+  /* Spec does not indicate a need for this either */
+  //mdelay(5);              // wait 5 msec before doing more
+  return ret;
+}
+
+static int lgdt3302_init(struct dvb_frontend* fe)
+{
+  // Hardware reset is done using gpio[0] of cx23880x chip.
+  // I'd like to do it here, but don't know how to find chip address.
+  // cx88-cards.c arranges for the reset bit to be inactive (high).
+  // Maybe there needs to be a callable function in cx88-core or
+  // the caller of this function needs to do it.
+  dprintk("%s entered\n", __FUNCTION__);
+
+  return lgdt3302_SwReset((struct lgdt3302_state*) fe->demodulator_priv);
+}
+
+static int lgdt3302_read_ber(struct dvb_frontend* fe, u32* ber)
+{
+  *ber = 0; // Dummy out for now
+  return 0;
+}
+
+static int lgdt3302_read_ucblocks(struct dvb_frontend* fe, u32* ucblocks)
+{
+  struct lgdt3302_state* state = (struct lgdt3302_state*) fe->demodulator_priv;
+  u8 buf[2];
+
+  i2c_selectreadbytes(state, PACKET_ERR_COUNTER1, buf, sizeof(buf));
+
+  *ucblocks = (buf[0] << 8) | buf[1];
+  return 0;
+}
+
+static int lgdt3302_set_parameters(struct dvb_frontend* fe,
+				   struct dvb_frontend_parameters *param)
+{
+  u8 buf[4];
+  struct lgdt3302_state* state = 
+    (struct lgdt3302_state*) fe->demodulator_priv;
+
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,10)
+
+  /* Use 50MHz parameter values from spec sheet since xtal is 50 */
+  static u8 top_ctrl_cfg[]   = { TOP_CONTROL, 0x03 };
+  static u8 vsb_freq_cfg[]   = { VSB_CARRIER_FREQ0, 0x00, 0x87, 0x8e, 0x01 };
+  static u8 demux_ctrl_cfg[] = { DEMUX_CONTROL, 0xfb };
+  static u8 agc_rf_cfg[]     = { AGC_RF_BANDWIDTH0, 0x40, 0x93, 0x00 };
+  static u8 agc_ctrl_cfg[]   = { AGC_FUNC_CTRL2, 0xc6, 0x40 };
+  static u8 agc_delay_cfg[]  = { AGC_DELAY0, 0x00, 0x00, 0x00 };
+  static u8 agc_loop_cfg[]   = { AGC_LOOP_BANDWIDTH0, 0x08, 0x9a };
+
+  /* Change only if we are actually changing the modulation */
+  if (state->current_modulation != param->u.vsb.modulation) {
+    switch(param->u.vsb.modulation) {
+    case VSB_8:
+      dprintk("%s: VSB_8 MODE\n", __FUNCTION__);
+
+      /* Select VSB mode and serial MPEG interface */
+      top_ctrl_cfg[1] = 0x07;
+      break;
+      
+    case QAM_64:
+      dprintk("%s: QAM_64 MODE\n", __FUNCTION__);
+      
+      /* Select QAM_64 mode and serial MPEG interface */
+      top_ctrl_cfg[1] = 0x04;
+      break;
+      
+    case QAM_256:
+      dprintk("%s: QAM_256 MODE\n", __FUNCTION__);
+      
+      /* Select QAM_256 mode and serial MPEG interface */
+      top_ctrl_cfg[1] = 0x05;
+      break;
+    default:
+      printk(KERN_WARNING "lgdt3302: %s: Modulation type(%d) UNSUPPORTED\n", __FUNCTION__, param->u.vsb.modulation);
+      return -1;
+    }
+    /* Initializations common to all modes */
+
+    /* Select the requested mode */
+    i2c_writebytes(state, state->config->demod_address,
+		   top_ctrl_cfg, sizeof(top_ctrl_cfg));
+
+    /* Change the value of IFBW[11:0]
+       of AGC IF/RF loop filter bandwidth register */
+    i2c_writebytes(state, state->config->demod_address,
+		   agc_rf_cfg, sizeof(agc_rf_cfg));
+
+    /* Change the value of bit 6, 'nINAGCBY' and
+       'NSSEL[1:0] of ACG function control register 2 */
+    /* Change the value of bit 6 'RFFIX'
+       of AGC function control register 3 */
+    i2c_writebytes(state, state->config->demod_address,
+		   agc_ctrl_cfg, sizeof(agc_ctrl_cfg));
+
+    /* Change the TPCLK pin polarity 
+       data is valid on falling clock */
+    i2c_writebytes(state, state->config->demod_address,
+		   demux_ctrl_cfg, sizeof(demux_ctrl_cfg));
+
+    if (param->u.vsb.modulation == VSB_8) {
+      /* Initialization for VSB modes only */
+      /* Change the value of NCOCTFV[25:0]of carrier
+	 recovery center frequency register for VSB */
+      i2c_writebytes(state, state->config->demod_address,
+		     vsb_freq_cfg, sizeof(vsb_freq_cfg));
+    } else { 
+      /* Initialization for QAM modes only */
+      /* Set the value of 'INLVTHD' register 0x2a/0x2c
+	 to value from 'IFACC' register 0x39/0x3b -1 */
+      int value;
+      i2c_selectreadbytes(state, AGC_RFIF_ACC0,
+			  &agc_delay_cfg[1], 3);
+      value = ((agc_delay_cfg[1] & 0x0f) << 8) | agc_delay_cfg[3];
+      value = value -1;
+      dprintk("%s IFACC -1 = 0x%03x\n", __FUNCTION__, value);
+      agc_delay_cfg[1] = (value >> 8) & 0x0f;
+      agc_delay_cfg[2] = 0x00;
+      agc_delay_cfg[3] = value & 0xff;
+      i2c_writebytes(state, state->config->demod_address,
+		     agc_delay_cfg, sizeof(agc_delay_cfg));
+
+      /* Change the value of IAGCBW[15:8]
+	 of inner AGC loop filter bandwith */
+      i2c_writebytes(state, state->config->demod_address,
+		     agc_loop_cfg, sizeof(agc_loop_cfg));
+    }
+
+    state->config->set_ts_params(fe, 0);
+    lgdt3302_SwReset(state);
+    state->current_modulation = param->u.vsb.modulation;
+  }
+#else
+  printk("lgdt3302: %s: you need a newer kernel for this, sorry\n",__FUNCTION__);
+#endif
+
+  /* Change only if we are actually changing the channel */
+  if (state->current_frequency != param->frequency) {
+    dvb_pll_configure(state->config->pll_desc, buf,
+		      param->frequency, 0);
+    dprintk("%s: tuner bytes: 0x%02x 0x%02x "
+	    "0x%02x 0x%02x\n", __FUNCTION__, buf[0],buf[1],buf[2],buf[3]);
+    i2c_writebytes(state, state->config->pll_address ,buf, 4);
+
+    /* Check the status of the tuner pll */
+    i2c_readbytes(state, state->config->pll_address, buf, 1);
+    dprintk("%s: tuner status byte = 0x%02x\n", __FUNCTION__, buf[0]);
+    
+    lgdt3302_SwReset(state);
+    
+    /* Update current frequency */
+    state->current_frequency = param->frequency;
+  }
+  return 0;
+}
+
+static int lgdt3302_get_frontend(struct dvb_frontend* fe,
+				 struct dvb_frontend_parameters* param)
+{
+  struct lgdt3302_state *state = fe->demodulator_priv;
+
+  param->frequency = state->current_frequency;
+  return 0;
+}
+
+
+static int lgdt3302_read_status(struct dvb_frontend* fe, fe_status_t* status)
+{
+  struct lgdt3302_state* state = (struct lgdt3302_state*) fe->demodulator_priv;
+  u8 buf[3];
+
+  *status = 0;// Reset status result
+
+  /* Check the status of the tuner pll */
+  i2c_readbytes(state, state->config->pll_address, buf, 1);
+  dprintk("%s: tuner status byte = 0x%02x\n", __FUNCTION__, buf[0]);
+  if ((buf[0] & 0xc0) != 0x40)
+    return 0; // Microtune tuner PLL not locked or not powered on
+
+  /*
+   * You must set the Mask bits to 1 in the IRQ_MASK in order
+   * to see that status bit in the IRQ_STATUS register.
+   * This is done in SwReset();
+   */
+  /* signal status */
+  i2c_selectreadbytes(state, TOP_CONTROL, buf, sizeof(buf));
+  dprintk("%s: TOP_CONTROL = 0x%02x, IRO_MASK = 0x%02x, IRQ_STATUS = 0x%02x\n", __FUNCTION__, buf[0], buf[1], buf[2]);
+  if ((buf[2] & 0x30) == 0x10)
+    *status |= FE_HAS_SIGNAL;
+
+  /* sync status */
+  if ((buf[2] & 0x03) == 0x01) {
+    *status |= FE_HAS_SYNC;
+  }
+
+  /* FEC error status */
+  if ((buf[2] & 0x0c) == 0x08)
+    *status |= FE_HAS_LOCK;
+  *status |= FE_HAS_VITERBI;
+
+#if 0
+  /* Alternative method to check for a signal */
+  /* AGC status register */
+  i2c_selectreadbytes(state, AGC_STATUS, buf, 1);
+  dprintk("%s: AGC_STATUS = 0x%02x\n", __FUNCTION__, buf[0]);
+  if ((buf[0] & 0x0c) == 0x80) // Test signal does not exist flag
+    // Test AGC lock flag
+    *status |= FE_HAS_SIGNAL;
+  else
+    return 0;
+
+  /* Carrier Recovery Lock Status Register */
+  i2c_selectreadbytes(state, CARRIER_LOCK, buf, 1);
+  dprintk("%s: CARRIER_LOCK = 0x%02x\n", __FUNCTION__, buf[0]);
+  switch (state->current_modulation) {
+  case QAM_256:
+  case QAM_64:
+    // Need to undestand why there are 3 lock levels here
+    if ((buf[0] & 0x07) == 0x07)
+      *status |= FE_HAS_CARRIER;
+    else
+      return 0;
+    break;
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,10)
+  case VSB_8:
+    if ((buf[0] & 0x80) == 0x80)
+      *status |= FE_HAS_CARRIER;
+    else
+      return 0;
+    break;
+#endif
+  default:
+    printk("KERN_WARNING lgdt3302: %s: Modulation set to unsupported value\n", __FUNCTION__);
+  }
+#endif
+
+  return 0;
+}
+
+static int lgdt3302_read_signal_strength(struct dvb_frontend* fe, u16* strength)
+{
+  // not directly available.
+  return 0;
+}
+
+static int lgdt3302_read_snr(struct dvb_frontend* fe, u16* snr)
+{
+  /*
+   * Spec sheet shows formula for SNR_EQ = 10 log10(25 * 24**2 / noise)
+   * and SNR_PH = 10 log10(25 * 32**2 / noise) for equalizer and phase tracker
+   * respectively. The following tables are built on these formulas.
+   * The usual definition is SNR = 20 log10(signal/noise)
+   * If the specification is wrong the value retuned is 1/2 the actual SNR in db.
+   *
+   * This table is a an ordered list of noise values computed by the
+   * formula from the spec sheet such that the index into the table
+   * starting at 43 or 45 is the SNR value in db. There are duplicate noise
+   * value entries at the beginning because the SNR varies more than
+   * 1 db for a change of 1 digit in noise at very small values of noise.
+   * 
+   * Examples from SNR_EQ table:
+   * noise SNR
+   *   0    43  
+   *   1    42  
+   *   2    39  
+   *   3    37
+   *   4    36  
+   *   5    35  
+   *   6    34
+   *   7    33  
+   *   8    33  
+   *   9    32
+   *   10   32
+   *   11   31
+   *   12   31
+   *   13   30
+   */
+
+
+  static const u32 SNR_EQ[] =
+    { 1,     2,      2,      2, 3,      3,      4,     4,     5,     7,
+      9,     11,     13,     17, 21,     26,     33,    41,    52,    65,
+      81,    102,    129,    162, 204,    257,    323,   406,   511,   644,
+      810,   1020,   1284,   1616, 2035,   2561,   3224,  4059,  5110,  6433,
+      8098,  10195,  12835,  16158, 20341,  25608,  32238, 40585, 51094, 64323,
+      80978, 101945, 128341, 161571, 203406, 256073, 0x40000
+    };
+
+  static const u32 SNR_PH[] = 
+    { 1,     2,      2,      2,      3,      3,     4,     5,     6,     8,
+      10,    12,     15,     19,     23,     29, 37,    46,    58,    73,
+      91,    115,    144,    182,    229,    288, 362,   456,   574,   722,
+      909,   1144,   1440,   1813,   2282,   2873, 3617,  4553,  5732,  7216,
+      9084,  11436,  14396,  18124,  22817,  28724,  36161, 45524, 57312, 72151,
+      90833, 114351, 143960, 181235, 228161, 0x040000
+    }; 
+
+
+  static u8 buf[5];/* read data buffer */
+  static u32 noise;   /* noise value */
+  static u32 snr_db;  /* index into SNR_EQ[] */
+  struct lgdt3302_state* state = (struct lgdt3302_state*) fe->demodulator_priv;
+
+  /* read both equalizer and pase tracker noise data */ 
+  i2c_selectreadbytes(state, EQPH_ERR0, buf, sizeof(buf));
+
+  if (state->current_modulation == VSB_8) {
+    /* Equalizer Mean-Square Error Register for VSB */
+    noise = ((buf[0] & 7) << 16) | (buf[1] << 8) | buf[2];
+
+    /* 
+     * Look up noise value in table. 
+     * A better search algorithm could be used...
+     * watch out there are duplicate entries.
+     */
+    for (snr_db = 0; snr_db < sizeof(SNR_EQ); snr_db++) {
+      if (noise < SNR_EQ[snr_db]) {
+	*snr = 43 - snr_db;
+	break;
+      }
+    }
+  } else {
+    /* Phase Tracker Mean-Square Error Register for QAM */
+    noise = ((buf[0] & 7<<3) << 13) | (buf[3] << 8) | buf[4];
+
+    /* Look up noise value in table. */
+    for (snr_db = 0; snr_db < sizeof(SNR_PH); snr_db++) {
+      if (noise < SNR_PH[snr_db]) {
+	*snr = 45 - snr_db;
+	break;
+      }
+    }
+  }
+
+  dprintk("%s: noise = 0x%05x, snr = %idb\n",__FUNCTION__, noise, *snr);
+
+  return 0;
+}
+
+static int lgdt3302_get_tune_settings(struct dvb_frontend* fe, struct dvb_frontend_tune_settings* fe_tune_settings)
+{
+  /* I have no idea about this - it may not be needed */
+  fe_tune_settings->min_delay_ms = 500;
+  fe_tune_settings->step_size = 0;
+  fe_tune_settings->max_drift = 0;
+
+  return 0;
+}
+
+
+static void lgdt3302_release(struct dvb_frontend* fe)
+{
+  struct lgdt3302_state* state = (struct lgdt3302_state*) fe->demodulator_priv;
+  kfree(state);
+}
+
+static struct dvb_frontend_ops lgdt3302_ops;
+
+struct dvb_frontend* lgdt3302_attach(const struct lgdt3302_config* config,
+				     struct i2c_adapter* i2c)
+{
+  struct lgdt3302_state* state = NULL;
+
+  /* Allocate memory for the internal state */
+  state = (struct lgdt3302_state*) kmalloc(sizeof(struct lgdt3302_state), GFP_KERNEL);
+  if (state == NULL)
+    goto error;
+  memset(state,0,sizeof(*state));
+
+  /* Setup the state */
+  state->config = config;
+  state->i2c = i2c;
+  memcpy(&state->ops, &lgdt3302_ops, sizeof(struct dvb_frontend_ops));
+  /* Verify communication with demod chip */
+  u8 buf[1];
+  if (i2c_selectreadbytes(state, 2, buf, 1))
+    goto error;
+
+  state->current_frequency = -1;
+  state->current_modulation = -1;
+
+  /* Create dvb_frontend */
+  state->frontend.ops = &state->ops;
+  state->frontend.demodulator_priv = state;
+  return &state->frontend;
+
+ error:
+  if (state)
+    kfree(state);
+  dprintk("%s: ERROR\n",__FUNCTION__);
+  return NULL;
+}
+
+static struct dvb_frontend_ops lgdt3302_ops = {
+
+  .info = {
+    .name= "LG Electronics LGDT3302 VSB/QAM Frontend",
+    .type = FE_ATSC,
+    .frequency_min= 54000000,
+    .frequency_max= 858000000,
+    .frequency_stepsize= 62500,
+    /* Symbol rate is for all VSB modes need to check QAM */
+    .symbol_rate_min    = 10762000,
+    .symbol_rate_max    = 10762000,
+    .caps = FE_CAN_QAM_64 | FE_CAN_QAM_256 | FE_CAN_8VSB
+  },
+  .init                 = lgdt3302_init,
+  .set_frontend         = lgdt3302_set_parameters,
+  .get_frontend         = lgdt3302_get_frontend,
+  .get_tune_settings    = lgdt3302_get_tune_settings,
+  .read_status          = lgdt3302_read_status,
+  .read_ber             = lgdt3302_read_ber,
+  .read_signal_strength = lgdt3302_read_signal_strength,
+  .read_snr             = lgdt3302_read_snr,
+  .read_ucblocks        = lgdt3302_read_ucblocks,
+  .release              = lgdt3302_release,
+};
+
+MODULE_DESCRIPTION("LGDT3302 [DViCO FusionHDTV 3 Gold] (ATSC 8VSB & ITU-T J.83 AnnexB 64/256 QAM) Demodulator Driver");
+MODULE_AUTHOR("Wilson Michaels");
+MODULE_LICENSE("GPL");
+
+EXPORT_SYMBOL(lgdt3302_attach);
+
+/*
+ * Local variables:
+ * c-basic-offset: 8
+ * compile-command: "make DVB=1"
+ * End:
+ */
diff -upN video4linux.orig/lgdt3302.h video4linux/lgdt3302.h
--- video4linux.orig/lgdt3302.h	1970-01-01 00:00:00.000000000 +0000
+++ video4linux/lgdt3302.h	2005-06-15 20:03:15.000000000 +0000
@@ -0,0 +1,47 @@
+/*
+ *    Support for LGDT3302 (DViCO FustionHDTV 3 Gold) - VSB/QAM
+ *
+ *    Copyright (C) 2005 Wilson Michaels <wilsonmichaels at earthlink.net>
+ *
+ *    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., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+#ifndef LGDT3302_H
+#define LGDT3302_H
+
+#include <linux/dvb/frontend.h>
+
+struct lgdt3302_config
+{
+  /* The demodulator's i2c address */
+  u8 demod_address;
+  u8 pll_address;
+  struct dvb_pll_desc *pll_desc;
+
+  /* Need to set device param for start_dma */
+  int (*set_ts_params)(struct dvb_frontend* fe, int is_punctured);
+};
+
+extern struct dvb_frontend* lgdt3302_attach(const struct lgdt3302_config* config,
+					    struct i2c_adapter* i2c);
+
+#endif // LGDT3302_H
+
+/*
+ * Local variables:
+ * c-basic-offset: 8
+ * End:
+ */
diff -upN video4linux.orig/lgdt3302_priv.h video4linux/lgdt3302_priv.h
--- video4linux.orig/lgdt3302_priv.h	1970-01-01 00:00:00.000000000 +0000
+++ video4linux/lgdt3302_priv.h	2005-06-15 20:04:54.000000000 +0000
@@ -0,0 +1,70 @@
+/*
+ *    Support for LGDT3302 (DViCO FustionHDTV 3 Gold) - VSB/QAM
+ *
+ *    Copyright (C) 2005 Wilson Michaels <wilsonmichaels at earthlink.net>
+ *
+ *    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., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+#ifndef _LGDT3302_PRIV_
+#define _LGDT3302_PRIV_
+
+/* i2c control register addresses */
+enum I2C_REG {
+  TOP_CONTROL= 0x00,
+  IRQ_MASK= 0x01,
+  IRQ_STATUS= 0x02,
+  VSB_CARRIER_FREQ0= 0x16,
+  VSB_CARRIER_FREQ1= 0x17,
+  VSB_CARRIER_FREQ2= 0x18,
+  VSB_CARRIER_FREQ3= 0x19,
+  CARRIER_MSEQAM1= 0x1a,
+  CARRIER_MSEQAM2= 0x1b,
+  CARRIER_LOCK= 0x1c,
+  TIMING_RECOVERY= 0x1d,
+  AGC_DELAY0= 0x2a,
+  AGC_DELAY1= 0x2b,
+  AGC_DELAY2= 0x2c,
+  AGC_RF_BANDWIDTH0= 0x2d,
+  AGC_RF_BANDWIDTH1= 0x2e,
+  AGC_RF_BANDWIDTH2= 0x2f,
+  AGC_LOOP_BANDWIDTH0= 0x30,
+  AGC_LOOP_BANDWIDTH1= 0x31,
+  AGC_FUNC_CTRL1= 0x32,
+  AGC_FUNC_CTRL2= 0x33,
+  AGC_FUNC_CTRL3= 0x34,
+  AGC_RFIF_ACC0= 0x39, 
+  AGC_RFIF_ACC1= 0x3a,    
+  AGC_RFIF_ACC2= 0x3b,     
+  AGC_STATUS= 0x3f,
+  SYNC_STATUS_VSB= 0x43,
+  EQPH_ERR0= 0x47,
+  EQ_ERR1= 0x48,
+  EQ_ERR2= 0x49,
+  PH_ERR1= 0x4a,
+  PH_ERR2= 0x4b,
+  DEMUX_CONTROL= 0x66,
+  PACKET_ERR_COUNTER1= 0x6a,
+  PACKET_ERR_COUNTER2= 0x6b,
+};
+
+#endif // _LGDT3302_PRIV_
+
+/*
+ * Local variables:
+ * c-basic-offset: 8
+ * End:
+ */
diff -upN video4linux.orig/tuner-simple.c video4linux/tuner-simple.c
--- video4linux.orig/tuner-simple.c	2005-06-15 19:42:53.000000000 +0000
+++ video4linux/tuner-simple.c	2005-06-15 20:29:51.000000000 +0000
@@ -414,7 +414,7 @@ static void default_set_tv_freq(struct i
 
 	if (t->type == TUNER_MICROTUNE_4042FI5) {
 		// FIXME - this may also work for other tuners
-		unsigned long timeout = jiffies + msecs_to_jiffies(1);
+		unsigned long timeout = jiffies + msecs_to_jiffies(50);
 		u8 status_byte = 0;
 
 		/* Wait until the PLL locks */


More information about the linux-dvb mailing list