lkml.org 
[lkml]   [2005]   [Jul]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: cx22702.c, 2.6.13-rc3 and a pci Hauppauge Nova-T DVB-T TV card
Andrew Benton wrote:
> My pci TV card (a Hauppauge Nova-T DVB-T) works fine with a 2.6.13-rc2
> kernel but won't work with a 2.6.13-rc3 by a process of elimination I've
> found that if I reverse this part of the 2.6.13-rc3 patch the card works
> fine. Please do not include this in the 2.6.13 kernel.

Reversing this patch is not the right fix as it would break
support for the cxusb.c driver. I guess the output_mode needs to
be set for the Hauppauge Nova-T DVB-T (cx88-dvb.c).
cx88-dvb.c is in video4linux CVS, not DVB CVS.

Patrick, can you send a patch for this?

Johannes

> diff --git a/drivers/media/dvb/frontends/cx22702.c
> b/drivers/media/dvb/frontends/cx22702.c
> --- a/drivers/media/dvb/frontends/cx22702.c
> +++ b/drivers/media/dvb/frontends/cx22702.c
> @@ -76,7 +76,6 @@ static u8 init_tab [] = {
> 0x49, 0x56,
> 0x6b, 0x1e,
> 0xc8, 0x02,
> - 0xf8, 0x02,
> 0xf9, 0x00,
> 0xfa, 0x00,
> 0xfb, 0x00,
> @@ -347,10 +346,11 @@ static int cx22702_init (struct dvb_fron
> for (i=0; i<sizeof(init_tab); i+=2)
> cx22702_writereg (state, init_tab[i], init_tab[i+1]);
>
> + cx22702_writereg (state, 0xf8, (state->config->output_mode << 1) & 0x02);
>
> /* init PLL */
> if (state->config->pll_init) {
> - cx22702_writereg (state, 0x0D, cx22702_readreg(state,0x0D) &0xfe);
> + cx22702_writereg (state, 0x0D, cx22702_readreg(state,0x0D) & 0xfe);
> state->config->pll_init(fe);
> cx22702_writereg (state, 0x0D, cx22702_readreg(state,0x0D) | 1);
> }
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-07-15 13:17    [W:1.982 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site