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

2011-09-07 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: Snip We've added a parameter for that on xawtv3 (--alsa-latency). We've parametrized it at the alsa stream function call.

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 mche...@redhat.com 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

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

2011-09-06 Thread Mauro Carvalho Chehab
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 xawtv, in order to make it to work on a wider range of V4L and sound

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 from

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 mche...@redhat.com 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: [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: snip 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

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 hdego...@redhat.com wrote: Hi, On 09/06/2011 06:24 PM, Devin Heitmueller wrote: snip 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

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 mche...@redhat.com 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

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 hdego...@redhat.com 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

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 Goedehdego...@redhat.com wrote: Hi, On 09/06/2011 06:24 PM, Devin Heitmueller wrote: snip I've been thinking for a while that perhaps the project should be renamed (or I considered prepending kl onto

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 hdego...@redhat.com wrote: Hi, On 09/06/2011 08:35 PM, Michael Krufky wrote: On Tue, Sep 6, 2011 at 2:19 PM, Hans de Goedehdego...@redhat.com  wrote: Hi, On 09/06/2011 06:24 PM, Devin Heitmueller wrote: snip I've been thinking for a while

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 hdego...@redhat.com 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

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 mche...@redhat.com 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

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 mche...@redhat.com 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

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 mche...@redhat.com 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

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 dheitmuel...@kernellabs.com wrote: On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab mche...@redhat.com wrote: There are several issues with the original alsa_stream code that got fixed on xawtv3, made by me and by Hans de Goede.

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 mche...@redhat.com 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

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 dheitmuel...@kernellabs.com wrote: On Tue, Sep 6, 2011 at 11:29 AM, Mauro Carvalho Chehab mche...@redhat.com wrote: There are several issues with the original alsa_stream code that got fixed

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 mche...@redhat.com 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

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 dheitmuel...@kernellabs.com wrote: On Tue, Sep 6, 2011 at 11:29 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: Basically the above starts at the *maximum* capture resolution and works its way down.  One might argue that this heuristic

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 dheitmuel...@kernellabs.com 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