Hello,
> In the FreeBSD kernel we have notion of keyboard represented by
> "keyboard_t" type. We operate on such structures throught another
> keyboard_switch structure. It lets us to use keyboard API without
> distinguishing exactly which hardware we're working with.
>
> My proposal is to unify a way in which we call such specialized
> methods, so that instead of direct array indexing used to call specific
> keyboard method:
>
> (*kbdsw[(kbd)->kb_index]->enable)((kbd))
>
> We would use unified interface:
>
> kbdd_enable(kbd)
>
> Proposed patch is here:
>
> http://people.freebsd.org/~wkoszek/patches/tty/kbdcleanup.patch
>
> Suggestions are welcome.
looks fine to me. it also would be nice to clean up and unify (where
possible) read_char method for all the keyboards.
thanks,
max