Re: Some fixes for alsa_stream

2011-06-16 Thread Mauro Carvalho Chehab
Em 16-06-2011 15:19, Hans de Goede escreveu: > Hi, > 1) pass -alsa-pb hw to xawtv (requires no pulseaudio knowledge) > 2) Set a different default alsa device in .alsarc >(requires no pulseaudio knowledge, trivial alsa knowledge) > 3) Do rpm -e alsa-plugins-pulseaudio > > Noe that 3 will p

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/16/2011 05:35 PM, Mauro Carvalho Chehab wrote: Em 16-06-2011 12:11, Hans de Goede escreveu: While developers are not comfortable with pulseaudio, turning it into a default is a bad idea. 1) Some developers are fine with pulseaudio, so please speak on behalf of yourself rathe

Re: Some fixes for alsa_stream

2011-06-16 Thread Mauro Carvalho Chehab
Em 16-06-2011 12:11, Hans de Goede escreveu: >>> I've also changed the default -alsa-pb value to "default" as we should >>> not be picking something else then the user / distro configured defaults >>> for output IMHO. The user can set a generic default in alsarc, and override >>> that on the cmdli

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/16/2011 04:38 PM, Mauro Carvalho Chehab wrote: Em 16-06-2011 09:29, Hans de Goede escreveu: Note that I've just pushed a patch set which includes rewritten period / buf size negotiation and a bunch of cleanups in general. This removes over 150 lines of code, while at the same time

Re: Some fixes for alsa_stream

2011-06-16 Thread Mauro Carvalho Chehab
Em 16-06-2011 09:29, Hans de Goede escreveu: > Hi, > > On 06/15/2011 05:45 PM, Mauro Carvalho Chehab wrote: > > > >> 1) try to find a common buffer size that are acceptable by both drivers, >> as using the same buffer size helps to avoid memcpy's, especially if >> mmap mode is enabled;

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/15/2011 05:45 PM, Mauro Carvalho Chehab wrote: 1) try to find a common buffer size that are acceptable by both drivers, as using the same buffer size helps to avoid memcpy's, especially if mmap mode is enabled; This is not needed, using the same buffer size does nothing t

Re: Some fixes for alsa_stream

2011-06-15 Thread Mauro Carvalho Chehab
Em 15-06-2011 11:35, Hans de Goede escreveu: >>> This is a combination of the assumption there is a shared period size >>> between >>> the input device and the output device + the broken error handling. >> >> The code is doing a negotiation, in order to find a period that are >> acceptable >> by

Re: Some fixes for alsa_stream

2011-06-15 Thread Mauro Carvalho Chehab
Em 15-06-2011 11:35, Hans de Goede escreveu: > Hi, > > On 06/15/2011 04:25 PM, Mauro Carvalho Chehab wrote: >> Em 15-06-2011 10:43, Hans de Goede escreveu: > > > >>> Right, because ConsoleKit ensures that devices like floppydrives, cdroms, >>> audio cards, >>> webcams, etc. are only available

Re: Some fixes for alsa_stream

2011-06-15 Thread Mauro Carvalho Chehab
Em 15-06-2011 11:25, Mauro Carvalho Chehab escreveu: > Em 15-06-2011 10:43, Hans de Goede escreveu: >> Hi, >> >> On 06/14/2011 04:45 PM, Mauro Carvalho Chehab wrote: >>> Em 14-06-2011 11:37, Hans de Goede escreveu: Hi, On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: > Em 14-

Re: Some fixes for alsa_stream

2011-06-15 Thread Hans de Goede
Hi, On 06/15/2011 04:25 PM, Mauro Carvalho Chehab wrote: Em 15-06-2011 10:43, Hans de Goede escreveu: >> Right, because ConsoleKit ensures that devices like floppydrives, cdroms, audio cards, >> webcams, etc. are only available to users sitting behind the console, > > This is a wrong assump

Re: Some fixes for alsa_stream

2011-06-15 Thread Mauro Carvalho Chehab
Em 15-06-2011 10:43, Hans de Goede escreveu: > Hi, > > On 06/14/2011 04:45 PM, Mauro Carvalho Chehab wrote: >> Em 14-06-2011 11:37, Hans de Goede escreveu: >>> Hi, >>> >>> On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 10:52, Devin Heitmueller escreveu: >>> >>> >>> Y

Re: Some fixes for alsa_stream

2011-06-15 Thread Hans de Goede
Hi, On 06/14/2011 04:45 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 11:37, Hans de Goede escreveu: Hi, On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 10:52, Devin Heitmueller escreveu: Yes. The default for capture is the one detected via sysfs. The default for pl

Re: Some fixes for alsa_stream

2011-06-14 Thread Mauro Carvalho Chehab
Em 14-06-2011 11:48, Devin Heitmueller escreveu: > On Tue, Jun 14, 2011 at 10:45 AM, Mauro Carvalho Chehab > wrote: >> Pulseaudio sucks. See what happens when I pass "-alsa-pb default" argument >> to pulseaudio: >> >> 1) ssh section. User is the same as the console owner: >> >> ALSA lib pulse.c:2

Re: Some fixes for alsa_stream

2011-06-14 Thread Devin Heitmueller
On Tue, Jun 14, 2011 at 10:45 AM, Mauro Carvalho Chehab wrote: > Pulseaudio sucks. See what happens when I pass "-alsa-pb default" argument to > pulseaudio: > > 1) ssh section. User is the same as the console owner: > > ALSA lib pulse.c:229:(pulse_connect) PulseAudio: Unable to connect: > Connec

Re: Some fixes for alsa_stream

2011-06-14 Thread Mauro Carvalho Chehab
Em 14-06-2011 11:37, Hans de Goede escreveu: > Hi, > > On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: >> Em 14-06-2011 10:52, Devin Heitmueller escreveu: > > > >> Yes. >> >> The default for capture is the one detected via sysfs. >> >> The default for playback is not really hw:0,0. It defa

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 10:52, Devin Heitmueller escreveu: Yes. The default for capture is the one detected via sysfs. The default for playback is not really hw:0,0. It defaults to the first hw: that it is not associated with a video device.

Re: Some fixes for alsa_stream

2011-06-14 Thread Mauro Carvalho Chehab
Em 14-06-2011 10:52, Devin Heitmueller escreveu: > On Tue, Jun 14, 2011 at 9:47 AM, Hans de Goede wrote: >> Hmm, we really don't need more cmdline options IMHO, it is quite easy to >> detect >> if an alsa device supports mmap mode, and if not fall back to r/w mode, I >> know >> several programs wh

Re: Some fixes for alsa_stream

2011-06-14 Thread Devin Heitmueller
On Tue, Jun 14, 2011 at 9:47 AM, Hans de Goede wrote: > Hmm, we really don't need more cmdline options IMHO, it is quite easy to > detect > if an alsa device supports mmap mode, and if not fall back to r/w mode, I > know > several programs which do that (some if which I've written the patches to d

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 03:05 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 09:48, Hans de Goede escreveu: Hi, On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: Hi Devin, I've made a few fixes for your alsa_stream.c, used on tvtime. They are at: http://git.linuxtv.org/xawtv3.git In part

Re: Some fixes for alsa_stream

2011-06-14 Thread Mauro Carvalho Chehab
Em 14-06-2011 10:05, Mauro Carvalho Chehab escreveu: > Em 14-06-2011 09:48, Hans de Goede escreveu: >> Hi, >> >> On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: >>> Hi Devin, >>> >>> I've made a few fixes for your alsa_stream.c, used on tvtime. >>> They are at: >>> http://git.linuxtv.org/x

Re: Some fixes for alsa_stream

2011-06-14 Thread Mauro Carvalho Chehab
Em 14-06-2011 09:48, Hans de Goede escreveu: > Hi, > > On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: >> Hi Devin, >> >> I've made a few fixes for your alsa_stream.c, used on tvtime. >> They are at: >> http://git.linuxtv.org/xawtv3.git >> >> >> In particular, those are the more interesti

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: Hi Devin, I've made a few fixes for your alsa_stream.c, used on tvtime. They are at: http://git.linuxtv.org/xawtv3.git In particular, those are the more interesting ones: commit a1bb5ade5c2b09d6d6d624d18025f9e2c4398495 als

Some fixes for alsa_stream

2011-06-13 Thread Mauro Carvalho Chehab
Hi Devin, I've made a few fixes for your alsa_stream.c, used on tvtime. They are at: http://git.linuxtv.org/xawtv3.git In particular, those are the more interesting ones: commit a1bb5ade5c2b09d6d6d624d18025f9e2c4398495 alsa_stream: negotiate the frame rate Without this patch, one o