Re: [PATCH] HID: multitouch: Fix system-control buttons not working

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 22-08-17 10:59, Benjamin Tissoires wrote:
> On Aug 22 2017 or thereabouts, Hans de Goede wrote:
>> Some laptops have system-control buttons (e.g. KEY_SLEEP) on the same
>> interface as a hid-multitouch touch-pad.
>>
>> This commit fixes these buttons not working.
>>
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>> ---
> 
> Acked-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>

Thanks.

Note this applies on top of:

https://git.kernel.org/pub/scm/linux/kernel/git/jikos/hid.git/log/?h=for-4.14/multitouch

(in case that was not clear)

Regards,

Hans


> 
>>   drivers/hid/hid-multitouch.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c
>> index 0a8689bb08e9..53d1413cd435 100644
>> --- a/drivers/hid/hid-multitouch.c
>> +++ b/drivers/hid/hid-multitouch.c
>> @@ -930,6 +930,7 @@ static int mt_input_mapping(struct hid_device *hdev, struct hid_input *hi,
>>   	    field->application != HID_DG_PEN &&
>>   	    field->application != HID_DG_TOUCHPAD &&
>>   	    field->application != HID_GD_KEYBOARD &&
>> +	    field->application != HID_GD_SYSTEM_CONTROL &&
>>   	    field->application != HID_CP_CONSUMER_CONTROL &&
>>   	    field->application != HID_GD_WIRELESS_RADIO_CTLS &&
>>   	    !(field->application == HID_VD_ASUS_CUSTOM_MEDIA_KEYS &&
>> -- 
>> 2.13.4
>>