public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* KWorld 330U Employs Samsung S5H1409X01 Demodulator
@ 2009-01-04  4:21 Robert Krakora
  2009-01-04  5:18 ` Devin Heitmueller
  0 siblings, 1 reply; 12+ messages in thread
From: Robert Krakora @ 2009-01-04  4:21 UTC (permalink / raw)
  To: Mauro Carvalho Chehab, video4linux-list; +Cc: Jerry Geis

Mauro:

The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
LGDT330X.  Hence the error initializing the LGDT330X in the current source
in em28xx-dvb.c.

Best Regards,

-- 
Rob Krakora
Software Engineer
MessageNet Systems
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
(317)663-0808 Fax
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  4:21 KWorld 330U Employs Samsung S5H1409X01 Demodulator Robert Krakora
@ 2009-01-04  5:18 ` Devin Heitmueller
  2009-01-04  5:55   ` Robert Krakora
  2009-01-04  6:45   ` CityK
  0 siblings, 2 replies; 12+ messages in thread
From: Devin Heitmueller @ 2009-01-04  5:18 UTC (permalink / raw)
  To: Robert Krakora; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
<rob.krakora@messagenetsystems.com> wrote:
> Mauro:
>
> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
> LGDT330X.  Hence the error initializing the LGDT330X in the current source
> in em28xx-dvb.c.
>
> Best Regards,

Hello Robert,

Well, that's good to know.  I don't think anyone has done any work on
that device recently, so I don't know why the code has it as an
lgdt3303.

Do you know which tuner chip the device has?  The reason I ask is
because I'm working on another device that also has the s5h1409, and
it's got an xc3028L (the low power version of the xc3028).  If the
330U also has the xc3028L, then we need to make sure to indicate that
in the device profile so it doesn't burn out the chip.

We're probably also going to need to get a Windows trace, so we know
how to setup the s5h1409 configuration.

Devin

-- 
Devin J. Heitmueller
http://www.devinheitmueller.com
AIM: devinheitmueller

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  5:18 ` Devin Heitmueller
@ 2009-01-04  5:55   ` Robert Krakora
  2009-01-04  6:31     ` Robert Krakora
  2009-01-04  6:45   ` CityK
  1 sibling, 1 reply; 12+ messages in thread
From: Robert Krakora @ 2009-01-04  5:55 UTC (permalink / raw)
  To: Devin Heitmueller; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

On Sun, Jan 4, 2009 at 12:18 AM, Devin Heitmueller <
devin.heitmueller@gmail.com> wrote:

> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
> <rob.krakora@messagenetsystems.com> wrote:
> > Mauro:
> >
> > The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
> > LGDT330X.  Hence the error initializing the LGDT330X in the current
> source
> > in em28xx-dvb.c.
> >
> > Best Regards,
>
> Hello Robert,
>
> Well, that's good to know.  I don't think anyone has done any work on
> that device recently, so I don't know why the code has it as an
> lgdt3303.
>
> Do you know which tuner chip the device has?  The reason I ask is
> because I'm working on another device that also has the s5h1409, and
> it's got an xc3028L (the low power version of the xc3028).  If the
> 330U also has the xc3028L, then we need to make sure to indicate that
> in the device profile so it doesn't burn out the chip.
>
> We're probably also going to need to get a Windows trace, so we know
> how to setup the s5h1409 configuration.
>
> Devin
>
> --
> Devin J. Heitmueller
> http://www.devinheitmueller.com
> AIM: devinheitmueller
>
>
Devin:

I believe that it has the 3028 and not the 3028L as the part gets a little
toasty even on Windows,  I will get the magnifying glass out and look again
though (I am over 40 and becoming near-sighted).  I can get the Windows
trace using USBTrace on Windows XP or with my Elisys USB Analyzer.

Best Regards,

-- 
Rob Krakora
Software Engineer
MessageNet Systems
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
(317)663-0808 Fax
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  5:55   ` Robert Krakora
@ 2009-01-04  6:31     ` Robert Krakora
  2009-01-04 13:47       ` Devin Heitmueller
  0 siblings, 1 reply; 12+ messages in thread
From: Robert Krakora @ 2009-01-04  6:31 UTC (permalink / raw)
  To: Devin Heitmueller; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

On Sun, Jan 4, 2009 at 12:55 AM, Robert Krakora <
rob.krakora@messagenetsystems.com> wrote:

>
>
> On Sun, Jan 4, 2009 at 12:18 AM, Devin Heitmueller <
> devin.heitmueller@gmail.com> wrote:
>
>> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
>> <rob.krakora@messagenetsystems.com> wrote:
>> > Mauro:
>> >
>> > The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
>> > LGDT330X.  Hence the error initializing the LGDT330X in the current
>> source
>> > in em28xx-dvb.c.
>> >
>> > Best Regards,
>>
>> Hello Robert,
>>
>> Well, that's good to know.  I don't think anyone has done any work on
>> that device recently, so I don't know why the code has it as an
>> lgdt3303.
>>
>> Do you know which tuner chip the device has?  The reason I ask is
>> because I'm working on another device that also has the s5h1409, and
>> it's got an xc3028L (the low power version of the xc3028).  If the
>> 330U also has the xc3028L, then we need to make sure to indicate that
>> in the device profile so it doesn't burn out the chip.
>>
>> We're probably also going to need to get a Windows trace, so we know
>> how to setup the s5h1409 configuration.
>>
>> Devin
>>
>> --
>> Devin J. Heitmueller
>> http://www.devinheitmueller.com
>> AIM: devinheitmueller
>>
>>
> Devin:
>
> I believe that it has the 3028 and not the 3028L as the part gets a little
> toasty even on Windows,  I will get the magnifying glass out and look again
> though (I am over 40 and becoming near-sighted).  I can get the Windows
> trace using USBTrace on Windows XP or with my Elisys USB Analyzer.
>
>
> Best Regards,
>
> --
> Rob Krakora
> Software Engineer
> MessageNet Systems
> 101 East Carmel Dr. Suite 105
> Carmel, IN 46032
> (317)566-1677 Ext. 206
> (317)663-0808 Fax
>

Devin:

I took a stab at attaching the Samsung demod to the em28xx and got the
following error.  My guess is that the demod address is wrong.  How does one
go about dissecting vendor requests in a USB trace to obtain setup
information for this part?  I was going to put an I2C analyzer on the I2C
line while running Windows to capture I2C traffic.  I will get a Windows USB
trace tomorrow and e-mail it out.  Sure would be easier with the specs or a
little help from KWorld.  ;-)

s5h1409_readreg: readreg error (ret == -19)
em28xx #0/2: dvb frontend not attached. Can't attach xc3028
Em28xx: Initialized (Em28xx dvb Extension) extension


[root@am2mm v4l-dvb]# hg diff
diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-audio.c
--- a/linux/drivers/media/video/em28xx/em28xx-audio.c   Fri Jan 02 18:34:28
2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-audio.c   Sun Jan 04 01:24:39
2009 -0500
@@ -63,9 +63,11 @@

        dprintk("Stopping isoc\n");
        for (i = 0; i < EM28XX_AUDIO_BUFS; i++) {
-               usb_unlink_urb(dev->adev.urb[i]);
+               usb_kill_urb(dev->adev.urb[i]);
                usb_free_urb(dev->adev.urb[i]);
                dev->adev.urb[i] = NULL;
+               kfree(dev->adev.transfer_buffer[i]);
+               dev->adev.transfer_buffer[i] = NULL;
        }

        return 0;
diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-cards.c
--- a/linux/drivers/media/video/em28xx/em28xx-cards.c   Fri Jan 02 18:34:28
2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-cards.c   Sun Jan 04 01:24:39
2009 -0500
@@ -1242,31 +1242,31 @@
                        .gpio     = hauppauge_wintv_hvr_900_analog,
                } },
        },
-       [EM2883_BOARD_KWORLD_HYBRID_A316] = {
-               .name         = "Kworld PlusTV HD Hybrid 330",
-               .tuner_type   = TUNER_XC2028,
-               .tuner_gpio   = default_tuner_gpio,
-               .decoder      = EM28XX_TVP5150,
-               .mts_firmware = 1,
-               .has_dvb      = 1,
-               .dvb_gpio     = default_digital,
-               .input        = { {
-                       .type     = EM28XX_VMUX_TELEVISION,
-                       .vmux     = TVP5150_COMPOSITE0,
-                       .amux     = EM28XX_AMUX_VIDEO,
-                       .gpio     = default_analog,
-               }, {
-                       .type     = EM28XX_VMUX_COMPOSITE1,
-                       .vmux     = TVP5150_COMPOSITE1,
-                       .amux     = EM28XX_AMUX_LINE_IN,
-                       .gpio     = hauppauge_wintv_hvr_900_analog,
-               }, {
-                       .type     = EM28XX_VMUX_SVIDEO,
-                       .vmux     = TVP5150_SVIDEO,
-                       .amux     = EM28XX_AMUX_LINE_IN,
-                       .gpio     = hauppauge_wintv_hvr_900_analog,
-               } },
-       },
+        [EM2883_BOARD_KWORLD_HYBRID_A316] = {
+                .name         = "Kworld PlusTV HD Hybrid 330",
+                .tuner_type   = TUNER_XC2028,
+                .tuner_gpio   = default_tuner_gpio,
+                .has_dvb      = 1,
+                .dvb_gpio     = default_digital,
+                .mts_firmware = 1,
+                .decoder      = EM28XX_TVP5150,
+                .input        = { {
+                        .type     = EM28XX_VMUX_TELEVISION,
+                        .vmux     = TVP5150_COMPOSITE0,
+                        .amux     = EM28XX_AMUX_VIDEO,
+                        .gpio     = default_analog,
+                }, {
+                        .type     = EM28XX_VMUX_COMPOSITE1,
+                        .vmux     = TVP5150_COMPOSITE1,
+                        .amux     = EM28XX_AMUX_LINE_IN,
+                        .gpio     = default_analog,
+                }, {
+                        .type     = EM28XX_VMUX_SVIDEO,
+                        .vmux     = TVP5150_SVIDEO,
+                        .amux     = EM28XX_AMUX_LINE_IN,
+                        .gpio     = default_analog,
+                } },
+        },
        [EM2820_BOARD_COMPRO_VIDEOMATE_FORYOU] = {
                .name         = "Compro VideoMate ForYou/Stereo",
                .tuner_type   = TUNER_LG_PAL_NEW_TAPC,
diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-dvb.c
--- a/linux/drivers/media/video/em28xx/em28xx-dvb.c     Fri Jan 02 18:34:28
2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-dvb.c     Sun Jan 04 01:24:39
2009 -0500
@@ -29,6 +29,7 @@

 #include "lgdt330x.h"
 #include "zl10353.h"
+#include "s5h1409.h"
 #ifdef EM28XX_DRX397XD_SUPPORT
 #include "drx397xD.h"
 #endif
@@ -231,6 +232,17 @@
        .no_tuner = 1,
        .parallel_ts = 1,
        .if2 = 45600,
+};
+
+static struct s5h1409_config em28xx_s5h1409_with_xc3028 = {
+       .demod_address = 0x32 >> 1,
+       .output_mode   = S5H1409_SERIAL_OUTPUT,
+       .gpio          = S5H1409_GPIO_ON,
+       .qam_if        = 44000,
+       .inversion     = S5H1409_INVERSION_OFF,
+       .status_mode   = S5H1409_DEMODLOCKING,
+       .mpeg_timing   = S5H1409_MPEGTIMING_CONTINOUS_NONINVERTING_CLOCK
+
 };

 #ifdef EM28XX_DRX397XD_SUPPORT
@@ -413,7 +425,6 @@
        case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_850:
        case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_950:
        case EM2880_BOARD_PINNACLE_PCTV_HD_PRO:
-       case EM2883_BOARD_KWORLD_HYBRID_A316:
        case EM2880_BOARD_AMD_ATI_TV_WONDER_HD_600:
                dvb->frontend = dvb_attach(lgdt330x_attach,
                                           &em2880_lgdt3303_dev,
@@ -447,6 +458,15 @@
                }
                break;
 #endif
+       case EM2883_BOARD_KWORLD_HYBRID_A316:
+               dvb->frontend = dvb_attach(s5h1409_attach,
+                                          &em28xx_s5h1409_with_xc3028,
+                                          &dev->i2c_adap);
+               if (attach_xc3028(0x61, dev) < 0) {
+                       result = -EINVAL;
+                       goto out_free;
+               }
+               break;
        default:
                printk(KERN_ERR "%s/2: The frontend of your DVB/ATSC card"
                                " isn't supported yet\n",
@@ -514,3 +534,4 @@

 module_init(em28xx_dvb_register);
 module_exit(em28xx_dvb_unregister);
+
[root@am2mm v4l-dvb]#

Best Regards,

-- 
Rob Krakora
Software Engineer
MessageNet Systems
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
(317)663-0808 Fax
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  5:18 ` Devin Heitmueller
  2009-01-04  5:55   ` Robert Krakora
@ 2009-01-04  6:45   ` CityK
       [not found]     ` <20090104135840.7de113de@gmail.com>
  2009-01-04 19:28     ` Robert Krakora
  1 sibling, 2 replies; 12+ messages in thread
From: CityK @ 2009-01-04  6:45 UTC (permalink / raw)
  To: Devin Heitmueller; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

Devin Heitmueller wrote:
> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
> <rob.krakora@messagenetsystems.com> wrote:
>   
>> Mauro:
>>
>> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
>> LGDT330X.  Hence the error initializing the LGDT330X in the current source
>> in em28xx-dvb.c.
>>
>> Best Regards,
>>     
>
> Hello Robert,
>
> Well, that's good to know.  I don't think anyone has done any work on
> that device recently, so I don't know why the code has it as an
> lgdt3303.

I believe Douglas submitted this patch
(http://linuxtv.org/hg/v4l-dvb/rev/77f789d59de8) that got committed. 

I've been meaning to get back to this because the "A316" part of the
name caught my attention -- I do not recall having seen such a reference
made by KWorld, nor is it typical of their nomenclature style, rather,
it is entirely consistent with that used by AVerMedia



--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  6:31     ` Robert Krakora
@ 2009-01-04 13:47       ` Devin Heitmueller
  0 siblings, 0 replies; 12+ messages in thread
From: Devin Heitmueller @ 2009-01-04 13:47 UTC (permalink / raw)
  To: Robert Krakora; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

On Sun, Jan 4, 2009 at 1:31 AM, Robert Krakora
<rob.krakora@messagenetsystems.com> wrote:
>
>
> On Sun, Jan 4, 2009 at 12:55 AM, Robert Krakora
> <rob.krakora@messagenetsystems.com> wrote:
>>
>>
>> On Sun, Jan 4, 2009 at 12:18 AM, Devin Heitmueller
>> <devin.heitmueller@gmail.com> wrote:
>>>
>>> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
>>> <rob.krakora@messagenetsystems.com> wrote:
>>> > Mauro:
>>> >
>>> > The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
>>> > LGDT330X.  Hence the error initializing the LGDT330X in the current
>>> > source
>>> > in em28xx-dvb.c.
>>> >
>>> > Best Regards,
>>>
>>> Hello Robert,
>>>
>>> Well, that's good to know.  I don't think anyone has done any work on
>>> that device recently, so I don't know why the code has it as an
>>> lgdt3303.
>>>
>>> Do you know which tuner chip the device has?  The reason I ask is
>>> because I'm working on another device that also has the s5h1409, and
>>> it's got an xc3028L (the low power version of the xc3028).  If the
>>> 330U also has the xc3028L, then we need to make sure to indicate that
>>> in the device profile so it doesn't burn out the chip.
>>>
>>> We're probably also going to need to get a Windows trace, so we know
>>> how to setup the s5h1409 configuration.
>>>
>>> Devin
>>>
>>> --
>>> Devin J. Heitmueller
>>> http://www.devinheitmueller.com
>>> AIM: devinheitmueller
>>>
>>
>> Devin:
>>
>> I believe that it has the 3028 and not the 3028L as the part gets a little
>> toasty even on Windows,  I will get the magnifying glass out and look again
>> though (I am over 40 and becoming near-sighted).  I can get the Windows
>> trace using USBTrace on Windows XP or with my Elisys USB Analyzer.
>>
>> Best Regards,
>>
>> --
>> Rob Krakora
>> Software Engineer
>> MessageNet Systems
>> 101 East Carmel Dr. Suite 105
>> Carmel, IN 46032
>> (317)566-1677 Ext. 206
>> (317)663-0808 Fax
>
> Devin:
>
> I took a stab at attaching the Samsung demod to the em28xx and got the
> following error.  My guess is that the demod address is wrong.  How does one
> go about dissecting vendor requests in a USB trace to obtain setup
> information for this part?  I was going to put an I2C analyzer on the I2C
> line while running Windows to capture I2C traffic.  I will get a Windows USB
> trace tomorrow and e-mail it out.  Sure would be easier with the specs or a
> little help from KWorld.  ;-)
>
> s5h1409_readreg: readreg error (ret == -19)
> em28xx #0/2: dvb frontend not attached. Can't attach xc3028
> Em28xx: Initialized (Em28xx dvb Extension) extension
>
>
> [root@am2mm v4l-dvb]# hg diff
> diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-audio.c
> --- a/linux/drivers/media/video/em28xx/em28xx-audio.c   Fri Jan 02 18:34:28
> 2009 -0200
> +++ b/linux/drivers/media/video/em28xx/em28xx-audio.c   Sun Jan 04 01:24:39
> 2009 -0500
> @@ -63,9 +63,11 @@
>
>         dprintk("Stopping isoc\n");
>         for (i = 0; i < EM28XX_AUDIO_BUFS; i++) {
> -               usb_unlink_urb(dev->adev.urb[i]);
> +               usb_kill_urb(dev->adev.urb[i]);
>                 usb_free_urb(dev->adev.urb[i]);
>                 dev->adev.urb[i] = NULL;
> +               kfree(dev->adev.transfer_buffer[i]);
> +               dev->adev.transfer_buffer[i] = NULL;
>         }
>
>         return 0;
> diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-cards.c
> --- a/linux/drivers/media/video/em28xx/em28xx-cards.c   Fri Jan 02 18:34:28
> 2009 -0200
> +++ b/linux/drivers/media/video/em28xx/em28xx-cards.c   Sun Jan 04 01:24:39
> 2009 -0500
> @@ -1242,31 +1242,31 @@
>                         .gpio     = hauppauge_wintv_hvr_900_analog,
>                 } },
>         },
> -       [EM2883_BOARD_KWORLD_HYBRID_A316] = {
> -               .name         = "Kworld PlusTV HD Hybrid 330",
> -               .tuner_type   = TUNER_XC2028,
> -               .tuner_gpio   = default_tuner_gpio,
> -               .decoder      = EM28XX_TVP5150,
> -               .mts_firmware = 1,
> -               .has_dvb      = 1,
> -               .dvb_gpio     = default_digital,
> -               .input        = { {
> -                       .type     = EM28XX_VMUX_TELEVISION,
> -                       .vmux     = TVP5150_COMPOSITE0,
> -                       .amux     = EM28XX_AMUX_VIDEO,
> -                       .gpio     = default_analog,
> -               }, {
> -                       .type     = EM28XX_VMUX_COMPOSITE1,
> -                       .vmux     = TVP5150_COMPOSITE1,
> -                       .amux     = EM28XX_AMUX_LINE_IN,
> -                       .gpio     = hauppauge_wintv_hvr_900_analog,
> -               }, {
> -                       .type     = EM28XX_VMUX_SVIDEO,
> -                       .vmux     = TVP5150_SVIDEO,
> -                       .amux     = EM28XX_AMUX_LINE_IN,
> -                       .gpio     = hauppauge_wintv_hvr_900_analog,
> -               } },
> -       },
> +        [EM2883_BOARD_KWORLD_HYBRID_A316] = {
> +                .name         = "Kworld PlusTV HD Hybrid 330",
> +                .tuner_type   = TUNER_XC2028,
> +                .tuner_gpio   = default_tuner_gpio,
> +                .has_dvb      = 1,
> +                .dvb_gpio     = default_digital,
> +                .mts_firmware = 1,
> +                .decoder      = EM28XX_TVP5150,
> +                .input        = { {
> +                        .type     = EM28XX_VMUX_TELEVISION,
> +                        .vmux     = TVP5150_COMPOSITE0,
> +                        .amux     = EM28XX_AMUX_VIDEO,
> +                        .gpio     = default_analog,
> +                }, {
> +                        .type     = EM28XX_VMUX_COMPOSITE1,
> +                        .vmux     = TVP5150_COMPOSITE1,
> +                        .amux     = EM28XX_AMUX_LINE_IN,
> +                        .gpio     = default_analog,
> +                }, {
> +                        .type     = EM28XX_VMUX_SVIDEO,
> +                        .vmux     = TVP5150_SVIDEO,
> +                        .amux     = EM28XX_AMUX_LINE_IN,
> +                        .gpio     = default_analog,
> +                } },
> +        },
>         [EM2820_BOARD_COMPRO_VIDEOMATE_FORYOU] = {
>                 .name         = "Compro VideoMate ForYou/Stereo",
>                 .tuner_type   = TUNER_LG_PAL_NEW_TAPC,
> diff -r 211ae674f601 linux/drivers/media/video/em28xx/em28xx-dvb.c
> --- a/linux/drivers/media/video/em28xx/em28xx-dvb.c     Fri Jan 02 18:34:28
> 2009 -0200
> +++ b/linux/drivers/media/video/em28xx/em28xx-dvb.c     Sun Jan 04 01:24:39
> 2009 -0500
> @@ -29,6 +29,7 @@
>
>  #include "lgdt330x.h"
>  #include "zl10353.h"
> +#include "s5h1409.h"
>  #ifdef EM28XX_DRX397XD_SUPPORT
>  #include "drx397xD.h"
>  #endif
> @@ -231,6 +232,17 @@
>         .no_tuner = 1,
>         .parallel_ts = 1,
>         .if2 = 45600,
> +};
> +
> +static struct s5h1409_config em28xx_s5h1409_with_xc3028 = {
> +       .demod_address = 0x32 >> 1,
> +       .output_mode   = S5H1409_SERIAL_OUTPUT,
> +       .gpio          = S5H1409_GPIO_ON,
> +       .qam_if        = 44000,
> +       .inversion     = S5H1409_INVERSION_OFF,
> +       .status_mode   = S5H1409_DEMODLOCKING,
> +       .mpeg_timing   = S5H1409_MPEGTIMING_CONTINOUS_NONINVERTING_CLOCK
> +
>  };
>
>  #ifdef EM28XX_DRX397XD_SUPPORT
> @@ -413,7 +425,6 @@
>         case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_850:
>         case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_950:
>         case EM2880_BOARD_PINNACLE_PCTV_HD_PRO:
> -       case EM2883_BOARD_KWORLD_HYBRID_A316:
>         case EM2880_BOARD_AMD_ATI_TV_WONDER_HD_600:
>                 dvb->frontend = dvb_attach(lgdt330x_attach,
>                                            &em2880_lgdt3303_dev,
> @@ -447,6 +458,15 @@
>                 }
>                 break;
>  #endif
> +       case EM2883_BOARD_KWORLD_HYBRID_A316:
> +               dvb->frontend = dvb_attach(s5h1409_attach,
> +                                          &em28xx_s5h1409_with_xc3028,
> +                                          &dev->i2c_adap);
> +               if (attach_xc3028(0x61, dev) < 0) {
> +                       result = -EINVAL;
> +                       goto out_free;
> +               }
> +               break;
>         default:
>                 printk(KERN_ERR "%s/2: The frontend of your DVB/ATSC card"
>                                 " isn't supported yet\n",
> @@ -514,3 +534,4 @@
>
>  module_init(em28xx_dvb_register);
>  module_exit(em28xx_dvb_unregister);
> +
> [root@am2mm v4l-dvb]#
>
> Best Regards,
>
> --
> Rob Krakora
> Software Engineer
> MessageNet Systems
> 101 East Carmel Dr. Suite 105
> Carmel, IN 46032
> (317)566-1677 Ext. 206
> (317)663-0808 Fax

Hello Robert,

For getting a USB trace, I typically use SniffUSB:

http://www.pcausa.com/Utilities/UsbSnoop/default.htm

There could be several possibilities regarding the reason it doesn't
answer i2c requests.  The two most likely are that either
you have the wrong i2c address or you need the GPIOs to be setup
correctly because the s5h1409 is being held in reset (more likely).
Both of these can be determined from the USB trace.

If you send me the USB trace, I will take a look, and since I am
already working on a similar device I can compare the two traces.

Regards,

Devin

-- 
Devin J. Heitmueller
http://www.devinheitmueller.com
AIM: devinheitmueller

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
       [not found]     ` <20090104135840.7de113de@gmail.com>
@ 2009-01-04 17:14       ` CityK
  2009-01-04 17:25         ` Douglas Schilling Landgraf
  0 siblings, 1 reply; 12+ messages in thread
From: CityK @ 2009-01-04 17:14 UTC (permalink / raw)
  To: Douglas Schilling Landgraf
  Cc: video4linux-list, linux-dvb, Chehab, linux-media

Douglas Schilling Landgraf wrote:
> Hello,
>
> On Sun, 04 Jan 2009 01:45:14 -0500
> CityK <cityk@rogers.com> wrote:
>
>   
>> Devin Heitmueller wrote:
>>     
>>> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
>>> <rob.krakora@messagenetsystems.com> wrote:
>>>   
>>>       
>>>> Mauro:
>>>>
>>>> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
>>>> LGDT330X.  Hence the error initializing the LGDT330X in the
>>>> current source in em28xx-dvb.c.
>>>>
>>>> Best Regards,
>>>>     
>>>>         
>>> Hello Robert,
>>>
>>> Well, that's good to know.  I don't think anyone has done any work
>>> on that device recently, so I don't know why the code has it as an
>>> lgdt3303.
>>>       
>> I believe Douglas submitted this patch
>> (http://linuxtv.org/hg/v4l-dvb/rev/77f789d59de8) that got committed. 
>>     
>
> I don't remember sending this specific patch.
>   

Sorry, that was a memory association thing --- I had originally looked
at the KWorld patch (Mauro, Dec 9) at the same time that I had with the
ever so slightly older, and topically similar, HVR-850 patch (Douglas,
Dec 8; see http://linuxtv.org/hg/v4l-dvb/rev/402de62fe6a6).

Mauro -- can you shed any light in regards the "A316" part of the name
descriptor ?

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04 17:14       ` CityK
@ 2009-01-04 17:25         ` Douglas Schilling Landgraf
  2009-01-04 18:35           ` CityK
  0 siblings, 1 reply; 12+ messages in thread
From: Douglas Schilling Landgraf @ 2009-01-04 17:25 UTC (permalink / raw)
  To: CityK; +Cc: video4linux-list, linux-dvb, Chehab, linux-media

Hello,

On Sun, 04 Jan 2009 12:14:27 -0500
CityK <cityk@rogers.com> wrote:

> Douglas Schilling Landgraf wrote:
> > Hello,
> >
> > On Sun, 04 Jan 2009 01:45:14 -0500
> > CityK <cityk@rogers.com> wrote:
> >
> >   
> >> Devin Heitmueller wrote:
> >>     
> >>> On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
> >>> <rob.krakora@messagenetsystems.com> wrote:
> >>>   
> >>>       
> >>>> Mauro:
> >>>>
> >>>> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not
> >>>> the LGDT330X.  Hence the error initializing the LGDT330X in the
> >>>> current source in em28xx-dvb.c.
> >>>>
> >>>> Best Regards,
> >>>>     
> >>>>         
> >>> Hello Robert,
> >>>
> >>> Well, that's good to know.  I don't think anyone has done any work
> >>> on that device recently, so I don't know why the code has it as an
> >>> lgdt3303.
> >>>       
> >> I believe Douglas submitted this patch
> >> (http://linuxtv.org/hg/v4l-dvb/rev/77f789d59de8) that got
> >> committed. 
> >
> > I don't remember sending this specific patch.
> >   
> 
> Sorry, that was a memory association thing --- I had originally looked
> at the KWorld patch (Mauro, Dec 9) at the same time that I had with
> the ever so slightly older, and topically similar, HVR-850 patch
> (Douglas, Dec 8; see http://linuxtv.org/hg/v4l-dvb/rev/402de62fe6a6).

Sure, no problem.

> Mauro -- can you shed any light in regards the "A316" part of the name
> descriptor ?

Probably inherited from vendor/product ID, as follow:

em28xx-cards.c:

{ USB_DEVICE(0xeb1a, 0xa316), .driver_info = EM2883_BOARD_KWORLD_HYBRID_A316 },
                        ^^^

Cheers,
Douglas

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04 17:25         ` Douglas Schilling Landgraf
@ 2009-01-04 18:35           ` CityK
       [not found]             ` <20090104225927.01103345@pedra.chehab.org>
  0 siblings, 1 reply; 12+ messages in thread
From: CityK @ 2009-01-04 18:35 UTC (permalink / raw)
  To: Douglas Schilling Landgraf
  Cc: video4linux-list, linux-dvb, Chehab, linux-media

Douglas Schilling Landgraf wrote:
> CityK <cityk@rogers.com> wrote:
>> Mauro -- can you shed any light in regards the "A316" part of the name
>> descriptor ?
>>     
>
> Probably inherited from vendor/product ID, as follow:
>
> em28xx-cards.c:
>
> { USB_DEVICE(0xeb1a, 0xa316), .driver_info = EM2883_BOARD_KWORLD_HYBRID_A316 },
>                         ^^^

Ahh, very good.

Mauro, or whomever is going to end up spinning the patch incorporating
the correction regarding the demod. information, I suggest that the
descriptor "EM2883_BOARD_KWORLD_HYBRID_A316" be simultaneously changed
to "EM2883_BOARD_KWORLD_ATSC_330U" to reflect the device's more common name.



--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04  6:45   ` CityK
       [not found]     ` <20090104135840.7de113de@gmail.com>
@ 2009-01-04 19:28     ` Robert Krakora
  2009-01-05  2:43       ` Douglas Schilling Landgraf
  1 sibling, 1 reply; 12+ messages in thread
From: Robert Krakora @ 2009-01-04 19:28 UTC (permalink / raw)
  To: CityK; +Cc: Jerry Geis, video4linux-list, Mauro Carvalho Chehab

On Sun, Jan 4, 2009 at 1:45 AM, CityK <cityk@rogers.com> wrote:

> Devin Heitmueller wrote:
> > On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
> > <rob.krakora@messagenetsystems.com> wrote:
> >
> >> Mauro:
> >>
> >> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not the
> >> LGDT330X.  Hence the error initializing the LGDT330X in the current
> source
> >> in em28xx-dvb.c.
> >>
> >> Best Regards,
> >>
> >
> > Hello Robert,
> >
> > Well, that's good to know.  I don't think anyone has done any work on
> > that device recently, so I don't know why the code has it as an
> > lgdt3303.
>
> I believe Douglas submitted this patch
> (http://linuxtv.org/hg/v4l-dvb/rev/77f789d59de8) that got committed.
>
> I've been meaning to get back to this because the "A316" part of the
> name caught my attention -- I do not recall having seen such a reference
> made by KWorld, nor is it typical of their nomenclature style, rather,
> it is entirely consistent with that used by AVerMedia
>
>
>
>
Douglas:

A316 is actually the product ID portion of the USB vendor/product IDs.  It
should be 330 instead of A316.

Best Regards,

-- 
Rob Krakora
Software Engineer
MessageNet Systems
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
(317)663-0808 Fax
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
       [not found]             ` <20090104225927.01103345@pedra.chehab.org>
@ 2009-01-05  1:19               ` CityK
  0 siblings, 0 replies; 12+ messages in thread
From: CityK @ 2009-01-05  1:19 UTC (permalink / raw)
  To: Robert Krakora; +Cc: video4linux-list, linux-dvb, linux-media

I created a skeleton page for the 330U (
http://www.linuxtv.org/wiki/index.php/KWorld_ATSC_330U ). Robert, if it
is possible, could you provide/upload any photos, and/or further info.
Thanks.


--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: KWorld 330U Employs Samsung S5H1409X01 Demodulator
  2009-01-04 19:28     ` Robert Krakora
@ 2009-01-05  2:43       ` Douglas Schilling Landgraf
  0 siblings, 0 replies; 12+ messages in thread
From: Douglas Schilling Landgraf @ 2009-01-05  2:43 UTC (permalink / raw)
  To: Robert Krakora
  Cc: Jerry Geis, video4linux-list, linux-media, CityK,
	Mauro Carvalho Chehab

On Sun, 4 Jan 2009 14:28:11 -0500
"Robert Krakora" <rob.krakora@messagenetsystems.com> wrote:

> On Sun, Jan 4, 2009 at 1:45 AM, CityK <cityk@rogers.com> wrote:
> 
> > Devin Heitmueller wrote:
> > > On Sat, Jan 3, 2009 at 11:21 PM, Robert Krakora
> > > <rob.krakora@messagenetsystems.com> wrote:
> > >
> > >> Mauro:
> > >>
> > >> The KWorld 330U employs the Samsung S5H1409X01 demodulator, not
> > >> the LGDT330X.  Hence the error initializing the LGDT330X in the
> > >> current
> > source
> > >> in em28xx-dvb.c.
> > >>
> > >> Best Regards,
> > >>
> > >
> > > Hello Robert,
> > >
> > > Well, that's good to know.  I don't think anyone has done any
> > > work on that device recently, so I don't know why the code has it
> > > as an lgdt3303.
> >
> > I believe Douglas submitted this patch
> > (http://linuxtv.org/hg/v4l-dvb/rev/77f789d59de8) that got committed.
> >
> > I've been meaning to get back to this because the "A316" part of the
> > name caught my attention -- I do not recall having seen such a
> > reference made by KWorld, nor is it typical of their nomenclature
> > style, rather, it is entirely consistent with that used by AVerMedia
> >
> >
> >
> >
> Douglas:
> 
> A316 is actually the product ID portion of the USB vendor/product
> IDs.  It should be 330 instead of A316.

I agree with that. It should be replaced.

Cheers,
Douglas

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2009-01-05  2:43 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-04  4:21 KWorld 330U Employs Samsung S5H1409X01 Demodulator Robert Krakora
2009-01-04  5:18 ` Devin Heitmueller
2009-01-04  5:55   ` Robert Krakora
2009-01-04  6:31     ` Robert Krakora
2009-01-04 13:47       ` Devin Heitmueller
2009-01-04  6:45   ` CityK
     [not found]     ` <20090104135840.7de113de@gmail.com>
2009-01-04 17:14       ` CityK
2009-01-04 17:25         ` Douglas Schilling Landgraf
2009-01-04 18:35           ` CityK
     [not found]             ` <20090104225927.01103345@pedra.chehab.org>
2009-01-05  1:19               ` CityK
2009-01-04 19:28     ` Robert Krakora
2009-01-05  2:43       ` Douglas Schilling Landgraf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox