Re: [PATCH][RESEND] New type of DTV2000H TV Card

From: Nicolas Marot
Date: Sat Aug 30 2008 - 18:36:19 EST


Hi Mauro,
The patch looks good for me.
Tested under 2.6.25 and 2.6.26

Reviewed-by: Nicolas Marot <nicolas.marot@xxxxxxxxx>

On Sun, Jun 29, 2008 at 2:13 PM, Mauro Carvalho Chehab
<mchehab@xxxxxxxxxxxxx> wrote:
>
> On Sat, 28 Jun 2008 19:56:37 +0200
> Zbynek Hrabovsky <hrabosh@xxxxxxxxxx> wrote:
>
> >
> > Hi Nicolas,
> >
> > so I should install git and run these commands, or .. ? I'm really confuzzled.
> >
> > But I had installed GPG and signed my patch, maybe this is what I should have done earlier. My key ID is: 7803B3E1
>
> We don't need a GPG signature. All we need is a "Signed-off-by".
>
> Please take a look at:
> http://linuxtv.org/hg/v4l-dvb/raw-file/tip/README.patches
>
> There are a quick explanation there on how we generally receive V4L/DVB
> patches. It also points to some docs at kernel Documentation, with the current
> rules for SubmittingPatches.
>
> Cheers,
> Mauro.
>
> >
> > If you guys want me somethink more to do, write me ... I'll do my best.
> >
> > Thanx for your time
> >
> > Zbynek
> >
> > On Fri, 20 Jun 2008 22:52:02 +0200
> > "Nicolas Marot" <nicolas.marot@xxxxxxxxx> wrote:
> >
> > > Hi Zbynek,
> > >
> > > I think lines Mauro asks us are some git commands on driver source code
> > >
> > > nicolas
> > >
> > >
> > > On Fri, Jun 20, 2008 at 11:52 PM, Zbynek Hrabovsky <hrabosh@xxxxxxxxxx>
> > > wrote:
> > >
> > > > Hi Mauro,
> > > >
> > > > thanks for your answer.
> > > >
> > > > I've tried to apply the updated patch you've sent me and it is working, but
> > > > ....
> > > >
> > > > ... there is a little problem with this:
> > > >
> > > > --- a/linux/drivers/media/video/cx88/cx88-mpeg.c Tue Jun 10 15:27:29
> > > > 2008 -0300
> > > > +++ b/linux/drivers/media/video/cx88/cx88-mpeg.c Fri Jun 13 15:07:34
> > > > 2008 -0300
> > > > @@ -148,6 +148,12 @@
> > > > cx_write(TS_SOP_STAT, 0);
> > > > cx_write(TS_VALERR_CNTRL, 0);
> > > > udelay(100);
> > > > + break;
> > > > + case CX88_BOARD_WINFAST_DTV2000H_2:
> > > > + /* switch signal input to antena */
> > > > + cx_write(MO_GP0_IO, 0x00017300);
> > > > +
> > > > + cx_write(TS_SOP_STAT, 0x00);
> > > > break;
> > > > default:
> > > > cx_write(TS_SOP_STAT, 0x00);
> > > >
> > > >
> > > > This piece of code used to switch the input of RF signal to "Air Antenna"
> > > > mode. When I was writing the patch, this code was executed every time I was
> > > > tunning DVB-T station. (using Kaffeine, or sth.) But now ( I don't know why
> > > > ... ) this code is ran only if the station is succesfully tuned ... which is
> > > > late for switching RF input.
> > > >
> > > > So ... all the things about sound in analog TV and video inputs, etc. are
> > > > working well, but switching between Cable and Air Antenna RF inputs must be
> > > > done using programs for analog TV ... such as TvTime for example.
> > > >
> > > > It would be nice, if this patch (and driver) would be able to switch RF
> > > > inputs in DVB-T mode somehow. My idea is, to have two DVB-T devices ... one
> > > > would use Air Antenna input as a source of signal, second the Cable input as
> > > > source of signal. This will provide us to receive DVB-T from both inputs
> > > > (BTW ... original Windows driver is not able to do this). The problem is,
> > > > that I'm not sure if I'm able to do it ... I have very poor idea how all the
> > > > things around this driver works ...for example which c. file I should focus
> > > > on. If you would give me a small piece of advise, where to start, I would be
> > > > pleased.
> > > >
> > > > OK, I'm not sure if I understood what you meant by adding sign-off-by and
> > > > reviewed-by line ... I hope I did what you want me to:
> > > >
> > > >
> > > > So .. thanks for your time,
> > > > Zbynek
> > > >
> > > > Signed-off-by: Zbynek Hrabovsky <hrabosh@xxxxxxxxxx>
> > > >
> > > > Reviewed-by: Zbynek Hrbaovsky <hrabosh@xxxxxxxxxx>
> > > >
> > > > diff -r 04ddbe145932 linux/drivers/media/video/cx88/cx88-cards.c
> > > > --- a/linux/drivers/media/video/cx88/cx88-cards.c Tue Jun 10 15:27:29
> > > > 2008 -0300
> > > > +++ b/linux/drivers/media/video/cx88/cx88-cards.c Fri Jun 13 15:07:33
> > > > 2008 -0300
> > > > @@ -1284,7 +1284,7 @@
> > > > },
> > > > [CX88_BOARD_WINFAST_DTV2000H] = {
> > > > /* video inputs and radio still in testing */
> > > > - .name = "WinFast DTV2000 H",
> > > > + .name = "WinFast DTV2000 H ver. I (old)",
> > > > .tuner_type = TUNER_PHILIPS_FMD1216ME_MK3,
> > > > .radio_type = UNSET,
> > > > .tuner_addr = ADDR_UNSET,
> > > > @@ -1298,6 +1298,45 @@
> > > > .gpio2 = 0x00017304,
> > > > .gpio3 = 0x02000000,
> > > > }},
> > > > + .mpeg = CX88_MPEG_DVB,
> > > > + },
> > > > + [CX88_BOARD_WINFAST_DTV2000H_2] = {
> > > > + /* this is just a try */
> > > > + .name = "WinFast DTV2000 H ver. J (new)",
> > > > + .tuner_type = TUNER_PHILIPS_FMD1216ME_MK3,
> > > > + .radio_type = UNSET,
> > > > + .tuner_addr = ADDR_UNSET,
> > > > + .radio_addr = ADDR_UNSET,
> > > > + .tda9887_conf = TDA9887_PRESENT,
> > > > + .input = { {
> > > > + .type = CX88_VMUX_TELEVISION,
> > > > + .vmux = 0,
> > > > + .gpio0 = 0x00017300,
> > > > + .gpio1 = 0x00008207,
> > > > + .gpio2 = 0x00000000,
> > > > + .gpio3 = 0x02000000,
> > > > + }, {
> > > > + .type = CX88_VMUX_TELEVISION,
> > > > + .vmux = 0,
> > > > + .gpio0 = 0x00018300,
> > > > + .gpio1 = 0x0000f207,
> > > > + .gpio2 = 0x00017304,
> > > > + .gpio3 = 0x02000000,
> > > > + }, {
> > > > + .type = CX88_VMUX_COMPOSITE1,
> > > > + .vmux = 1,
> > > > + .gpio0 = 0x00018301,
> > > > + .gpio1 = 0x0000f207,
> > > > + .gpio2 = 0x00017304,
> > > > + .gpio3 = 0x02000000,
> > > > + }, {
> > > > + .type = CX88_VMUX_SVIDEO,
> > > > + .vmux = 2,
> > > > + .gpio0 = 0x00018301,
> > > > + .gpio1 = 0x0000f207,
> > > > + .gpio2 = 0x00017304,
> > > > + .gpio3 = 0x02000000,
> > > > + } },
> > > > .mpeg = CX88_MPEG_DVB,
> > > > },
> > > > [CX88_BOARD_GENIATECH_DVBS] = {
> > > > @@ -1963,6 +2002,10 @@
> > > > .subdevice = 0x665e,
> > > > .card = CX88_BOARD_WINFAST_DTV2000H,
> > > > },{
> > > > + .subvendor = 0x107d,
> > > > + .subdevice = 0x6f2b,
> > > > + .card = CX88_BOARD_WINFAST_DTV2000H_2,
> > > > + }, {
> > > > .subvendor = 0x18ac,
> > > > .subdevice = 0xd800, /* FusionHDTV 3 Gold (original
> > > > revision) */
> > > > .card = CX88_BOARD_DVICO_FUSIONHDTV_3_GOLD_Q,
> > > > diff -r 04ddbe145932 linux/drivers/media/video/cx88/cx88-dvb.c
> > > > --- a/linux/drivers/media/video/cx88/cx88-dvb.c Tue Jun 10 15:27:29 2008
> > > > -0300
> > > > +++ b/linux/drivers/media/video/cx88/cx88-dvb.c Fri Jun 13 15:07:34 2008
> > > > -0300
> > > > @@ -561,6 +561,7 @@
> > > > }
> > > > break;
> > > > case CX88_BOARD_WINFAST_DTV2000H:
> > > > + case CX88_BOARD_WINFAST_DTV2000H_2:
> > > > case CX88_BOARD_HAUPPAUGE_HVR1100:
> > > > case CX88_BOARD_HAUPPAUGE_HVR1100LP:
> > > > case CX88_BOARD_HAUPPAUGE_HVR1300:
> > > > diff -r 04ddbe145932 linux/drivers/media/video/cx88/cx88-input.c
> > > > --- a/linux/drivers/media/video/cx88/cx88-input.c Tue Jun 10 15:27:29
> > > > 2008 -0300
> > > > +++ b/linux/drivers/media/video/cx88/cx88-input.c Fri Jun 13 15:07:34
> > > > 2008 -0300
> > > > @@ -242,6 +242,7 @@
> > > > ir->sampling = 1;
> > > > break;
> > > > case CX88_BOARD_WINFAST_DTV2000H:
> > > > + case CX88_BOARD_WINFAST_DTV2000H_2:
> > > > ir_codes = ir_codes_winfast;
> > > > ir->gpio_addr = MO_GP0_IO;
> > > > ir->mask_keycode = 0x8f8;
> > > >
> > > > diff -r 04ddbe145932 linux/drivers/media/video/cx88/cx88.h
> > > > --- a/linux/drivers/media/video/cx88/cx88.h Tue Jun 10 15:27:29 2008
> > > > -0300
> > > > +++ b/linux/drivers/media/video/cx88/cx88.h Fri Jun 13 15:07:34 2008
> > > > -0300
> > > > @@ -224,6 +224,7 @@
> > > > #define CX88_BOARD_DVICO_FUSIONHDTV_7_GOLD 65
> > > > #define CX88_BOARD_PROLINK_PV_8000GT 66
> > > > #define CX88_BOARD_KWORLD_ATSC_120 67
> > > > +#define CX88_BOARD_WINFAST_DTV2000H_2 68
> > > >
> > > > enum cx88_itype {
> > > > CX88_VMUX_COMPOSITE1 = 1,
> > > >
> > > >
> > > >
> > > >
> > >
>
>
>
>
> Cheers,
> Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/