Re: Gene's pcHDTV 3000 analog problem

Gene Heskett
2005-12-02T00:05:06Z

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.