| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <IA1PR11MB6098B58B7944838A46F81AF2F4ED2@IA1PR11MB6098.namprd11.prod.outlook.com>
Date: Sun, 26 Jan 2025 01:10:09 +0000
From: "Xu, Even" <even.xu@...el.com>
To: Arnd Bergmann <arnd@...nel.org>, Jiri Kosina <jikos@...nel.org>, "Benjamin
Tissoires" <bentiss@...nel.org>, Srinivas Pandruvada
<srinivas.pandruvada@...ux.intel.com>, Mark Pearson
<mpearson-lenovo@...ebb.ca>
CC: Arnd Bergmann <arnd@...db.de>, Basavaraj Natikar
<basavaraj.natikar@....com>, "Sun, Xinpeng" <xinpeng.sun@...el.com>,
Maximilian Luz <luzmaximilian@...il.com>, Marcel Holtmann
<marcel@...tmann.org>, Johan Hedberg <johan.hedberg@...il.com>, "Luiz Augusto
von Dentz" <luiz.dentz@...il.com>, Dmitry Torokhov
<dmitry.torokhov@...il.com>, "linux-input@...r.kernel.org"
<linux-input@...r.kernel.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, "platform-driver-x86@...r.kernel.org"
<platform-driver-x86@...r.kernel.org>, "linux-usb@...r.kernel.org"
<linux-usb@...r.kernel.org>, "linux-bluetooth@...r.kernel.org"
<linux-bluetooth@...r.kernel.org>
Subject: RE: [PATCH] hid: intel-thc: fix CONFIG_HID dependency
> -----Original Message-----
> From: Arnd Bergmann <arnd@...nel.org>
> Sent: Thursday, January 23, 2025 9:48 PM
> To: Jiri Kosina <jikos@...nel.org>; Benjamin Tissoires <bentiss@...nel.org>;
> Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>; Mark Pearson
> <mpearson-lenovo@...ebb.ca>
> Cc: Arnd Bergmann <arnd@...db.de>; Basavaraj Natikar
> <basavaraj.natikar@....com>; Xu, Even <even.xu@...el.com>; Sun, Xinpeng
> <xinpeng.sun@...el.com>; Maximilian Luz <luzmaximilian@...il.com>; Marcel
> Holtmann <marcel@...tmann.org>; Johan Hedberg
> <johan.hedberg@...il.com>; Luiz Augusto von Dentz <luiz.dentz@...il.com>;
> Dmitry Torokhov <dmitry.torokhov@...il.com>; linux-input@...r.kernel.org;
> linux-kernel@...r.kernel.org; platform-driver-x86@...r.kernel.org; linux-
> usb@...r.kernel.org; linux-bluetooth@...r.kernel.org
> Subject: [PATCH] hid: intel-thc: fix CONFIG_HID dependency
>
> From: Arnd Bergmann <arnd@...db.de>
>
> In drivers/hid/, most drivers depend on CONFIG_HID, while a couple of the drivers
> in subdirectories instead depend on CONFIG_HID_SUPPORT and use 'select HID'.
> With the newly added INTEL_THC_HID, this causes a build warning for a circular
> dependency:
>
> WARNING: unmet direct dependencies detected for HID
> Depends on [m]: HID_SUPPORT [=y] && INPUT [=m]
> Selected by [y]:
> - INTEL_THC_HID [=y] && HID_SUPPORT [=y] && X86_64 [=y] && PCI [=y] &&
> ACPI [=y]
>
> WARNING: unmet direct dependencies detected for INPUT_FF_MEMLESS
> Depends on [m]: INPUT [=m]
> Selected by [y]:
> - HID_MICROSOFT [=y] && HID_SUPPORT [=y] && HID [=y]
> - GREENASIA_FF [=y] && HID_SUPPORT [=y] && HID [=y] && HID_GREENASIA
> [=y]
> - HID_WIIMOTE [=y] && HID_SUPPORT [=y] && HID [=y] && LEDS_CLASS [=y]
> - ZEROPLUS_FF [=y] && HID_SUPPORT [=y] && HID [=y] && HID_ZEROPLUS
> [=y]
> Selected by [m]:
> - HID_ACRUX_FF [=y] && HID_SUPPORT [=y] && HID [=y] && HID_ACRUX [=m]
> - HID_EMS_FF [=m] && HID_SUPPORT [=y] && HID [=y]
> - HID_GOOGLE_STADIA_FF [=m] && HID_SUPPORT [=y] && HID [=y]
> - PANTHERLORD_FF [=y] && HID_SUPPORT [=y] && HID [=y] &&
> HID_PANTHERLORD [=m]
>
> It's better to be consistent and always use 'depends on HID' for HID drivers. The
> notable exception here is USB_KBD/USB_MOUSE, which are alternative
> implementations that do not depend on the HID subsystem.
>
> Do this by extending the "if HID" section below, which means that a few of the
> duplicate "depends on HID" and "depends on INPUT" statements can be removed
> in the process.
>
> Fixes: 1b2d05384c29 ("HID: intel-thc-hid: Add basic THC driver skeleton")
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> ---
> drivers/hid/Kconfig | 10 ++++++----
> drivers/hid/amd-sfh-hid/Kconfig | 1 -
> drivers/hid/i2c-hid/Kconfig | 2 +-
> drivers/hid/intel-ish-hid/Kconfig | 1 - drivers/hid/intel-thc-hid/Kconfig | 1 -
> drivers/hid/surface-hid/Kconfig | 2 --
> drivers/hid/usbhid/Kconfig | 3 +--
> net/bluetooth/hidp/Kconfig | 3 +--
> 8 files changed, 9 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index
> 8adb745c5b28..ed657ef7281c 100644
> --- a/drivers/hid/Kconfig
> +++ b/drivers/hid/Kconfig
> @@ -1376,10 +1376,6 @@ endmenu
>
> source "drivers/hid/bpf/Kconfig"
>
> -endif # HID
> -
> -source "drivers/hid/usbhid/Kconfig"
> -
> source "drivers/hid/i2c-hid/Kconfig"
>
> source "drivers/hid/intel-ish-hid/Kconfig"
> @@ -1390,4 +1386,10 @@ source "drivers/hid/surface-hid/Kconfig"
>
> source "drivers/hid/intel-thc-hid/Kconfig"
>
> +endif # HID
> +
> +# USB support may be used with HID disabled
> +
> +source "drivers/hid/usbhid/Kconfig"
> +
> endif # HID_SUPPORT
> diff --git a/drivers/hid/amd-sfh-hid/Kconfig b/drivers/hid/amd-sfh-hid/Kconfig
> index 329de5e12c1a..3291786a5ee6 100644
> --- a/drivers/hid/amd-sfh-hid/Kconfig
> +++ b/drivers/hid/amd-sfh-hid/Kconfig
> @@ -5,7 +5,6 @@ menu "AMD SFH HID Support"
>
> config AMD_SFH_HID
> tristate "AMD Sensor Fusion Hub"
> - depends on HID
> depends on X86
> help
> If you say yes to this option, support will be included for the diff --git
> a/drivers/hid/i2c-hid/Kconfig b/drivers/hid/i2c-hid/Kconfig index
> ef7c595c9403..e8d51f410cc1 100644
> --- a/drivers/hid/i2c-hid/Kconfig
> +++ b/drivers/hid/i2c-hid/Kconfig
> @@ -2,7 +2,7 @@
> menuconfig I2C_HID
> tristate "I2C HID support"
> default y
> - depends on I2C && INPUT && HID
> + depends on I2C
>
> if I2C_HID
>
> diff --git a/drivers/hid/intel-ish-hid/Kconfig b/drivers/hid/intel-ish-hid/Kconfig
> index 253dc10d35ef..568c8688784e 100644
> --- a/drivers/hid/intel-ish-hid/Kconfig
> +++ b/drivers/hid/intel-ish-hid/Kconfig
> @@ -6,7 +6,6 @@ config INTEL_ISH_HID
> tristate "Intel Integrated Sensor Hub"
> default n
> depends on X86
> - depends on HID
> help
> The Integrated Sensor Hub (ISH) enables the ability to offload
> sensor polling and algorithm processing to a dedicated low power diff --
> git a/drivers/hid/intel-thc-hid/Kconfig b/drivers/hid/intel-thc-hid/Kconfig
> index 91ec84902db8..0351d1137607 100644
> --- a/drivers/hid/intel-thc-hid/Kconfig
> +++ b/drivers/hid/intel-thc-hid/Kconfig
> @@ -7,7 +7,6 @@ menu "Intel THC HID Support"
> config INTEL_THC_HID
> tristate "Intel Touch Host Controller"
> depends on ACPI
> - select HID
> help
> THC (Touch Host Controller) is the name of the IP block in PCH that
> interfaces with Touch Devices (ex: touchscreen, touchpad etc.). It diff --
> git a/drivers/hid/surface-hid/Kconfig b/drivers/hid/surface-hid/Kconfig index
> 7ce9b5d641eb..d0cfd0d29926 100644
> --- a/drivers/hid/surface-hid/Kconfig
> +++ b/drivers/hid/surface-hid/Kconfig
> @@ -1,7 +1,6 @@
> # SPDX-License-Identifier: GPL-2.0+
> menu "Surface System Aggregator Module HID support"
> depends on SURFACE_AGGREGATOR
> - depends on INPUT
>
> config SURFACE_HID
> tristate "HID transport driver for Surface System Aggregator Module"
> @@ -39,4 +38,3 @@ endmenu
>
> config SURFACE_HID_CORE
> tristate
> - select HID
> diff --git a/drivers/hid/usbhid/Kconfig b/drivers/hid/usbhid/Kconfig index
> 7c2032f7f44d..f3194767a45e 100644
> --- a/drivers/hid/usbhid/Kconfig
> +++ b/drivers/hid/usbhid/Kconfig
> @@ -5,8 +5,7 @@ menu "USB HID support"
> config USB_HID
> tristate "USB HID transport layer"
> default y
> - depends on USB && INPUT
> - select HID
> + depends on HID
> help
> Say Y here if you want to connect USB keyboards,
> mice, joysticks, graphic tablets, or any other HID based devices diff --git
> a/net/bluetooth/hidp/Kconfig b/net/bluetooth/hidp/Kconfig index
> 6746be07e222..e08aae35351a 100644
> --- a/net/bluetooth/hidp/Kconfig
> +++ b/net/bluetooth/hidp/Kconfig
> @@ -1,8 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only config BT_HIDP
> tristate "HIDP protocol support"
> - depends on BT_BREDR && INPUT && HID_SUPPORT
> - select HID
> + depends on BT_BREDR && HID
> help
> HIDP (Human Interface Device Protocol) is a transport layer
> for HID reports. HIDP is required for the Bluetooth Human
> --
> 2.39.5
For hid/intel-thc-hid:
Reviewed-by: Even Xu <even.xu@...el.com>
Best Regards,
Even Xu
Powered by blists - more mailing lists