Re: [ANNOUNCE] experimental alsa stream support at xawtv3

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

 



On Monday, May 23, 2011 22:17:06 Mauro Carvalho Chehab wrote:
> Due to the alsa detection code that I've added at libv4l2util (at v4l2-utils)
> during the weekend, I decided to add alsa support also on xawtv3, basically
> to provide a real usecase example. Of course, for it to work, it needs the
> very latest v4l2-utils version from the git tree.

Please, please add at the very least some very big disclaimer in libv4l2util
that the API/ABI is likely to change. As mentioned earlier, this library is
undocumented, has not gone through any peer-review, and I am very unhappy with
it and with the decision (without discussion it seems) to install it.

Once you install it on systems it becomes much harder to change.

Regards,

	Hans

> I've basically added there the code that Devin wrote for tvtime, with a few
> small fixes and with the audio device auto-detection.
> 
> With this patch, xawtv will now get the alsa device associated with a video
> device node (if any), and start streaming from it, on a separate thread.
> 
> As the code is the same as the one at tvtime, it should work at the
> same devices that are supported there. I tested it only on two em28xx devices:
> 	- HVR-950;
> 	- WinTV USB-2.
> 
> It worked with HVR-950, but it didn't work with WinTV USB-2. It seems that
> snd-usb-audio do something different to set the framerate, that the alsa-stream
> code doesn't recognize. While I didn't test, I think it probably won't work
> with saa7134, as the code seems to hardcode the frame rate to 48 kHz, but
> saa7134 supports only 32 kHz.
> 
> It would be good to add an option to disable this behavior and to allow manually
> select the alsa out device, so please send us patches ;)
> 
> Anyway, patches fixing it and more tests are welcome.
> 
> The git repositories for xawtv3 and v4l-utils is at:
> 
> http://git.linuxtv.org/xawtv3.git
> http://git.linuxtv.org/v4l-utils.git
> 
> Thanks,
> Mauro.
> --
> 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
> 
> 
--
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