HID: make full-fledged hid-bus drivers properly selectable
authorJiri Kosina <jkosina@suse.cz>
Wed, 3 Feb 2010 15:11:12 +0000 (16:11 +0100)
committerJiri Kosina <jkosina@suse.cz>
Wed, 3 Feb 2010 15:23:45 +0000 (16:23 +0100)
For historical reasons, we don't have most of the in-tree
drivers residing on hid-bus properly selectable in kernel
configuration unless CONFIG_EMBEDDED is set.

This has been introduced on Linus' request from 14 Oct

===
As to the Kconfig options - do they really add so much space that you need to
ask for the quirks? You didn't use to. Can you make the questions depend on
EMBEDDED, or at least on the HID_COMPAT thing or whatever?
===

This still makes perfect sense for small and tiny drivers, which
just fix report descriptors, fix up HID->input mappings that slightly
violates HUT standard, send one extra packet to the device that is
needed before it becomes functional, etc.

Since then, we have been gathering more and more HID-bus drivers,
which are full-fledged drivers. For these, the size argument becomes
more valid. Plus the devices are much more special than "just violates
HID specification in this one or two tiny unimportant points".

Therefore I am marking such drivers as properly selectable no matter
the setting of CONFIG_EMBEDDED, while keeping all the small and tiny
ones compiled by default.

Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/Kconfig

index 139668d6382a0ca779ee49c416dc0afb3c8e9ce3..617ed7db67d1e24b39ccceb413f3657005b6f762 100644 (file)
@@ -212,9 +212,8 @@ config HID_MONTEREY
        Support for Monterey Genius KB29E.
 
 config HID_NTRIG
-       tristate "NTrig" if EMBEDDED
+       tristate "NTrig"
        depends on USB_HID
-       default !EMBEDDED
        ---help---
        Support for N-Trig touch screen.
 
@@ -338,9 +337,8 @@ config THRUSTMASTER_FF
          Rumble Force or Force Feedback Wheel.
 
 config HID_WACOM
-       tristate "Wacom Bluetooth devices support" if EMBEDDED
+       tristate "Wacom Bluetooth devices support"
        depends on BT_HIDP
-       default !EMBEDDED
        ---help---
        Support for Wacom Graphire Bluetooth tablet.