Messages in this thread | | | Subject | Re: [PATCH] Input: rework USB Kconfig dependencies | From | Randy Dunlap <> | Date | Thu, 22 Apr 2021 09:32:27 -0700 |
| |
On 4/22/21 6:36 AM, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > A lot of input drivers traditionally depend on CONFIG_USB_ARCH_HAS_HCD > and select CONFIG_USB. This works but is different from almost every > other subsystem in the kernel. > > I found this when debugging a build failure in the RC subsystem that > had the same logic. > > The problem here is that CONFIG_USB_ARCH_HAS_HCD no longer has > a meaning since the host controller support has been changed to > use machine specific loadable modules for the USB host. Selecting > a subsystem that a driver needs is confusing and can lead to > recursive dependency chains in Kconfig. > > In both cases, the normal logic is to specify 'depends on USB'. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.
> --- > drivers/input/joystick/Kconfig | 6 ++---- > drivers/input/misc/Kconfig | 15 +++++---------- > drivers/input/mouse/Kconfig | 9 +++------ > drivers/input/tablet/Kconfig | 15 +++++---------- > drivers/input/touchscreen/Kconfig | 3 +-- > 5 files changed, 16 insertions(+), 32 deletions(-) >
-- ~Randy
| |