On Fri, Mar 28, 2014 at 09:12:43AM +0100, Hans de Goede wrote:

Hi,

On 03/28/2014 08:56 AM, Dmitry Torokhov wrote:

quoted

On Thu, Mar 20, 2014 at 05:21:59PM +0000, Matthew Garrett wrote:

quoted

On Thu, Mar 20, 2014 at 11:12:08AM +0100, Hans de Goede wrote:

quoted

Which in the end turns out to be much nicer too, since it gets rid of needing
a udev-helper too. After this much too long introduction I'll let the patches
speak for themselves.

Yeah, I was coming to the conclusion that this was probably the best we 
could do. It's unfortunate that "id" is already in use - we'd be able to 
get away without any X server modifications otherwise.

Long term we probably still want to tie serio devices to the ACPI 
devices in case the vendor provides power management calls there, but we 
can leave that until there's an actual example.

I am still unsure if we shoudl be adding these new IDs to serio core...
Can't the X driver take a peek at ACPI devices on it's own?

The problem is there is no way for userspace to know which /sys/devices/pnp0/00:xx
device is the serio bus host.

Practically speaking you should not care - there is only one touchpad in
Lenovos.

Thanks.

-- 
Dmitry