[PATCH] evdev: add phys property (EVIOCGPHYS) as stable identifier

Peter Korsgaard jacmet at sunsite.dk
Wed Apr 28 11:08:25 PDT 2010


>>>>> "Dan" == Dan Nicholson <dbn.lists at gmail.com> writes:

Hi,

 >> Which also brings us to the question of whether this should be
 >> implemented in the server instead then? I'm sure the other drivers
 >> could benefit from this as well.

 Dan> You probably could get the information from udev (DEVPATH/DEVNAME)
 Dan> and hal (linux.sysfs_path/input.device), but I'm not sure exactly
 Dan> how to pass that down to the driver to be associated since we
 Dan> throw away the InputAttributes. Currently we stuff things into
 Dan> Options, but that seems wrong. Maybe in some ABI breaking future
 Dan> we could pass along the InputAttributes to the driver so it has
 Dan> all that information on hand.

We do get the device node name and can get the evdev phys using the
EVIOCGPHYS ioctl, and sysfs can be retrieved from those by lookup in
/proc/bus/input/devices, so it's not that bad.

 Dan> That's kind of orthogonal to this issue, though. It seems you could
 Dan> carry the driver specific property for now until a server level
 Dan> solution could be built up.

True.

-- 
Bye, Peter Korsgaard


More information about the xorg-devel mailing list