Re: [ivtv-devel] PVR150 Tinny/fuzzy audio w/ patch?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Donnerstag, 31. Dezember 2009 20:40:15 schrieb Andy Walls:
> On Thu, 2009-12-31 at 14:26 -0500, Andy Walls wrote:
> > On Thu, 2009-12-31 at 18:15 +0100, Martin Dauskardt wrote:
> 
> Some corrections to errors:
> > My preferences in summary, is that not matter what the digitizer chip:
> 
> My preferences are, in summary, that no matter what the digitizer chip:
> > a. I'd like to keep the audio clocks always up to avoid tinny audio.
> >
> > b. I'd also like to inhibit the video clock and add the delay after
> 
>                                                   ^^^
>                                                   refine
> 
> > re-enabling the digitizer to avoid the *potential* for a hung machine.
> 
> A value smaller than 300 ms should work, but a value smaller than 40 ms
> may not work, if my hypothesis is correct.
> 
> > c. I do not care to much about the delay after disbaling the video
> > clock, only that it is empirically "long enough".
> >
> > Thanks for taking the time to test and comment.
> >
> > Regards,
> > Andy
> 
> Regards,
> Andy

I tested various sleep values:
http://home.arcor-online.de/martin.dauskardt/digitizer_msleep.xls

It seems that we only need a total delay of 300ms between the previous actions 
in ivtv-streams.c and the start of the capture. 

This also secures that we don't see disturbance from a previous frequency 
switch. This happens with smaller sleep values:
http://home.arcor-online.de/martin.dauskardt/channelswitch.mpg
and can lead to the infamous flickering problem (http://www.gossamer-
threads.com/lists/ivtv/devel/32970) . The current driver has this problem only 
with cx23415 (PVR350), not cx23416.

My preference is:
-let it how it is (300 ms sleep before the firmware call)
or
-split the 300ms to 150 and 150

Greets,
Martin
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux