Re: Gene's pcHDTV 3000 analog problem

[Gene Heskett]

On Thursday 01 December 2005 17:45, Michael Krufky wrote:
>Gene Heskett wrote:
>>On Thursday 01 December 2005 17:06, Perry Gilfillan wrote:
>>>Mauro Carvalho Chehab wrote:
>>>>After checking the datasheets of Thompson tuner, and I have one
>>>>guess:
>>>>
>>>>At board description, tda9887 is not there. This tuner needs to work
>>>>properly.
>>>>
>>>>This small patch does enable it for your board.
>>>>
>>>>You should notice that you may need to use some parameters for
>>>>tda0887 to work properly, like using port1=0 port2=0 qss=0 as insmod
>>>>options for this module. (these are some on/off bits at the chip, to
>>>>enable some special functions - if 0/0/0 doesn't work you may need
>>>> to test 0/0/1, .. 1/1/1).
>>>
>>>This has fixed it for me!!  I compiled todays cvs, with out this
>>> patch, to watch it fail, then added the line as noted, and reloaded
>>> the modules without rebooting, and it worked.  I did a cold start to
>>> see that it is repeatable, and it continues to work.  I used no
>>> extra parameters, so the defaults are working fine.
>>
>>I haven't built it yet, had to apply the patch by hand for some reason
>>here, after doing a cvs up -D today.
>>
>>2.6.15-rc4 under construction to test it.
>
>Gene, dont bother applying the patch to the cvs code -- It is obviously
>correct, so I have merged it into v4l-dvb cvs.

I did a cvs up -D 2005-12-1, about 1.5 hours ago and it hadn't been
applied yet.

>A simple cvs checkout will get you everything you need.
I'll resuck the whole thing.

>If you like, try to apply it against the kernel, without using cvs.

Ok, that I can probably do.  Unforch, there is a miss-cue in that
file, rc4 says:
struct cx88_board cx88_boards[] = {
        [CX88_BOARD_UNKNOWN] = {
                .name           = "UNKNOWN/GENERIC",
                .tuner_type     = UNSET,
                .radio_type     = UNSET,
                .tuner_addr     = ADDR_UNSET,
                .radio_addr     = ADDR_UNSET,
                .input          = {{
                        .type   = CX88_VMUX_COMPOSITE1,

Note also that the .type = label has been changed.  I'm going to change
that line back to CX88_VMUX_TELEVISION, just for grins.

While the patch says:
@@ -569,6 +569,7 @@ struct cx88_board cx88_boards[] = {
                .radio_type     = UNSET,
                .tuner_addr     = ADDR_UNSET,
                .radio_addr     = ADDR_UNSET,
                .tda9887_conf   = TDA9887_PRESENT,
                .input          = {{
                        .type   = CX88_VMUX_TELEVISION,
                        .vmux   = 0,
And I've added that line from the patch to just above the ".input"
line, its rebuilding now.  And a cold reboot will test it, so I'll send
this first just to let you know not all is well in tv land.

>Anyhow, surely it will work.  Thank you Gene, Don and Perry for helping
>us to solve this bug......... Hopefully we'll be able to get the fix
>into 2.6.15.
>
>Cheers,
>
>Michael Krufky

-- 
Cheers, Gene
"There are four boxes to be used in defense of liberty:
 soap, ballot, jury, and ammo. Please use in that order."
-Ed Howdershelt (Author)
99.36% setiathome rank, not too shabby for a WV hillbilly
Yahoo.com and AOL/TW attorneys please note, additions to the above
message by Gene Heskett are:
Copyright 2005 by Maurice Eugene Heskett, all rights reserved.

-
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/