Re: [PATCH] Add support for USB Keyboard attached to UHCI

Aaron Young <[email protected]> Fri, 11 Jan 2008 09:16:31 -0800 (PST)
Newsgroups gmane.linux.kernel.debugging
Message-ID <[email protected]>
> 
> Aaron Young wrote:
> 
> Hi Aaron and Konstantin,
> 
> > 
> > 
> > 
> >   I guess I'm good with it. Couple of nits:
> > 
> > 1. The routine kdb_uhci_keyboard_urb() is really not uhci specific
> >    and so could be renamed to simply kdb_keyboard_urb(). Or
> 
> We now have three copies of kdb_usb_kdbs[], kdb_usb_keycode[], and
> kdb_uhci_keyboard_urb().
> 
> We need separate kdb_uhci_keyboard_urb() because we have  three arch-
> dependent kdb_usb_kbds[]. My brief examination seems to indicate
> they are the same. Do we need to have arch-depedent kdb_usb_kbds[]?
> Also, do we need three copies of kdb_usb_keycode[]?

  I see no reason off-hand why we would require arch-independent version
  of these routines/structs.

---------------------------
Use http://oss.sgi.com/ecartis to modify your settings or to unsubscribe.