Messages in this thread | | | Date | Mon, 9 Oct 2000 19:05:59 -0600 | From | Harold Oga <> | Subject | Re: [PATCH] drivers/usb/Config.in |
| |
On Mon, Oct 09, 2000 at 05:19:37PM -0700, Matthew Dharm wrote: >I'm not sure this is correct -- the options that this if conditional >controls are the Keyboard and Mouse _Boot_Protocol_ support, which is >separate form the regular HID support -- I believe it's for motherboards >which can boot from a USB keyboard. > >Can somone on the linux-usb-devel mailinglist comment? (I've CC'ed them) > >Matt > >On Tue, Oct 10, 2000 at 12:14:11AM -0700, James Simmons wrote: >> >> This patch fixes a minor config error for drivers/usb/Config.in. When you >> select USB Human Interface Device (HID) support I assume you should be >> able to select a USB mouse and/or USB keyboard. With the current Config.in >> you can't. >> >> --- Config.in.orig Tue Oct 10 00:10:06 2000 >> +++ Config.in Tue Oct 10 00:10:27 2000 >> @@ -80,7 +80,7 @@ >> comment ' Input core support is needed for USB HID' >> else >> dep_tristate ' USB Human Interface Device (HID) support' CONFIG_USB_HID $CONFIG_USB $CONFIG_INPUT >> - if [ "$CONFIG_USB_HID" != "y" ]; then >> + if [ "$CONFIG_USB_HID" = "y" ]; then >> dep_tristate ' USB HIDBP Keyboard support' CONFIG_USB_KBD $CONFIG_USB $CONFIG_INPUT >> dep_tristate ' USB HIDBP Mouse support' CONFIG_USB_MOUSE $CONFIG_USB $CONFIG_INPUT >> fi Hi, I'm just a humble user, and not a developer, but this patch looks wrong. As far as I remember, the whole point to the HIDBP keyboard and HIDBP mouse drivers was to provide minimal support for USB mice and keyboards, if someone didn't need everything in the full hid driver. If the full hid driver is included in the kernel, there is no point to building in the stripped down drivers as well. The full hid driver supports everything the HIDBP drivers do, plus a whole lot more. The stripped down drivers for example don't support any special keys on the keyboard, or any of the fancy mouse protocols like IMPS/2, or at least, they didn't the last time I looked at them.
-Harold -- "Life sucks, deal with it!" - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
| |