On Fri, Jun 17, 2011 at 14:51, Takashi Iwai <tiwai@xxxxxxx> wrote: > At Fri, 17 Jun 2011 14:45:52 +0200, > Lennart Poettering wrote: >> >> On Fri, 17.06.11 14:42, Takashi Iwai (tiwai@xxxxxxx) wrote: >> >> > > If you have the patches ready to convert all drivers which currently can >> > > do jack sensing over to the new scheme with control elements, then we >> > > should probably focus on that for the future, shouldn't we? >> > >> > For PA work, I suppose so (although I won't object to someone working >> > on the implementation with the present input-jack layer :) >> > But you can see David's patch basically independent from PA. >> >> Well, but I'd guess that the access permissions only really matter for >> PA, right? > > Not really. For example, I was asked about the input-jack stuff by > a couple of vendors for their own products, apparently running without > PA. I can't imagine they asked for it because they wanted input devces. I guess they just asked for jack-events, didn't they? >> So if we say that proper jack sensing via control elements is >> the way to go, then PA should adopt this new scheme and this new scheme >> only, and not bother with the input device cruft, and the udev patch >> should not be merged either as PA would be its only consumer... > > I would be against the patch if the change were too intrusive, > but this change itself looks small enough to me. > The question whether PA should use it or not is a different thing. The thing is that currently PA doesn't use input jack devices, and honestly I think all the input device misuse must stop. From my point of view jack senses belong entirely to the alsa device domain and not to unrelated fake input devices. So if you can get the control events working in a reasonable time frame, I would really prefer to add support for that to PA, and nothing is needed on udev's side. Kay -- To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
