* [PATCH] Terratec Cinergy Hybrid T USB XS FR
@ 2010-10-04 19:32 Catimimi
2010-10-16 2:38 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 3+ messages in thread
From: Catimimi @ 2010-10-04 19:32 UTC (permalink / raw)
To: linux-media
New gpio definitions.
XC3028_FE_ZARLINK456 was not loaded.
Signed-off-by: Michel Garnier<catimimi@libertysurf.fr>
---
diff -Nru v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c
v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c
--- v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-09-19 07:23:09.000000000 +0200
+++ v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-10-04 19:05:11.000000000 +0200
@@ -200,6 +200,18 @@
{ -1, -1, -1, -1},
};
+static struct em28xx_reg_seq terratec_cinergy_USB_XS_analog[] = {
+ {EM28XX_R08_GPIO, 0x6d, ~EM_GPIO_4, 10},
+ {EM2880_R04_GPO, 0x00, 0xff, 10},
+ { -1, -1, -1, -1},
+};
+
+static struct em28xx_reg_seq terratec_cinergy_USB_XS_digital[] = {
+ {EM28XX_R08_GPIO, 0x6e, ~EM_GPIO_4, 10},
+ {EM2880_R04_GPO, 0x08, 0xff, 10},
+ { -1, -1, -1, -1},
+};
+
/* eb1a:2868 Reddo DVB-C USB TV Box
GPIO4 - CU1216L NIM
Other GPIOs seems to be don't care. */
@@ -824,22 +836,22 @@
.tuner_gpio = default_tuner_gpio,
.decoder = EM28XX_TVP5150,
.has_dvb = 1,
- .dvb_gpio = default_digital,
+ .dvb_gpio = terratec_cinergy_USB_XS_digital,
.input = { {
.type = EM28XX_VMUX_TELEVISION,
.vmux = TVP5150_COMPOSITE0,
.amux = EM28XX_AMUX_VIDEO,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_analog,
}, {
.type = EM28XX_VMUX_COMPOSITE1,
.vmux = TVP5150_COMPOSITE1,
.amux = EM28XX_AMUX_LINE_IN,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_analog,
}, {
.type = EM28XX_VMUX_SVIDEO,
.vmux = TVP5150_SVIDEO,
.amux = EM28XX_AMUX_LINE_IN,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_analog,
} },
},
[EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900] = {
@@ -2259,6 +2271,7 @@
ctl->demod = XC3028_FE_ZARLINK456;
break;
case EM2880_BOARD_TERRATEC_HYBRID_XS:
+ case EM2880_BOARD_TERRATEC_HYBRID_XS_FR:
case EM2881_BOARD_PINNACLE_HYBRID_PRO:
ctl->demod = XC3028_FE_ZARLINK456;
break;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Terratec Cinergy Hybrid T USB XS FR
2010-10-04 19:32 [PATCH] Terratec Cinergy Hybrid T USB XS FR Catimimi
@ 2010-10-16 2:38 ` Mauro Carvalho Chehab
2010-10-16 17:42 ` Catimimi
0 siblings, 1 reply; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-16 2:38 UTC (permalink / raw)
To: Catimimi; +Cc: linux-media
Em 04-10-2010 16:32, Catimimi escreveu:
> New gpio definitions.
> XC3028_FE_ZARLINK456 was not loaded.
>
> Signed-off-by: Michel Garnier<catimimi@libertysurf.fr>
>
> ---
>
> diff -Nru v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c
> --- v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-09-19 07:23:09.000000000 +0200
> +++ v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-10-04 19:05:11.000000000 +0200
> @@ -200,6 +200,18 @@
> { -1, -1, -1, -1},
> };
>
> +static struct em28xx_reg_seq terratec_cinergy_USB_XS_analog[] = {
> + {EM28XX_R08_GPIO, 0x6d, ~EM_GPIO_4, 10},
> + {EM2880_R04_GPO, 0x00, 0xff, 10},
> + { -1, -1, -1, -1},
> +};
> +
> +static struct em28xx_reg_seq terratec_cinergy_USB_XS_digital[] = {
> + {EM28XX_R08_GPIO, 0x6e, ~EM_GPIO_4, 10},
> + {EM2880_R04_GPO, 0x08, 0xff, 10},
> + { -1, -1, -1, -1},
> +};
> +
> /* eb1a:2868 Reddo DVB-C USB TV Box
> GPIO4 - CU1216L NIM
> Other GPIOs seems to be don't care. */
> @@ -824,22 +836,22 @@
> .tuner_gpio = default_tuner_gpio,
> .decoder = EM28XX_TVP5150,
> .has_dvb = 1,
> - .dvb_gpio = default_digital,
> + .dvb_gpio = terratec_cinergy_USB_XS_digital,
> .input = { {
> .type = EM28XX_VMUX_TELEVISION,
> .vmux = TVP5150_COMPOSITE0,
> .amux = EM28XX_AMUX_VIDEO,
> - .gpio = default_analog,
> + .gpio = terratec_cinergy_USB_XS_analog,
> }, {
> .type = EM28XX_VMUX_COMPOSITE1,
> .vmux = TVP5150_COMPOSITE1,
> .amux = EM28XX_AMUX_LINE_IN,
> - .gpio = default_analog,
> + .gpio = terratec_cinergy_USB_XS_analog,
> }, {
> .type = EM28XX_VMUX_SVIDEO,
> .vmux = TVP5150_SVIDEO,
> .amux = EM28XX_AMUX_LINE_IN,
> - .gpio = default_analog,
> + .gpio = terratec_cinergy_USB_XS_analog,
> } },
> },
> [EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900] = {
> @@ -2259,6 +2271,7 @@
> ctl->demod = XC3028_FE_ZARLINK456;
> break;
> case EM2880_BOARD_TERRATEC_HYBRID_XS:
> + case EM2880_BOARD_TERRATEC_HYBRID_XS_FR:
Hmm... do you have a different device, right? Please, don't change the entries
of the original Hybrid XS, or it will cause a regression for the others. Instead,
create another entry describing your board.
Also, please use tabs for indent. A tab in Linux have 8 spaces, and not four.
> case EM2881_BOARD_PINNACLE_HYBRID_PRO:
> ctl->demod = XC3028_FE_ZARLINK456;
> break;
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Terratec Cinergy Hybrid T USB XS FR
2010-10-16 2:38 ` Mauro Carvalho Chehab
@ 2010-10-16 17:42 ` Catimimi
0 siblings, 0 replies; 3+ messages in thread
From: Catimimi @ 2010-10-16 17:42 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Catimimi, linux-media
[-- Attachment #1: Type: text/plain, Size: 4013 bytes --]
Le 16/10/2010 04:38, Mauro Carvalho Chehab a écrit :
> Em 04-10-2010 16:32, Catimimi escreveu:
>
>> New gpio definitions.
>> XC3028_FE_ZARLINK456 was not loaded.
>>
>> Signed-off-by: Michel Garnier<catimimi@libertysurf.fr>
>>
>> ---
>>
>> diff -Nru v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c
>> --- v4l-dvb-1da5fed5c8b2-orig/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-09-19 07:23:09.000000000 +0200
>> +++ v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-10-04 19:05:11.000000000 +0200
>> @@ -200,6 +200,18 @@
>> { -1, -1, -1, -1},
>> };
>>
>> +static struct em28xx_reg_seq terratec_cinergy_USB_XS_analog[] = {
>> + {EM28XX_R08_GPIO, 0x6d, ~EM_GPIO_4, 10},
>> + {EM2880_R04_GPO, 0x00, 0xff, 10},
>> + { -1, -1, -1, -1},
>> +};
>> +
>> +static struct em28xx_reg_seq terratec_cinergy_USB_XS_digital[] = {
>> + {EM28XX_R08_GPIO, 0x6e, ~EM_GPIO_4, 10},
>> + {EM2880_R04_GPO, 0x08, 0xff, 10},
>> + { -1, -1, -1, -1},
>> +};
>> +
>> /* eb1a:2868 Reddo DVB-C USB TV Box
>> GPIO4 - CU1216L NIM
>> Other GPIOs seems to be don't care. */
>> @@ -824,22 +836,22 @@
>> .tuner_gpio = default_tuner_gpio,
>> .decoder = EM28XX_TVP5150,
>> .has_dvb = 1,
>> - .dvb_gpio = default_digital,
>> + .dvb_gpio = terratec_cinergy_USB_XS_digital,
>> .input = { {
>> .type = EM28XX_VMUX_TELEVISION,
>> .vmux = TVP5150_COMPOSITE0,
>> .amux = EM28XX_AMUX_VIDEO,
>> - .gpio = default_analog,
>> + .gpio = terratec_cinergy_USB_XS_analog,
>> }, {
>> .type = EM28XX_VMUX_COMPOSITE1,
>> .vmux = TVP5150_COMPOSITE1,
>> .amux = EM28XX_AMUX_LINE_IN,
>> - .gpio = default_analog,
>> + .gpio = terratec_cinergy_USB_XS_analog,
>> }, {
>> .type = EM28XX_VMUX_SVIDEO,
>> .vmux = TVP5150_SVIDEO,
>> .amux = EM28XX_AMUX_LINE_IN,
>> - .gpio = default_analog,
>> + .gpio = terratec_cinergy_USB_XS_analog,
>> } },
>> },
>> [EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900] = {
>> @@ -2259,6 +2271,7 @@
>> ctl->demod = XC3028_FE_ZARLINK456;
>> break;
>> case EM2880_BOARD_TERRATEC_HYBRID_XS:
>> + case EM2880_BOARD_TERRATEC_HYBRID_XS_FR:
>>
> Hmm... do you have a different device, right? Please, don't change the entries
> of the original Hybrid XS, or it will cause a regression for the others. Instead,
> create another entry describing your board.
>
> Also, please use tabs for indent. A tab in Linux have 8 spaces, and not four.
>
>
>> case EM2881_BOARD_PINNACLE_HYBRID_PRO:
>> ctl->demod = XC3028_FE_ZARLINK456;
>> break;
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> ,
>
Hello
I didn't change the entries of original Hybrid XS. I created new entries
which I use for XS_FR.
These new entries were necessaries for a good working with 64 bits kernels.
So there is no regression.
In order to be clear I renamed the new entries to XS_FR.
OK for the tabs, the mail agents converted them to spaces, so I include
a file.
My last proposal is :
New gpio definitions
XC3028_FE_ZARLINK456 was not loaded.
Signed-off-by: Michel Garnier<catimimi@libertysurfhel
Regards
Michel.
[-- Attachment #2: Terratec_cynergy_hybrid_usb_xs_fr2.patch --]
[-- Type: text/x-patch, Size: 2022 bytes --]
diff -ru v4l-dvb-1da5fed5c8b2-old/linux/drivers/media/video/em28xx/em28xx-cards.c v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c
--- v4l-dvb-1da5fed5c8b2-old/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-03-04 06:49:46.000000000 +0100
+++ v4l-dvb-1da5fed5c8b2-new/linux/drivers/media/video/em28xx/em28xx-cards.c 2010-03-05 21:16:36.000000000 +0100
@@ -200,6 +200,18 @@
{ -1, -1, -1, -1},
};
+static struct em28xx_reg_seq terratec_cinergy_USB_XS_FR_analog[] = {
+ {EM28XX_R08_GPIO, 0x6d, ~EM_GPIO_4, 10},
+ {EM2880_R04_GPO, 0x00, 0xff, 10},
+ { -1, -1, -1, -1},
+};
+
+static struct em28xx_reg_seq terratec_cinergy_USB_XS_FR_digital[] = {
+ {EM28XX_R08_GPIO, 0x6e, ~EM_GPIO_4, 10},
+ {EM2880_R04_GPO, 0x08, 0xff, 10},
+ { -1, -1, -1, -1},
+};
+
/* eb1a:2868 Reddo DVB-C USB TV Box
GPIO4 - CU1216L NIM
Other GPIOs seems to be don't care. */
@@ -824,22 +836,22 @@
.tuner_gpio = default_tuner_gpio,
.decoder = EM28XX_TVP5150,
.has_dvb = 1,
- .dvb_gpio = default_digital,
+ .dvb_gpio = terratec_cinergy_USB_XS_FR_digital,
.input = { {
.type = EM28XX_VMUX_TELEVISION,
.vmux = TVP5150_COMPOSITE0,
.amux = EM28XX_AMUX_VIDEO,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_FR_analog,
}, {
.type = EM28XX_VMUX_COMPOSITE1,
.vmux = TVP5150_COMPOSITE1,
.amux = EM28XX_AMUX_LINE_IN,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_FR_analog,
}, {
.type = EM28XX_VMUX_SVIDEO,
.vmux = TVP5150_SVIDEO,
.amux = EM28XX_AMUX_LINE_IN,
- .gpio = default_analog,
+ .gpio = terratec_cinergy_USB_XS_FR_analog,
} },
},
[EM2880_BOARD_HAUPPAUGE_WINTV_HVR_900] = {
@@ -2259,6 +2271,7 @@
ctl->demod = XC3028_FE_ZARLINK456;
break;
case EM2880_BOARD_TERRATEC_HYBRID_XS:
+ case EM2880_BOARD_TERRATEC_HYBRID_XS_FR:
case EM2881_BOARD_PINNACLE_HYBRID_PRO:
ctl->demod = XC3028_FE_ZARLINK456;
break;
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-10-16 17:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-04 19:32 [PATCH] Terratec Cinergy Hybrid T USB XS FR Catimimi
2010-10-16 2:38 ` Mauro Carvalho Chehab
2010-10-16 17:42 ` Catimimi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox