On Tue, Apr 07, 2015 at 02:06:33PM +0200, David Herrmann wrote: > Hi > > On Tue, Apr 7, 2015 at 1:52 PM, Hans de Goede <hdego...@redhat.com> wrote: > > Hi, > > > > > > On 07-04-15 13:46, David Herrmann wrote: > >> > >> Hi > >> > >> On Tue, Apr 7, 2015 at 1:40 PM, Hans de Goede <hdego...@redhat.com> wrote: > >>>> > >>>> The series looks reasonable, but I wonder why we cannot merge it into > >>>> 60-keyboard.rule (60-evdev.rule now) like Peter's series does? In that > >>>> case, please make this a builtin of input_id which uses > >>>> udev_device_get_parent_subsystem_devtype() and look for the > >>>> serio/atkbd (?) parent device, instead of hard-coding the path. > >>> > >>> > >>> > >>> Why would we want to merge this into keyboard rules/hwdb? Trackpoints > >>> always have a separate driver, input device and event node. > >> > >> > >> Right now, we do hwdb lookups for keyboard:foobar, evdev:foobar and > >> now trackpoint:foobar, instead of a single lookup for evdev:foobar. > >> > >> To me, it looks much easier if we have one generic evdev rule, which > >> searches the hwdb based on the input-modalias via > >> evdev:input:<modalias> and the legacy fallback, > >> evdev:name:<name>:<dmi> > >> > >> Depending on what properties where found, we simply apply them to the > >> evdev node. > > > > > > Ah, I see so what you're suggesting is in essence dropping the > > > > ENV{ID_INPUT_KEY}=="", GOTO="keyboard_end" > > > > Rule from rules/60-keyboard.rules > > > > And s/keyboard/evdev/ as one of Peter's updated patches is already doing, > > while keeping the hwdb bits the same, except that the prefix for the > > matches would be evdev instead of pointingstick ? > > > > And teaching the keyboard^Wevdev builtin to handle > > POINTINGSTICK_SENSITIVITY and write the sysfs attribute ? > > Exactly. Peter already did most of this in his series, so we might > wanna wait for him to push it. > I'm actually only waiting for Kay to ACK the prefix 'evdev', then I'd > push it myself and we can rebase your series.
just in case, the current tree is here: https://github.com/whot/systemd/tree/wip/abs-override Once I have Kay's ack I'll push it unless you beat me to it. Cheers, Peter _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel