Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-07 Thread Mauro Carvalho Chehab
Em 07-09-2011 00:37, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 11:29 PM, Mauro Carvalho Chehab > wrote: >>> Basically the above starts at the *maximum* capture resolution and >>> works its way down. One might argue that this heuristic makes more >>> sense anyway - why *wouldn't* you wa

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Hans de Goede
Hi, Lots of good stuff in this thread! It seems Mauro has answered most things, so I'm just going to respond to this bit. On 09/07/2011 05:37 AM, Devin Heitmueller wrote: We've added a parameter for that on xawtv3 (--alsa-latency). We've parametrized it at the alsa stream function call. So, a

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 11:42 PM, Devin Heitmueller wrote: > One more thing worth noting before I quit for the night: > > What audio processor is on your WinTV USB 2 device?  The DVC-90 has an > emp202.  Perhaps the WInTV uses a different audio processor (while > still using an em2820 as the bridge

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 11:37 PM, Devin Heitmueller wrote: > On Tue, Sep 6, 2011 at 11:29 PM, Mauro Carvalho Chehab > wrote: >>> Basically the above starts at the *maximum* capture resolution and >>> works its way down.  One might argue that this heuristic makes more >>> sense anyway - why *wouldn

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 11:29 PM, Mauro Carvalho Chehab wrote: >> Basically the above starts at the *maximum* capture resolution and >> works its way down.  One might argue that this heuristic makes more >> sense anyway - why *wouldn't* you want the highest quality audio >> possible by default (rat

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Em 07-09-2011 00:20, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 10:58 PM, Devin Heitmueller > wrote: >> On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab >> wrote: >>> There are several issues with the original alsa_stream code that got >>> fixed on xawtv3, made by me and by Hans d

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Em 06-09-2011 23:58, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab > wrote: >> There are several issues with the original alsa_stream code that >> got fixed on xawtv3, made by me and by Hans de Goede. Basically, >> the code were re-written, in order to follo

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 10:58 PM, Devin Heitmueller wrote: > On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab > wrote: >> There are several issues with the original alsa_stream code that got >> fixed on xawtv3, made by me and by Hans de Goede. Basically, the >> code were re-written, in order

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab wrote: > There are several issues with the original alsa_stream code that got > fixed on xawtv3, made by me and by Hans de Goede. Basically, the > code were re-written, in order to follow the alsa best practises. > > Backport the changes from

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Em 06-09-2011 18:18, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 3:12 PM, Mauro Carvalho Chehab > wrote: >>> From a practical standpoint, the Ubuntu folks have the original tvtime >>> tarball and all their changes in one patch, which is clearly a bunch >>> of patches that are mashed toget

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 3:12 PM, Mauro Carvalho Chehab wrote: >> From a practical standpoint, the Ubuntu folks have the original tvtime >> tarball and all their changes in one patch, which is clearly a bunch >> of patches that are mashed together probably in their build system. I >> need to reach o

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Em 06-09-2011 15:41, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goede wrote: >> I think that what should be done is contact the debian / ubuntu maintainers, >> get any interesting fixes they have which the kl version misses merged, >> and then just declare the kl version

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Michael Krufky
On Tue, Sep 6, 2011 at 2:43 PM, Hans de Goede wrote: > Hi, > > On 09/06/2011 08:35 PM, Michael Krufky wrote: >> >> On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goede  wrote: >>> >>> Hi, >>> >>> On 09/06/2011 06:24 PM, Devin Heitmueller wrote: >>> >>> >>> I've been thinking for a while that perhap

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Hans de Goede
Hi, On 09/06/2011 08:35 PM, Michael Krufky wrote: On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goede wrote: Hi, On 09/06/2011 06:24 PM, Devin Heitmueller wrote: I've been thinking for a while that perhaps the project should be renamed (or I considered prepending "kl" onto the front resulting i

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goede wrote: > I think that what should be done is contact the debian / ubuntu maintainers, > get any interesting fixes they have which the kl version misses merged, > and then just declare the kl version as being the new official upstream > (with the blessi

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Em 06-09-2011 13:24, Devin Heitmueller escreveu: > On Tue, Sep 6, 2011 at 11:40 AM, Mauro Carvalho Chehab > wrote: >> Hi Devin, >> >> Em 06-09-2011 12:29, Mauro Carvalho Chehab escreveu: >>> There are several issues with the original alsa_stream code that got >>> fixed on xawtv3, made by me and b

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Michael Krufky
On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goede wrote: > Hi, > > On 09/06/2011 06:24 PM, Devin Heitmueller wrote: > > > >> I've been thinking for a while that perhaps the project should be >> renamed (or I considered prepending "kl" onto the front resulting in >> it being called "kl-tvtime").  This

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Hans de Goede
Hi, On 09/06/2011 06:24 PM, Devin Heitmueller wrote: I've been thinking for a while that perhaps the project should be renamed (or I considered prepending "kl" onto the front resulting in it being called "kl-tvtime"). This isn't out of vanity but rather my concern that the fork will get conf

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Devin Heitmueller
On Tue, Sep 6, 2011 at 11:40 AM, Mauro Carvalho Chehab wrote: > Hi Devin, > > Em 06-09-2011 12:29, Mauro Carvalho Chehab escreveu: >> There are several issues with the original alsa_stream code that got >> fixed on xawtv3, made by me and by Hans de Goede. Basically, the >> code were re-written, in

Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

2011-09-06 Thread Mauro Carvalho Chehab
Hi Devin, Em 06-09-2011 12:29, Mauro Carvalho Chehab escreveu: > There are several issues with the original alsa_stream code that got > fixed on xawtv3, made by me and by Hans de Goede. Basically, the > code were re-written, in order to follow the alsa best practises. > > Backport the changes fro