From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: "Rémi Denis-Courmont" <remi@remlab.net>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [ANNOUNCE] experimental alsa stream support at xawtv3
Date: Sat, 28 May 2011 11:39:54 -0300 [thread overview]
Message-ID: <4DE1093A.6070003@redhat.com> (raw)
In-Reply-To: <201105281555.28285.remi@remlab.net>
Em 28-05-2011 09:55, Rémi Denis-Courmont escreveu:
> Le mardi 24 mai 2011 17:09:45 Mauro Carvalho Chehab, vous avez écrit :
>> If we do that, then all other places where the association between an alsa
>> device and a video4linux node is needed will need to copy it, and we'll
>> have a fork. Also, we'll keep needing it at v4l-utils, as it is now needed
>> by the new version of v4l2-sysfs-path tool.
>>
>> Btw, this lib were created due to a request from the vlc maintainer that
>> something like that would be needed. After finishing it, I decided to add
>> it at xawtv in order to have an example about how to use it.
>
> Hmm errm, I said VLC would need to be able to match a V4L2 device to an ALSA
> input (where applicable). Currently, V4L2 devices are enumerated with
> (lib)udev though. I am not very clear how v4l2-utils fits there (and oh, ALSA
> is a bitch for udev-hotplugging but I'm getting side tracked).
Once you have a V4L2 device, it will use something similar to (lib)udev to get
the associated alsa device for that video input.
> I guess I misunderstood that /dev/media would logically group related devices.
> Now I guess it is _solely_ intended to plug DSPs together à la OpenMAX IL.
> Sorry about that.
Although people is thinking and discussing about using it also on other subsystems,
it is currently limited to video4linux only. As you said, the current focus
is to plug DSPs.
>>> Mauro, I plan to do a new v4l-utils release soon (*), maybe even today. I
>>> consider it unpolite to revert other peoples commits, so I would prefer
>>> for you to revert the install libv4l2util.a patch yourself. But if you
>>> don't (or don't get around to doing it before I do the release), I will
>>> revert it, as this clearly needs more discussion before making it into
>>> an official release tarbal (we can always re-introduce the patch after
>>> the release).
>>
>> I'm not a big fan or exporting the rest of stuff at libv4l2util.a either,
>> but I think that at least the get_media_devices stuff should be exported
>> somewhere, maybe as part of libv4l.
>
> Should it be exposed as a udev device attribute instead then?
An udev attribute can be added to allow such association on devices where both
audio and video are handled by a driver at /drivers/media (I'm actually thinking
on using udev uevent instead, as there's nothing that you can control with it).
This won't cover 100% of the cases, as some devices just provide a standard Usb
Audio Class for audio. So, the standard driver (snd-usb-audio) will handle the
audio part without knowing anything about the video part of the device.
The current library will handle such case by detecting that both audio and video
nodes belong to the same physical device.
Cheers,
Mauro
next prev parent reply other threads:[~2011-05-28 14:39 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-23 20:17 [ANNOUNCE] experimental alsa stream support at xawtv3 Mauro Carvalho Chehab
2011-05-23 20:19 ` Devin Heitmueller
2011-05-23 20:30 ` Mauro Carvalho Chehab
2011-05-23 20:32 ` Mauro Carvalho Chehab
2011-05-24 6:50 ` Hans Verkuil
2011-05-24 7:21 ` Hans de Goede
2011-05-24 14:09 ` Mauro Carvalho Chehab
2011-05-24 15:55 ` Hans de Goede
2011-05-28 12:44 ` Mauro Carvalho Chehab
2011-05-28 13:01 ` Rémi Denis-Courmont
2011-05-28 14:41 ` Mauro Carvalho Chehab
2011-05-28 14:10 ` Mauro Carvalho Chehab
2011-05-28 12:55 ` Rémi Denis-Courmont
2011-05-28 14:39 ` Mauro Carvalho Chehab [this message]
2011-05-24 14:15 ` Mauro Carvalho Chehab
2011-05-24 14:57 ` Devin Heitmueller
2011-05-26 6:53 ` Hans Verkuil
2011-05-28 12:17 ` Mauro Carvalho Chehab
2011-05-28 12:26 ` Hans de Goede
2011-05-28 15:24 ` Hans Verkuil
2011-05-28 16:04 ` Mauro Carvalho Chehab
2011-05-28 16:20 ` Mauro Carvalho Chehab
2011-05-29 1:01 ` [RFCv2] Add a library to retrieve associated media devices - was: " Mauro Carvalho Chehab
2011-05-29 11:19 ` Hans Verkuil
2011-05-29 11:47 ` Andy Walls
2011-05-29 12:58 ` Mauro Carvalho Chehab
2011-05-29 11:54 ` Hans de Goede
2011-05-29 13:08 ` Mauro Carvalho Chehab
2011-05-29 13:30 ` Hans de Goede
2011-05-29 14:55 ` Mauro Carvalho Chehab
2011-05-30 7:14 ` Hans Verkuil
2011-05-30 13:15 ` Mauro Carvalho Chehab
2011-05-29 12:11 ` Mauro Carvalho Chehab
2011-05-29 14:39 ` Mauro Carvalho Chehab
2011-05-30 6:34 ` Hans Verkuil
2011-05-30 11:37 ` Mauro Carvalho Chehab
2011-05-30 6:54 ` Hans Verkuil
2011-05-30 13:03 ` Mauro Carvalho Chehab
2011-05-28 12:00 ` Mauro Carvalho Chehab
-- strict thread matches above, loose matches on Subject: below --
2011-05-24 17:26 Emil Meier
2011-05-24 17:46 ` Devin Heitmueller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4DE1093A.6070003@redhat.com \
--to=mchehab@redhat.com \
--cc=linux-media@vger.kernel.org \
--cc=remi@remlab.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox