[RFC, PATCH] Getting rid of direct calls to keyboard's methods via kbdsw[] array

Maksim Yevmenkin maksim.yevmenkin at gmail.com
Sat Dec 22 15:43:36 PST 2007


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


More information about the freebsd-current mailing list