Perry Gilfillan wrote:
> I've been able to pinpoint the time when the v4l-dvb cvs failes for
> the pcHDTV 3000 tuner!!
>
> This test is with gentoo-sources-2.6.13-r5, with v4l-dvb cvs. As
> Michael pointed out, 2.6.15 has bugs. For my hardware, 2.6.13, and 14
> have major issues as well...
>
> My procedure is to do a distclean in the v4l-dvb cvs directory,
> checkout the repository for the given date, make, and install. Then I
> inspect the /lib/modules/.../media/video dirtree to be sure no old ko
> files are lingering. I do a cold boot to be sure the hardware is
> fully initialized from a known state. After many iterations, I've
> narrowed it down to this change set:
>
> cvs as of 2005-10-17 16:01:
> Analog TV works, as well as composite and s-video.
>
> Changes made at 2005-10-17 16:02: bROKEN!!
>
> cvs as of 2005-10-17 16:10:
> Analog failes, s-video and composite work.
>
>
> I'll leave it to more capable persons to figure how it failed.
Hmm... These times of day don't coincide with any actual commits....
Anyhow, this was the day Hartmut added tda8275a support. It broke many
things, which we thought that we had eventually fixed. Maybe there are
still a few bugs as a result of this?
This is enough for me to put together a test patch for you, but I cant
do it right now..... I'll try to email you an experimental patch tonight
or tomorrow that will remove tda8275a support, just to confirm that this
is in fact the problem.
If it turns out that I'm right, then we'll have to figure out a way to
fix it, since reverting Hartmut's patch is NOT an option.
Thanks for the testing!
-Mike
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).
Cheers,
Mauro.
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.
Cheers,
Perry
>Cheers,
>Mauro.
>
>
>------------------------------------------------------------------------
>
>Index: linux/drivers/media/video/cx88/cx88-cards.c
>===================================================================
>RCS file: /cvs/video4linux/v4l-dvb/linux/drivers/media/video/cx88/cx88-cards.c,v
>retrieving revision 1.108
>diff -u -p -r1.108 cx88-cards.c
>--- linux/drivers/media/video/cx88/cx88-cards.c 25 Nov 2005 10:24:13 -0000 1.108
>+++ linux/drivers/media/video/cx88/cx88-cards.c 1 Dec 2005 20:56:43 -0000
>@@ -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,
>
>
On Thursday 01 December 2005 16:00, Mauro Carvalho Chehab wrote:
I also took Hartmut out of the Cc: list, he is mailbombing me with a
sender id scheme of some kind for every reply I make to All:
Its very poor form to do that to a mailing list although I suspect its
because he is in the CC list and getting it directly if I don't remove
him from it. He doesn't need 2 copies anyway.
> 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).
I'm puzzled. In 2.6.14.3, that tuner is not loaded, and its working
fine other than spamming the logs by the megabyte about
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561c
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561c
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561b
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561b
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561b
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561b
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc02c5638
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc054561d
Nov 29 22:56:18 coyote last message repeated 2 times
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561c
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc0445624
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc008561c
Nov 29 22:56:18 coyote kernel: CORE IOCTL: 0xc054561d
Nov 29 22:56:49 coyote last message repeated 907 times
Nov 29 22:57:50 coyote last message repeated 1828 times
Nov 29 22:58:46 coyote last message repeated 1703 times
Nov 29 22:58:46 coyote kernel: CORE IOCTL: 0xc008561c
Other than that, its fine in ntsc.
I'm currently doing a 'modprobe cx88-dvb' to load everything, but you
mention using insmod with arguments above, so please clarify as to how
I should proceed after applying that patch to, oh lets live dangerously
and do it to 2.6.15-rc4.
>Cheers,
>Mauro.
--
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.
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.
>
> Cheers,
>
> Perry
>
>> Cheers, Mauro.
>>
>> ------------------------------------------------------------------------
>>
>> Index: linux/drivers/media/video/cx88/cx88-cards.c
>> ===================================================================
>> RCS file:
>> /cvs/video4linux/v4l-dvb/linux/drivers/media/video/cx88/cx88-cards.c,v
>> retrieving revision 1.108
>> diff -u -p -r1.108 cx88-cards.c
>> --- linux/drivers/media/video/cx88/cx88-cards.c 25 Nov 2005
>> 10:24:13 -0000 1.108
>> +++ linux/drivers/media/video/cx88/cx88-cards.c 1 Dec 2005
>> 20:56:43 -0000
>> @@ -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,
>
HURRAY!!!!!
Signed-off-by: Michael Krufky <[email protected]>
... I'll go add this to cvs now.
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.
>
>Cheers,
>
>Perry
>
>>Cheers,
>>Mauro.
>>
>>
>>----------------------------------------------------------------------
>>--
>>
>>Index: linux/drivers/media/video/cx88/cx88-cards.c
>>===================================================================
>>RCS file:
>> /cvs/video4linux/v4l-dvb/linux/drivers/media/video/cx88/cx88-cards.c,
>>v retrieving revision 1.108
>>diff -u -p -r1.108 cx88-cards.c
>>--- linux/drivers/media/video/cx88/cx88-cards.c 25 Nov 2005 10:24:13
>> -0000 1.108 +++ linux/drivers/media/video/cx88/cx88-cards.c 1 Dec
>> 2005 20:56:43 -0000 @@ -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,
--
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.
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.
A simple cvs checkout will get you everything you need.
If you like, try to apply it against the kernel, without using cvs.
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
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.
On Thu, 01 Dec 2005 19:00:59 -0200
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).
>
> Cheers,
> Mauro.
>
Works fine here! Thanks, Mauro!!
The default settings worked.
-d