[PATCH x11proto] Add XF86XK_TouchpadOn/Off

[Peter Hutterer]

On Mon, Nov 08, 2010 at 09:17:23PM -0600, Chris Bagwell wrote:
> On Mon, Nov 8, 2010 at 4:37 PM, Bastien Nocera <hadess at hadess.net> wrote:
> > On Mon, 2010-11-08 at 16:32 -0600, Chris Bagwell wrote:
> >> On Sun, Nov 7, 2010 at 11:24 PM, Peter Hutterer
> >> <peter.hutterer at who-t.net> wrote:
> >> > Patch is fine with me. any NAKs?
> >> >
> >> > The udev patch to standardise the behaviour is already in, see
> >> > http://git.kernel.org/?p=linux/hotplug/udev.git;a=commit;h=a1ca5f60e0770299c5c5f21bd371f5823802412b
> >> > It requires an update to xkeyboard-config as well, synchronised with udev to
> >> > maintain consistency. That's possible once this change is in.
> >> >
> >>
> >> And just to stress, at this point we need a commit made somewhere.
> >> Might as well make this one.
> >>
> >> udev submission above has changed existing F22 values to F21 so at
> >> minimum to align with them we need to update below value from
> >> xkeyboard-config's inet + synchronized udev release to prevent
> >> breakage of existing feature.
> >
> > Except that it never actually worked properly, so you wouldn't be
> > breaking that much.

hey, it worked on my lenovo :)

> 
> Yep.  I'm not trying to stress the broken part but the mismatch part.
> 
> I was motivated by your work and I'm working to get eee pc's working.
> In their ACPI driver (eeepc-laptop or eeepc-wmi) they map a hotkey
> meant for touchpad toggle to F13 (mapped before udev).  Its been this
> way for quite a while and so not to useful out of the box.
> 
> I want to get that aligned to F21 or F22 or what ever people agree to
> so it can benefit from the nice work you guys did in this area.
> 
> It makes sense for me to wait until both udev and xkeyboard-config
> match before proceeding though.

This is mostly juggling by distributions anyway, since they need to update
kernel, udev, xkeyboard-config and x11proto at the same time. We don't have
to perfectly synchronise it upstream, the number of people running these
four from latest git is reasonably small :)
 
Cheers,
  Peter