Fwd: Re: xorg.conf.d - InputClass feature request

[Martin Pitt]

Hello all,

happy new year!

Alberto Milone [2010-01-04 10:14 +0100]:
> I'm forwarding this email to Martin Pitt who works on udev.

Peter Hutterer:
> On Sun, Jan 03, 2010 at 04:01:27PM +0100, Alberto Milone wrote:
> > On Sunday 03 Jan 2010 12:42:19 you wrote:
> > > this is too specific to the particular product and its quirks. It would be
> > > better to use a tagging system or something similar. udev assigns a tag
> > > based on whatever condition and that tag may be used by specific xorg.conf
> > > sections. this way, you can tag your device as "need_touchpad_quirk" or
> > > simply "inspiron" or whatever but the actual configuration is in
> > > xorg.conf.d.
> > >  
> > > I think this should fit into the current design and it is not bound to a
> > > specific backend and splits configuration from quirk detection. Plus, how a
> > > tag is assigned doesn't matter to us and what happens with the tags doesn't
> > > matter for the tagger. How does that sound?
> > > 
> > I really like this approach. Let's see if I fully understood your idea:

Me too, it keeps the notion of "DMI" (whose detection and
representation is OS specific) out of X.org, and uses udev rules where
they are good at.

> > ATTR{[dmi/id]product_name}=="Inspiron 1011|Inspiron 1012", \
> >   ENV{TAG}="Inspiron"

> >          MatchTag "Inspiron"
> >          Driver "synaptics"
> >          Option "AreaBottomEdge" 4100
>
> yes, this was the idea, though we might need to think about namespacing the
> tags. some discussion with the udev guys would be beneficial here.

Indeed, just using a "TAG" property is too generic. How about

  ENV{ID_INPUT.xorg.InspironBottomEdge}="1"
  ENV{ID_INPUT.xorg.SlowMotion}="1"

  MatchTag "InspironBottomEdge"

This is assuming that you only want boolean matches and potentially
have more than one quirk. Then MatchTag "foo" would look for a set
ID_INPUT.xorg.foo property.

If you want/can support string matches as well, this could look a tad
nicer:

  ENV{ID_INPUT.quirks}="InspironBottomEdge SlowMotion"

  MatchTag "quirks" "*InspironBottomEdge*"

From an udev POV I'm not fussed about which to take. Here, udev just
provides a property database.

Thanks!

Martin
-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
Url : http://lists.x.org/archives/xorg-devel/attachments/20100104/195bd061/attachment-0001.pgp