On Dec 27, 2012 12:09 PM, "Tom Gundersen" <teg@jklm.no> wrote:
>
> On Thu, Dec 27, 2012 at 2:44 AM, Dave Reisner <d@falconindy.com> wrote:
> > Does atkbd have modaliases? There seems to be aliases for stuff in
> > drivers/input/serio/* and other keyboard drivers in
> > drivers/input/keyboard/*, but I see no evidence that atkbd can be
> > autoloaded.
>
> I haven't actually tried it (as I don't have the controller), however,
> it looks like it should work to me:
>
> % modinfo atkbd
> filename:
> /lib/modules/3.8.0-rc1-TEG/kernel/drivers/input/keyboard/atkbd.ko
> license:        GPL
> description:    AT and PS/2 keyboard driver
> author:         Vojtech Pavlik <vojtech@suse.cz>
> srcversion:     88F58DCA130FB37430C00C5
> alias:          serio:ty02pr22id*ex*
> alias:          serio:ty06pr*id*ex*
> alias:          serio:ty01pr*id*ex*

Good enough for me. Thanks for checking.

> > 'input' seems like too generic a name, since we aren't adding drivers
> > for things like analog joysticks or mice, but at the same time, we're
> > adding a bunch of usb HID stuff that might be used for just that. I'm
> > okay with the the name 'input', as I don't have any better ideas.
>
> We could call it 'keyboard' as whatever joystick/mouse/... drivers are
> not used anyway in the initramfs, and we don't include any of the
> non-hid, non-keyboard modules.
>
> >> +help() {
> >> +    cat <<HELPEOF
> >> +This hook loads the necessary modules for input devices.  Detection
> >
> > Maybe change the verbiage here to note that this hook is aimed at
> > providing drivers for a keyboard (as we generally take anything else
> > added as a side effect).
>
> I'll rephrase and rename to 'keyboard', you can rename back to 'input'
> if you prefer.

Keyboard sounds fine.

>
> -t