From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lb2-smtp-cloud6.xs4all.net ([194.109.24.28]:35957 "EHLO lb2-smtp-cloud6.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755498AbbBPLIg (ORCPT ); Mon, 16 Feb 2015 06:08:36 -0500 Message-ID: <54E1CFA0.50400@xs4all.nl> Date: Mon, 16 Feb 2015 12:08:16 +0100 From: Hans Verkuil MIME-Version: 1.0 To: Mauro Carvalho Chehab CC: Linux Media Mailing List , Mauro Carvalho Chehab , Laurent Pinchart Subject: Re: [PATCHv4 00/25] dvb core: add basic support for the media controller References: <54DF1625.20808@xs4all.nl> <20150214090019.798b6d18@recife.lan> <54DF34E2.2020709@xs4all.nl> <20150215082716.45165770@recife.lan> <54E1BEA6.8060400@xs4all.nl> <20150216085051.01f01e19@recife.lan> In-Reply-To: <20150216085051.01f01e19@recife.lan> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 02/16/2015 11:50 AM, Mauro Carvalho Chehab wrote: > Em Mon, 16 Feb 2015 10:55:50 +0100 > Hans Verkuil escreveu: > >> On 02/15/2015 11:27 AM, Mauro Carvalho Chehab wrote: > >>> In any case, for ALSA, we should do the right thing here: remove (actually >>> deprecate) whatever definition is there, and then re-add it only when we >>> actually have the patches inside the ALSA subsystem to support the media >>> controller, plus having the corresponding patches for the media-ctl in order >>> to support the devnode discovery using both udev and sysfs for their nodes. >> >> I actually thought about how alsa should be handled and it is doing the >> right thing. See my patch that I posted today, partially reverting your >> patch. > > Well, I can live with that patch for now, but I suspect that removing > major/minor from ALSA will make things very complex at the userspace side. > > Do you know how to convert from card/device/subdevice into a device node > patch using libudev and sysfs? You don't *want* a device. The user space alsa utilities all operate on card/device/subdevice level. I've never used an alsa device node directly, or seen anyone do that. That's all done by the alsa library. But regardless of whether or not it is right or wrong, as long as we do not actually implement this in a driver I would keep this and wait until we do have a working example. Replacing the alsa struct by a simple major/minor is certainly not enough, of that I am 100% certain. Let's keep it as is, and only touch it again when we actually get the first user. Regards, Hans