HID: we do not randomly make new drivers 'default y'
authorLinus Torvalds <torvalds@linux-foundation.org>
Sat, 27 Oct 2018 18:03:27 +0000 (11:03 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 27 Oct 2018 18:03:27 +0000 (11:03 -0700)
.. even when that "default y" is hidden syntactically as a

default !EXPERT

it's wrong.

The only reason something should be 'default y' is if it used to be
built-in, and it was made configurable, and the 'default y' is just
retaining the status quo.

Altheratively, the hardware for the driver has become _so_ common that
it really makes sense for everybody to build it.  Finally, one possible
reason for 'default y' is because the option is not enabling any new
code at all, but is just enabling other options (the networking people
do this for vendor options, for example, so that you can disable whole
vendors at a time).

Clearly, none of these cases hold for the BigBen Interactive Kids'
gamepad, and HID_BIGBEN_FF should thus most definitely not default
to on for everybody.

Cc: Hanno Zulla <kontakt@hanno.de>
Cc: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/hid/Kconfig

index 18c846477ba2586f7ba6381aea515990d4c915a7..5ed319e3b084d920cddf6adc5a665c5732c8c77c 100644 (file)
@@ -188,7 +188,6 @@ config HID_BIGBEN_FF
        depends on NEW_LEDS
        depends on LEDS_CLASS
        select INPUT_FF_MEMLESS
-       default !EXPERT
        help
          Support for the "Kid-friendly Wired Controller" PS3OFMINIPAD
          gamepad made by BigBen Interactive, originally sold as a PS3