diff mbox series

hid: add info to Kconfig

Message ID 20221120033456.4332-1-albert.zhou.50@gmail.com (mailing list archive)
State New, archived
Delegated to: Jiri Kosina
Headers show
Series hid: add info to Kconfig | expand

Commit Message

Albert Zhou Nov. 20, 2022, 3:34 a.m. UTC
Nowadays many laptops use multitouch for their touchpads. This is now
indicated in the help message for HID_MULTITOUCH in the Kconfig.

Signed-off-by: Albert Zhou <albert.zhou.50@gmail.com>
---
 drivers/hid/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)


base-commit: fe24a97cf2543e8832e7a2124802e5c32aac05aa

Comments

Jeff LaBundy Nov. 20, 2022, 6:48 p.m. UTC | #1
Hi Albert,

On Sun, Nov 20, 2022 at 02:34:56PM +1100, Albert Zhou wrote:
> Nowadays many laptops use multitouch for their touchpads. This is now
> indicated in the help message for HID_MULTITOUCH in the Kconfig.
> 
> Signed-off-by: Albert Zhou <albert.zhou.50@gmail.com>
> ---
>  drivers/hid/Kconfig | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
> index 185a077d59cd..86f65258a912 100644
> --- a/drivers/hid/Kconfig
> +++ b/drivers/hid/Kconfig
> @@ -697,7 +697,8 @@ config HID_MONTEREY
>  config HID_MULTITOUCH
>  	tristate "HID Multitouch panels"
>  	help
> -	  Generic support for HID multitouch panels.
> +	  Generic support for HID multitouch panels. Many newer laptops (like
> +	  the Dell XPS 13 9360) use multitouch for their touchpads.

This is just my opinion, but the intent is to describe what
the feature does, not make claims about what products might
happen to use it as of today.

Furthermore, "newer" is relative. This statement won't have
the same meaning years from now. I think the description is
fine as-is.

>  
>  	  Say Y here if you have one of the following devices:
>  	  - 3M PCT touch screens
> 
> base-commit: fe24a97cf2543e8832e7a2124802e5c32aac05aa
> -- 
> 2.34.1
> 

Kind regards,
Jeff LaBundy
Albert Zhou Nov. 21, 2022, 5:42 a.m. UTC | #2
On 21/11/22 05:48, Jeff LaBundy wrote:
> This is just my opinion, but the intent is to describe what
> the feature does, not make claims about what products might
> happen to use it as of today.
> 
> Furthermore, "newer" is relative. This statement won't have
> the same meaning years from now. I think the description is
> fine as-is.

Hi Jeff,

Currently the long list of supported products consists almost
exclusively of touchscreen devices, giving the false impression it's not
relevant for laptop touchpads.

Actually I turned this off because I thought I wouldn't need it, but
later I discovered it's used for my touchpad, which is why I thought it
might be useful to add this to the help message.

 From Albert Zhou
diff mbox series

Patch

diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
index 185a077d59cd..86f65258a912 100644
--- a/drivers/hid/Kconfig
+++ b/drivers/hid/Kconfig
@@ -697,7 +697,8 @@  config HID_MONTEREY
 config HID_MULTITOUCH
 	tristate "HID Multitouch panels"
 	help
-	  Generic support for HID multitouch panels.
+	  Generic support for HID multitouch panels. Many newer laptops (like
+	  the Dell XPS 13 9360) use multitouch for their touchpads.
 
 	  Say Y here if you have one of the following devices:
 	  - 3M PCT touch screens