From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH] input: joystick: make USB drivers depend on USB Date: Wed, 17 Jan 2018 13:53:16 -0800 Message-ID: <20180117215316.7qe5o2pspb3dqgny@dtor-ws> References: <20180117213010.810-1-marcus.folkesson@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pf0-f194.google.com ([209.85.192.194]:38806 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752746AbeAQVxT (ORCPT ); Wed, 17 Jan 2018 16:53:19 -0500 Content-Disposition: inline In-Reply-To: <20180117213010.810-1-marcus.folkesson@gmail.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Marcus Folkesson Cc: Tomohiro Yoshidomi , David Herrmann , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org On Wed, Jan 17, 2018 at 10:30:10PM +0100, Marcus Folkesson wrote: > A driver should not enable an entire subsystem. I disagree. As you go through menuconfig and you encounter this option and you have the hardware and you want to enable it, you should be able to do so. Otherwise you enable bunch of functionality, then go back, see what new options appeared, enable them, go back, see if any more new options appeared, and so on. What exactly prompted this change? > > Signed-off-by: Marcus Folkesson > --- > drivers/input/joystick/Kconfig | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/input/joystick/Kconfig b/drivers/input/joystick/Kconfig > index 332c0cc1b2ab..4a199cff8c68 100644 > --- a/drivers/input/joystick/Kconfig > +++ b/drivers/input/joystick/Kconfig > @@ -279,8 +279,7 @@ config JOYSTICK_JOYDUMP > > config JOYSTICK_XPAD > tristate "X-Box gamepad support" > - depends on USB_ARCH_HAS_HCD > - select USB > + depends on USB > help > Say Y here if you want to use the X-Box pad with your computer. > Make sure to say Y to "Joystick support" (CONFIG_INPUT_JOYDEV) > -- > 2.15.1 > -- Dmitry