Re: [PATCH] platform/x86: thinkpad_acpi: suppress warning about palm detection

[Henrique de Moraes Holschuh]

On Fri, 12 Jan 2018, David Herrmann wrote:
> This patch prevents the thinkpad_acpi driver from warning about 2 event
> codes returned for keyboard palm-detection. No behavioral changes,
> other than suppressing the warning in the kernel log. The events are
> still forwarded via acpi-netlink channels.
> 
> We could, optionally, decide to forward the event through a
> input-switch on the tpacpi input device. However, so far no suitable
> input-code exists, and no similar drivers report such events. Hence,
> leave it an acpi event for now.
> 
> Note that the event-codes are named based on empirical studies. On the
> ThinkPad X1 5th Gen the sensor can be found underneath the arrow key.
> 
> Cc: Matthew Thode <mthode@xxxxxxxxxx>
> Signed-off-by: David Herrmann <dh.herrmann@xxxxxxxxx>

Well, we certainly should drop the warning, and I am fine with the
current name for the events (they are not ABI in any way, so we can
change that at any time if we find out it should be named differently).

Acked-by: Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>

> ---
>  drivers/platform/x86/thinkpad_acpi.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
> index 117be48ff4de..128f91af716e 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -214,6 +214,10 @@ enum tpacpi_hkey_event_t {
>  	/* AC-related events */
>  	TP_HKEY_EV_AC_CHANGED		= 0x6040, /* AC status changed */
>  
> +	/* Further user-interface events */
> +	TP_HKEY_EV_PALM_DETECTED	= 0x60b0, /* palm hoveres keyboard */
> +	TP_HKEY_EV_PALM_UNDETECTED	= 0x60b1, /* palm removed */
> +
>  	/* Misc */
>  	TP_HKEY_EV_RFKILL_CHANGED	= 0x7000, /* rfkill switch changed */
>  };
> @@ -4079,6 +4083,12 @@ static bool hotkey_notify_6xxx(const u32 hkey,
>  		*send_acpi_ev = false;
>  		break;
>  
> +	case TP_HKEY_EV_PALM_DETECTED:
> +	case TP_HKEY_EV_PALM_UNDETECTED:
> +		/* palm detected hovering the keyboard, forward to user-space
> +		 * via netlink for consumption */
> +		return true;
> +
>  	default:
>  		pr_warn("unknown possible thermal alarm or keyboard event received\n");
>  		known = false;

-- 
  Henrique Holschuh