Mailing List archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[linux-dvb] cx22702 patch
This does several things (I thought I should run it past you first to check
everything is ok) :
* Fixes a few coding errors.
* Removes hardcoded i2c addresses.
* Uncommented bandwidth setting on tuner.
* Finalise DTT7579 support (except for unknown/missing bandwidth setting)
A couple of questions for you:
* Where does 264000000 come from in pll_dtt759x_set_tv_freq - why not use the
charge pump settings given in the datasheet in the VHF range?
* "dvbtune fails to set LP correctly, default to HP". IMO, dvbtune should be
fixed, and the driver do the correct behaviour.
* why is the AUTO configuration commented out??
I hope to start using it all here today.
Index: cx22702.c
===================================================================
RCS file: /cvs/linuxtv/dvb-kernel/linux/drivers/media/dvb/frontends/cx22702.c,v
retrieving revision 1.2
diff -a -u -r1.2 cx22702.c
--- cx22702.c 6 Sep 2004 17:00:21 -0000 1.2
+++ cx22702.c 9 Sep 2004 08:02:59 -0000
@@ -36,9 +36,7 @@
#define FRONTEND_NAME "dvbfe_cx22702"
-#define I2C_DEMOD_SLAVE_ADDR 0x43
#define I2C_EEPROM_SLAVE_ADDR 0x50
-#define I2C_PLL_SLAVE_ADDR 0x61
#define PLLTYPE_DTT7592 1
#define PLLTYPE_DTT7595 2
@@ -49,7 +47,7 @@
#define dprintk if (debug) printk
static struct dvb_frontend_info cx22702_info = {
- .name = "CX22702 Demod Thompson 759x/7579 PLL",
+ .name = "CX22702 Demod Thomson 759x/7579 PLL",
.type = FE_OFDM,
.frequency_min = 177000000,
.frequency_max = 858000000,
@@ -65,6 +63,8 @@
struct dvb_adapter *dvb;
struct dvb_frontend_info cx22702_info;
char pll_type;
+ int pll_addr;
+ int demod_addr;
};
/* Register values to initialise the demod */
@@ -83,11 +83,11 @@
static struct i2c_client client_template;
-static int cx22702_writereg (struct i2c_adapter *i2c, u8 reg, u8 data)
+static int cx22702_writereg (struct i2c_adapter *i2c, int demod_addr, u8 reg, u8 data)
{
int ret;
u8 buf [] = { reg, data };
- struct i2c_msg msg = { .addr = I2C_DEMOD_SLAVE_ADDR, .flags = 0, .buf = buf, .len = 2 };
+ struct i2c_msg msg = { .addr = demod_addr, .flags = 0, .buf = buf, .len = 2 };
ret = i2c_transfer(i2c, &msg, 1);
@@ -98,15 +98,15 @@
return (ret != 1) ? -1 : 0;
}
-static u8 cx22702_readreg (struct i2c_adapter *i2c, u8 reg)
+static u8 cx22702_readreg (struct i2c_adapter *i2c, int demod_addr, u8 reg)
{
int ret;
u8 b0 [] = { reg };
u8 b1 [] = { 0 };
struct i2c_msg msg [] = {
- { .addr = I2C_DEMOD_SLAVE_ADDR, .flags = 0, .buf = b0, .len = 1 },
- { .addr = I2C_DEMOD_SLAVE_ADDR, .flags = I2C_M_RD, .buf = b1, .len = 1 } };
+ { .addr = demod_addr, .flags = 0, .buf = b0, .len = 1 },
+ { .addr = demod_addr, .flags = I2C_M_RD, .buf = b1, .len = 1 } };
ret = i2c_transfer(i2c, msg, 2);
@@ -116,14 +116,14 @@
return b1[0];
}
-static int pll_readreg(struct i2c_adapter *i2c, u8 reg)
+static int pll_readreg(struct i2c_adapter *i2c, int pll_addr, u8 reg)
{
u8 b0 [] = { reg };
u8 b1 [] = { 0 };
struct i2c_msg msg [] = {
- { .addr = I2C_PLL_SLAVE_ADDR, .flags = 0, .buf = b0, .len = 1 },
- { .addr = I2C_PLL_SLAVE_ADDR, .flags = I2C_M_RD, .buf = b1, .len = 1 }
+ { .addr = pll_addr, .flags = 0, .buf = b0, .len = 1 },
+ { .addr = pll_addr, .flags = I2C_M_RD, .buf = b1, .len = 1 }
};
/* cx22702_writereg (i2c, 0x0D, cx22702_readreg(i2c,0x0D) &0xfe); // Enable PLL bus */
@@ -136,10 +136,10 @@
return b1[0];
}
-static int pll_write (struct i2c_adapter *i2c, u8 data [4])
+static int pll_write (struct i2c_adapter *i2c, int pll_addr, u8 data [4])
{
int ret=0;
- struct i2c_msg msg = { .addr = I2C_PLL_SLAVE_ADDR, .flags = 0, .buf = data, .len = 4 };
+ struct i2c_msg msg = { .addr = pll_addr, .flags = 0, .buf = data, .len = 4 };
/* cx22702_writereg (i2c, 0x0D, cx22702_readreg(i2c,0x0D) &0xfe); // Enable PLL bus */
ret = i2c_transfer(i2c, &msg, 1);
@@ -151,13 +151,13 @@
return (ret != 1) ? -1 : 0;
}
-static int pll_dtt759x_set_tv_freq (struct i2c_adapter *i2c, u32 freq)
+static int pll_dtt759x_set_tv_freq (struct i2c_adapter *i2c, struct cx22702_state *state, u32 freq, int bandwidth)
{
int ret;
u32 div = (freq + 36166667) / 166666;
- /* dividerhigh, divierlow, control, bandwidth switch tuner args */
+ /* dividerhigh, dividerlow, control, bandwidth switch tuner args */
unsigned char buf [4] = {
(div >> 8) & 0x7f,
div & 0xff,
@@ -171,13 +171,11 @@
buf[3] = 0x08;
}
-#if 0
- if(bandw == BANDWIDTH_7MHz) {
- buf[4] |= 0x10;
+ if(bandwidth == BANDWIDTH_7_MHZ) {
+ buf[3] |= 0x10;
}
-#endif
- // Now compensate for the charge pump osc
+ // Now compensate for the charge pump osc
if(freq <= 264000000) {
buf[2] = buf[2] | 0x30;
} else if (freq <= 735000000) {
@@ -188,30 +186,29 @@
buf[2] = buf[2] | 0x78;
}
- buf[2]=0xbc;
-
dprintk ("%s: freq == %i, div == 0x%04x\n", __FUNCTION__, (int) freq, (int) div);
- ret= pll_write (i2c, buf);
+ ret= pll_write (i2c, state->pll_addr, buf);
if(ret<0) {
dprintk ("%s: first pll_write failed\n",__FUNCTION__);
return ret;
}
- /* Set the AGC to search */
+ /* Set the AGC during search */
buf[2]=0x9c;
buf[3]=0xa0;
- ret=pll_write (i2c, buf);
+ ret=pll_write (i2c, state->pll_addr, buf);
if(ret<0) {
dprintk ("%s: second pll_write failed\n",__FUNCTION__);
return ret;
}
/* Tuner needs a small amount of time */
- __udelay(100*1000);
+ msleep(100);
+ /* Set the AGC post-search */
buf[3]=0x20;
- ret=pll_write (i2c, buf);
+ ret=pll_write (i2c, state->pll_addr, buf);
if(ret<0) {
dprintk ("%s: third pll_write failed\n",__FUNCTION__);
return ret;
@@ -221,13 +218,13 @@
}
-static int pll_dtt7579_set_tv_freq (struct i2c_adapter *i2c, u32 freq)
+static int pll_dtt7579_set_tv_freq (struct i2c_adapter *i2c, struct cx22702_state *state, u32 freq, int bandwidth)
{
int ret;
u32 div = (freq + 36166667) / 166666;
- /* dividerhigh, divierlow */
+ /* dividerhigh, dividerlow */
unsigned char buf [4] = {
div >> 8,
div & 0xff,
@@ -235,6 +232,8 @@
0x00
};
+ // FIXME: bandwidth setting unknown
+
// Now compensate for the charge pump osc
if(freq <= 506000000) {
buf[2] = 0xb4;
@@ -252,7 +251,7 @@
dprintk ("%s: freq == %i, div == 0x%04x\n", __FUNCTION__, (int) freq, (int) div);
- ret= pll_write (i2c, buf);
+ ret= pll_write (i2c, state->pll_addr, buf);
if(ret<0) {
dprintk ("%s: first pll_write failed\n",__FUNCTION__);
return ret;
@@ -261,7 +260,7 @@
/* Set the AGC to search */
buf[2]=(buf[2] & 0xdc) | 0x9c;
buf[3]=0xa0;
- ret=pll_write (i2c, buf);
+ ret=pll_write (i2c, state->pll_addr, buf);
if(ret<0) {
dprintk ("%s: second pll_write failed\n",__FUNCTION__);
return ret;
@@ -273,23 +272,23 @@
/* Reset the demod hardware and reset all of the configuration registers
to a default state. */
-static int cx22702_init (struct i2c_adapter *i2c)
+static int cx22702_init (struct i2c_adapter *i2c, struct cx22702_state *state)
{
int i;
- cx22702_writereg (i2c, 0x00, 0x02);
- cx22702_writereg (i2c, 0x00, 0x00);
+ cx22702_writereg (i2c, state->demod_addr, 0x00, 0x02);
+ cx22702_writereg (i2c, state->demod_addr, 0x00, 0x00);
msleep(10);
for (i=0; i<sizeof(init_tab); i+=2)
- cx22702_writereg (i2c, init_tab[i], init_tab[i+1]);
+ cx22702_writereg (i2c, state->demod_addr, init_tab[i], init_tab[i+1]);
- cx22702_writereg (i2c, 0x00, 0x01);
+ cx22702_writereg (i2c, state->demod_addr, 0x00, 0x01);
return 0;
}
-static int cx22702_set_inversion (struct i2c_adapter *i2c, int inversion)
+static int cx22702_set_inversion (struct i2c_adapter *i2c, struct cx22702_state *state, int inversion)
{
u8 val;
@@ -299,12 +298,12 @@
return -EOPNOTSUPP;
case INVERSION_ON:
- val = cx22702_readreg (i2c, 0x0C);
- return cx22702_writereg (i2c, 0x0C, val | 0x01);
+ val = cx22702_readreg (i2c, state->demod_addr, 0x0C);
+ return cx22702_writereg (i2c, state->demod_addr, 0x0C, val | 0x01);
case INVERSION_OFF:
- val = cx22702_readreg (i2c, 0x0C);
- return cx22702_writereg (i2c, 0x0C, val & 0xfe);
+ val = cx22702_readreg (i2c, state->demod_addr, 0x0C);
+ return cx22702_writereg (i2c, state->demod_addr, 0x0C, val & 0xfe);
default:
return -EINVAL;
@@ -314,20 +313,20 @@
}
/* Talk to the demod, set the FEC, GUARD, QAM settings etc */
-static int cx22702_set_tps (struct i2c_adapter *i2c,
- struct dvb_ofdm_parameters *p)
+static int cx22702_set_tps (struct i2c_adapter *i2c, struct cx22702_state *state,
+ struct dvb_ofdm_parameters *p)
{
u8 val;
switch(p->bandwidth) {
case BANDWIDTH_6_MHZ:
- cx22702_writereg(i2c, 0x0C, cx22702_readreg(i2c, 0x0C) | 0x20 );
+ cx22702_writereg(i2c, state->demod_addr, 0x0C, (cx22702_readreg(i2c, state->demod_addr, 0x0C) & 0xcf) | 0x20 );
break;
case BANDWIDTH_7_MHZ:
- cx22702_writereg(i2c, 0x0C, cx22702_readreg(i2c, 0x0C) | 0x10 );
+ cx22702_writereg(i2c, state->demod_addr, 0x0C, (cx22702_readreg(i2c, state->demod_addr, 0x0C) & 0xcf) | 0x10 );
break;
case BANDWIDTH_8_MHZ:
- cx22702_writereg(i2c, 0x0C, cx22702_readreg(i2c, 0x0C) &0xcf );
+ cx22702_writereg(i2c, state->demod_addr, 0x0C, cx22702_readreg(i2c, state->demod_addr, 0x0C) &0xcf );
break;
default:
dprintk ("%s: invalid bandwidth\n",__FUNCTION__);
@@ -345,8 +344,8 @@
(p->transmission_mode==TRANSMISSION_MODE_AUTO) ) {
/* TPS Source - use hardware driven values */
- cx22702_writereg(i2c, 0x0B, cx22702_readreg(i2c, 0x0B) & 0xfd );
- cx22702_writereg(i2c, 0x00, 0x01); /* Begin aquisition */
+ cx22702_writereg(i2c, state->demod_addr, 0x0B, cx22702_readreg(i2c, state->demod_addr, 0x0B) & 0xfd );
+ cx22702_writereg(i2c, state->demod_addr, 0x00, 0x01); /* Begin aquisition */
printk("%s: Autodetecting\n",__FUNCTION__);
return 0;
}
@@ -354,9 +353,9 @@
val=0;
switch(p->constellation) {
- case QPSK: val &= 0xe7; break;
- case QAM_16: val |= 0x08; break;
- case QAM_64: val |= 0x10; break;
+ case QPSK: val = (val&0xe7); break;
+ case QAM_16: val = (val&0xe7)|0x08; break;
+ case QAM_64: val = (val&0xe7)|0x10; break;
default:
dprintk ("%s: invalid constellation\n",__FUNCTION__);
return -EINVAL;
@@ -370,7 +369,7 @@
dprintk ("%s: invalid hierarchy\n",__FUNCTION__);
return -EINVAL;
}
- cx22702_writereg (i2c, 0x06, val);
+ cx22702_writereg (i2c, state->demod_addr, 0x06, val);
val=0;
switch(p->code_rate_HP) {
@@ -397,7 +396,7 @@
dprintk ("%s: invalid code_rate_LP\n",__FUNCTION__);
return -EINVAL;
}
- cx22702_writereg (i2c, 0x07, val);
+ cx22702_writereg (i2c, state->demod_addr, 0x07, val);
val=0;
switch(p->guard_interval) {
@@ -416,28 +415,28 @@
dprintk ("%s: invalid transmission_mode\n",__FUNCTION__);
return -EINVAL;
}
- cx22702_writereg (i2c, 0x08, val);
+ cx22702_writereg (i2c, state->demod_addr, 0x08, val);
/* Force TPS Source - use user given values */
- cx22702_writereg(i2c, 0x0B, cx22702_readreg(i2c, 0x0B) | 0x02 );
+ cx22702_writereg(i2c, state->demod_addr, 0x0B, (cx22702_readreg(i2c, state->demod_addr, 0x0B) & 0xfc) | 0x02 );
/* Begin channel aquisition */
- cx22702_writereg(i2c, 0x00, 0x01);
+ cx22702_writereg(i2c, state->demod_addr, 0x00, 0x01);
return 0;
}
/* Retrieve the demod settings */
-static int cx22702_get_tps (struct i2c_adapter *i2c,
- struct dvb_ofdm_parameters *p)
+static int cx22702_get_tps (struct i2c_adapter *i2c, struct cx22702_state *state,
+ struct dvb_ofdm_parameters *p)
{
u8 val;
/* Make sure the TPS regs are valid */
- if (!(cx22702_readreg(i2c, 0x0A) & 0x20))
+ if (!(cx22702_readreg(i2c, state->demod_addr, 0x0A) & 0x20))
return -EAGAIN;
- val = cx22702_readreg (i2c, 0x01);
+ val = cx22702_readreg (i2c, state->demod_addr, 0x01);
switch( (val&0x18)>>3) {
case 0: p->constellation = QPSK; break;
case 1: p->constellation = QAM_16; break;
@@ -451,7 +450,7 @@
}
- val = cx22702_readreg (i2c, 0x02);
+ val = cx22702_readreg (i2c, state->demod_addr, 0x02);
switch( (val&0x38)>>3 ) {
case 0: p->code_rate_HP = FEC_1_2; break;
case 1: p->code_rate_HP = FEC_2_3; break;
@@ -468,7 +467,7 @@
}
- val = cx22702_readreg (i2c, 0x03);
+ val = cx22702_readreg (i2c, state->demod_addr, 0x03);
switch( (val&0x0c)>>2 ) {
case 0: p->guard_interval = GUARD_INTERVAL_1_32; break;
case 1: p->guard_interval = GUARD_INTERVAL_1_16; break;
@@ -501,11 +500,11 @@
*status = 0;
- reg0A = cx22702_readreg (i2c, 0x0A);
- reg23 = cx22702_readreg (i2c, 0x23);
+ reg0A = cx22702_readreg (i2c, state->demod_addr, 0x0A);
+ reg23 = cx22702_readreg (i2c, state->demod_addr, 0x23);
dprintk ("%s: status demod=0x%02x agc=0x%02x pll=0x%02x\n"
- ,__FUNCTION__,reg0A,reg23,pll_readreg(i2c,0xc2));
+ ,__FUNCTION__,reg0A,reg23,pll_readreg(i2c,state->pll_addr,0xc2));
if(reg0A & 0x10) {
*status |= FE_HAS_LOCK;
@@ -524,20 +523,20 @@
}
case FE_READ_BER:
- if(cx22702_readreg (i2c, 0xE4) & 0x02) {
+ if(cx22702_readreg (i2c, state->demod_addr, 0xE4) & 0x02) {
/* Realtime statistics */
- *((u32*) arg) = (cx22702_readreg (i2c, 0xDE) & 0x7F) << 7
- | (cx22702_readreg (i2c, 0xDF)&0x7F);
+ *((u32*) arg) = (cx22702_readreg (i2c, state->demod_addr, 0xDE) & 0x7F) << 7
+ | (cx22702_readreg (i2c, state->demod_addr, 0xDF)&0x7F);
} else {
/* Averagtine statistics */
- *((u32*) arg) = (cx22702_readreg (i2c, 0xDE) & 0x7F) << 7
- | cx22702_readreg (i2c, 0xDF);
+ *((u32*) arg) = (cx22702_readreg (i2c, state->demod_addr, 0xDE) & 0x7F) << 7
+ | cx22702_readreg (i2c, state->demod_addr, 0xDF);
}
break;
case FE_READ_SIGNAL_STRENGTH:
{
- u16 ss = cx22702_readreg (i2c, 0x23);
+ u16 ss = cx22702_readreg (i2c, state->demod_addr, 0x23);
*((u16*) arg) = ss;
break;
}
@@ -547,14 +546,14 @@
case FE_READ_SNR:
{
u16 rs_ber=0;
- if(cx22702_readreg (i2c, 0xE4) & 0x02) {
+ if(cx22702_readreg (i2c, state->demod_addr, 0xE4) & 0x02) {
/* Realtime statistics */
- rs_ber = (cx22702_readreg (i2c, 0xDE) & 0x7F) << 7
- | (cx22702_readreg (i2c, 0xDF)& 0x7F);
+ rs_ber = (cx22702_readreg (i2c, state->demod_addr, 0xDE) & 0x7F) << 7
+ | (cx22702_readreg (i2c, state->demod_addr, 0xDF)& 0x7F);
} else {
/* Averagine statistics */
- rs_ber = (cx22702_readreg (i2c, 0xDE) & 0x7F) << 8
- | cx22702_readreg (i2c, 0xDF);
+ rs_ber = (cx22702_readreg (i2c, state->demod_addr, 0xDE) & 0x7F) << 8
+ | cx22702_readreg (i2c, state->demod_addr, 0xDF);
}
*((u16*) arg) = ~rs_ber;
break;
@@ -562,8 +561,8 @@
case FE_READ_UNCORRECTED_BLOCKS:
/* RS Uncorrectable Packet Count then reset */
- *((u32*) arg) = cx22702_readreg (i2c, 0xE3);
- cx22702_writereg (i2c, 0xE3, 0x00);
+ *((u32*) arg) = cx22702_readreg (i2c, state->demod_addr, 0xE3);
+ cx22702_writereg (i2c, state->demod_addr, 0xE3, 0x00);
break;
case FE_SET_FRONTEND:
@@ -572,40 +571,40 @@
int ret;
/* Disable aquisition */
- cx22702_writereg (i2c, 0x00, 0x00);
+ cx22702_writereg (i2c, state->demod_addr, 0x00, 0x00);
switch(state->pll_type) {
case PLLTYPE_DTT7592:
case PLLTYPE_DTT7595:
- pll_dtt759x_set_tv_freq (i2c, p->frequency);
+ pll_dtt759x_set_tv_freq (i2c, state, p->frequency, p->u.ofdm.bandwidth);
break;
case PLLTYPE_DTT7579:
- pll_dtt7579_set_tv_freq (i2c, p->frequency);
+ pll_dtt7579_set_tv_freq (i2c, state, p->frequency, p->u.ofdm.bandwidth);
break;
}
- cx22702_set_inversion (i2c, p->inversion);
- if((ret=cx22702_set_tps (i2c, &p->u.ofdm))<0) {
+ cx22702_set_inversion (i2c, state, p->inversion);
+ if((ret=cx22702_set_tps (i2c, state, &p->u.ofdm))<0) {
dprintk ("%s: set_tps failed ret=%d\n",__FUNCTION__,ret);
return ret;
}
/* Enable aquisition */
- cx22702_writereg (i2c, 0x00, 0x01);
+ cx22702_writereg (i2c, state->demod_addr, 0x00, 0x01);
break;
}
case FE_GET_FRONTEND:
{
struct dvb_frontend_parameters *p = arg;
- u8 reg0C = cx22702_readreg (i2c, 0x0C);
+ u8 reg0C = cx22702_readreg (i2c, state->demod_addr, 0x0C);
p->inversion = reg0C & 0x1 ? INVERSION_ON : INVERSION_OFF;
- return cx22702_get_tps (i2c, &p->u.ofdm);
+ return cx22702_get_tps (i2c, state, &p->u.ofdm);
}
case FE_INIT:
- return cx22702_init (i2c);
+ return cx22702_init (i2c, state);
default:
return -EOPNOTSUPP;
@@ -616,7 +615,7 @@
/* Validate the eeprom contents, make sure content look ok.
Get the eeprom data. */
-static int cx22702_validate_eeprom(struct i2c_adapter *i2c, int* minfreq, int* pll_type)
+static int cx22702_validate_eeprom(struct i2c_adapter *i2c, int* minfreq, int* pll_type, int* pll_addr, int* demod_addr)
{
u8 b0 [] = { 0 };
u8 b1 [128];
@@ -669,12 +668,12 @@
tuner = b1[0x2d];
switch(tuner) {
case 0x4B:
- dprintk ("%s: Tuner Thompson DTT 7595\n",__FUNCTION__);
+ dprintk ("%s: Tuner Thomson DTT 7595\n",__FUNCTION__);
*minfreq = 177000000;
*pll_type = PLLTYPE_DTT7595;
break;
case 0x4C:
- dprintk ("%s: Tuner Thompson DTT 7592\n",__FUNCTION__);
+ dprintk ("%s: Tuner Thomson DTT 7592\n",__FUNCTION__);
*minfreq = 474000000;
*pll_type = PLLTYPE_DTT7592;
break;
@@ -682,20 +681,21 @@
printk ("%s: Unknown tuner 0x%02x not supported\n",__FUNCTION__,tuner);
return -ENODEV;
}
-
+ *pll_addr = 0x61;
+ *demod_addr = 0x43;
return 0;
}
/* Validate the demod, make sure we understand the hardware */
-static int cx22702_validate_demod(struct i2c_adapter *i2c)
+static int cx22702_validate_demod(struct i2c_adapter *i2c, int demod_addr)
{
u8 b0 [] = { 0x1f };
u8 b1 [] = { 0 };
struct i2c_msg msg [] = {
- { .addr = I2C_DEMOD_SLAVE_ADDR, .flags = 0, .buf = b0, .len = 1 },
- { .addr = I2C_DEMOD_SLAVE_ADDR, .flags = I2C_M_RD, .buf = b1, .len = 1 }
+ { .addr = demod_addr, .flags = 0, .buf = b0, .len = 1 },
+ { .addr = demod_addr, .flags = I2C_M_RD, .buf = b1, .len = 1 }
};
if (i2c_transfer(i2c, msg, 2) != 2) {
@@ -712,11 +712,11 @@
}
/* Validate the tuner PLL, make sure we understand the hardware */
-static int cx22702_validate_pll(struct i2c_adapter *i2c)
+static int cx22702_validate_pll(struct i2c_adapter *i2c, int pll_addr)
{
int result=0;
- if( (result=pll_readreg(i2c,0xc2)) < 0)
+ if( (result=pll_readreg(i2c,pll_addr,0xc2)) < 0)
return result;
if( (result >= 0) && (result&0x30) )
@@ -733,6 +733,8 @@
int ret;
int minfreq;
int pll_type;
+ int pll_addr;
+ int demod_addr;
dprintk("Trying to attach to adapter 0x%x:%s.\n",
adapter->id, adapter->name);
@@ -740,19 +742,21 @@
if (!strcmp(adapter->name, "Conexant DVB-t")) {
printk("cx22702: Detected Conexant DVB-t card - PLL Thomson DTT7579\n");
pll_type = PLLTYPE_DTT7579;
- minfreq = 474000000; // guess
- } else {
- // FIXME
- ret=cx22702_validate_eeprom(adapter, &minfreq, &pll_type);
+ pll_addr = 0xC0;
+ demod_addr = 0x43;
+ minfreq = 177000000; // guess
+ } else if (!strcmp(adapter->name, "Hauppuage Nova-T DVB-t")) {
+ printk("cx22702: Detected Hauppuage Nova-T DVB-t - PLL Thomson DTT759x\n");
+ ret=cx22702_validate_eeprom(adapter, &minfreq, &pll_type, &pll_addr, &demod_addr);
if(ret < 0)
return ret;
}
- ret=cx22702_validate_demod(adapter);
+ ret=cx22702_validate_demod(adapter, demod_addr);
if(ret < 0)
return ret;
- ret=cx22702_validate_pll(adapter);
+ ret=cx22702_validate_pll(adapter, pll_addr);
if(ret < 0)
return ret;
@@ -764,6 +768,8 @@
memcpy(&state->cx22702_info, &cx22702_info, sizeof(struct dvb_frontend_info));
state->cx22702_info.frequency_min = minfreq;
state->pll_type = pll_type;
+ state->pll_addr = pll_addr;
+ state->demod_addr = demod_addr;
if ( !(client = kmalloc(sizeof(struct i2c_client), GFP_KERNEL)) ) {
kfree(state);
@@ -772,7 +778,7 @@
memcpy(client, &client_template, sizeof(struct i2c_client));
client->adapter = adapter;
- client->addr = I2C_DEMOD_SLAVE_ADDR;
+ client->addr = state->pll_addr;
i2c_set_clientdata(client, state);
if ((ret = i2c_attach_client(client))) {
@@ -855,7 +861,7 @@
MODULE_PARM(debug,"i");
MODULE_PARM_DESC(debug, "Enable verbose debug messages");
-MODULE_DESCRIPTION("CX22702 / Thompson DTT 759x / Thompson DTT 7579 PLL DVB Frontend driver");
+MODULE_DESCRIPTION("CX22702 / Thomson DTT 759x / Thomson DTT 7579 PLL DVB Frontend driver");
MODULE_AUTHOR("Steven Toth");
MODULE_LICENSE("GPL");
Home |
Main Index |
Thread Index