Re: cx22702.c, 2.6.13-rc3 and a pci Hauppauge Nova-T DVB-T TV card

[Andrew Benton <b3nt@ukonline.co.uk>]

Patrick Boettcher wrote:
> On Fri, 15 Jul 2005, Johannes Stezenbach wrote:
>
> > 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?
>
> Hmm, yes. When I changed the cx22702-driver to make it work with the
> cxusb-driver, I added another field to the struct cx22702_config to
> determine the output type.
>
> I was well aware that this breaks support for the PCI cards, that's why I
> created a patch for the cx88-dvb.c and posted it the v4l-mailing list and
> ask for inclusion.
>
> This was the Mail:
> http://www.linuxtv.org/pipermail/linux-dvb/2005-June/002383.html
>
> This is the patch:
> Index: cx88-dvb.c
> ===================================================================
> RCS file: /cvs/video4linux/video4linux/cx88-dvb.c,v
> retrieving revision 1.42
> diff -u -3 -p -r1.42 cx88-dvb.c
> --- cx88-dvb.c    12 Jul 2005 15:44:55 -0000    1.42
> +++ cx88-dvb.c    15 Jul 2005 11:06:22 -0000
> @@ -166,12 +166,14 @@ static int mt352_pll_set(struct dvb_fron
>
>  static struct mt352_config dvico_fusionhdtv = {
>      .demod_address = 0x0F,
> +    .output_mode   = CX22702_SERIAL_OUTPUT,
>      .demod_init    = dvico_fusionhdtv_demod_init,
>      .pll_set       = mt352_pll_set,
>  };
>
>  static struct mt352_config dntv_live_dvbt_config = {
>      .demod_address = 0x0f,
> +    .output_mode   = CX22702_SERIAL_OUTPUT,
>      .demod_init    = dntv_live_dvbt_demod_init,
>      .pll_set       = mt352_pll_set,
>  };
>
> Please include. Thanks
>
> Signed-off-by: Patrick Boettcher <pb@linuxtv.org>
>
> best regards,
> Patrick.
>
> --
>   Mail: patrick.boettcher@desy.de
>   WWW:  http://www.wi-bw.tfh-wildau.de/~pboettch/

Hi, I tried the patch but unfortunately the kernel didn't compile, it ended like this

 CC      drivers/media/video/cx88/cx88-blackbird.o
 CC      drivers/media/video/cx88/cx88-dvb.o
drivers/media/video/cx88/cx88-dvb.c:169: error: unknown field `output_mode' specified in initializer
drivers/media/video/cx88/cx88-dvb.c:176: error: unknown field `output_mode' specified in initializer
make[4]: *** [drivers/media/video/cx88/cx88-dvb.o] Error 1
make[3]: *** [drivers/media/video/cx88] Error 2
make[2]: *** [drivers/media/video] Error 2
make[1]: *** [drivers/media] Error 2
make: *** [drivers] Error 2
andy:~$
-
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/