From patchwork Mon Jan 14 15:01:26 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: William Whistler X-Patchwork-Id: 10762905 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 5A0E2139A for ; Mon, 14 Jan 2019 15:07:36 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4908D28FDF for ; Mon, 14 Jan 2019 15:07:36 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3C87829011; Mon, 14 Jan 2019 15:07:36 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id F1D4328FDF for ; Mon, 14 Jan 2019 15:07:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726591AbfANPHe convert rfc822-to-8bit (ORCPT ); Mon, 14 Jan 2019 10:07:34 -0500 Received: from rs159.luxsci.com ([64.49.224.181]:50332 "EHLO rs159.luxsci.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726579AbfANPHe (ORCPT ); Mon, 14 Jan 2019 10:07:34 -0500 X-Greylist: delayed 359 seconds by postgrey-1.27 at vger.kernel.org; Mon, 14 Jan 2019 10:07:32 EST Received: from rs159.luxsci.com (localhost.localdomain [127.0.0.1]) by rs159.luxsci.com (8.14.4/8.14.9) with ESMTP id x0EF1X5l062848 for ; Mon, 14 Jan 2019 09:01:33 -0600 Received: (from root@localhost) by rs159.luxsci.com (8.14.4/8.14.9/Submit) id x0EF1WOi062806 for linux-input@vger.kernel.org; Mon, 14 Jan 2019 15:01:32 GMT Received: (from sender 49604) (rs159.luxsci.com [127.0.0.1]) by LuxSci SP; Mon, 14 Jan 2019 15:01:31 +0000 Date: Mon, 14 Jan 2019 15:01:26 +0000 From: William Whistler To: "open list:HID CORE LAYER" Cc: Jiri Kosina , Benjamin Tissoires Subject: [PATCH v2] Support for Maltron L90 keyboard media keys In-Reply-To: References: <20180918123825.215c9810@overkill> <1547404895-2384718.53650169.fx0DIfU14007744@rs159.luxsci.com> X-Lux-Comment: Message x0EF1Rkn062652 sent by user #49604 Message-Id: <1547478091-4815931.98030297.fx0EF1Rkn062652@rs159.luxsci.com> MIME-Version: 1.0 X-Comment: LuxSci SP Message ID - 1547478091-4815931.98030297 Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Mon, 14 Jan 2019 13:35:02 +0100 Benjamin Tissoires wrote: > I guess the driver also supports the other keys on the keyboard. Maybe > you should tell that this driver adds those various keys (well, there > are a lot of example in this file, I am sure you can make something > correct). I have altered both the Kconfig description and the patch title to clarify that this patch merely adds support for the media keys. Without it, the other keys on the keyboard already function normally. > That's a lot of copyright for a small driver that just does a report > fixup. In order to try to conform to the kernel code style, I based my code on two existing drivers. Out of an abundance of caution, I used the union of the copyright lines from them both (and added mine). > You should use the SPDX tag now, and drop the license text. > > No C++ comments. Please use /* */ even for one liners (so in the > report descriptors too). I've fixed the comments in the body of the file as requested. However, the SPDX tag style guide[1] says to use the // style for .c source files. I have followed that guideline despite the apparent conflict with your second statement here. [1] https://www.kernel.org/doc/html/v4.20/process/license-rules.html > Hint, you can run the script on your git tip by running: > `./scripts/checkpatch.pl -g HEAD`. Thank you for the useful tip. > Please drop this hunk. It shouldn't be required since v4.17 Indeed, I have tested without and it still works fine. > Thanks for the submission. This is a small driver that shouldn't take > too many revisions to apply. Thank you for the guidance! I have appended the updated patch. diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index 41e9935fc584..37f82ef27b11 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -590,6 +590,13 @@ config HID_MAGICMOUSE Say Y here if you want support for the multi-touch features of the Apple Wireless "Magic" Mouse and the Apple Wireless "Magic" Trackpad. +config HID_MALTRON + tristate "Maltron L90 keyboard" + depends on HID + help + Adds support for the volume up, volume down, mute, and play/pause buttons + of the Maltron L90 keyboard. + config HID_MAYFLASH tristate "Mayflash game controller adapter force feedback" depends on HID diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile index 896a51ce7ce0..cf2752003253 100644 --- a/drivers/hid/Makefile +++ b/drivers/hid/Makefile @@ -66,6 +66,7 @@ obj-$(CONFIG_HID_LOGITECH) += hid-logitech.o obj-$(CONFIG_HID_LOGITECH_DJ) += hid-logitech-dj.o obj-$(CONFIG_HID_LOGITECH_HIDPP) += hid-logitech-hidpp.o obj-$(CONFIG_HID_MAGICMOUSE) += hid-magicmouse.o +obj-$(CONFIG_HID_MALTRON) += hid-maltron.o obj-$(CONFIG_HID_MAYFLASH) += hid-mf.o obj-$(CONFIG_HID_MICROSOFT) += hid-microsoft.o obj-$(CONFIG_HID_MONTEREY) += hid-monterey.o diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 518fa76414f5..01d565357dbe 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid/hid-ids.h @@ -72,6 +72,7 @@ #define USB_VENDOR_ID_ALCOR 0x058f #define USB_DEVICE_ID_ALCOR_USBRS232 0x9720 +#define USB_DEVICE_ID_ALCOR_MALTRON_KB 0x9410 #define USB_VENDOR_ID_ALPS 0x0433 #define USB_DEVICE_ID_IBM_GAMEPAD 0x1101 diff --git a/drivers/hid/hid-maltron.c b/drivers/hid/hid-maltron.c new file mode 100644 index 000000000000..c37790d95967 --- /dev/null +++ b/drivers/hid/hid-maltron.c @@ -0,0 +1,166 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * HID driver for Maltron L90 + * + * Copyright (c) 1999 Andreas Gal + * Copyright (c) 2000-2005 Vojtech Pavlik + * Copyright (c) 2005 Michael Haboustak for Concept2, Inc + * Copyright (c) 2008 Jiri Slaby + * Copyright (c) 2012 David Dillow + * Copyright (c) 2006-2013 Jiri Kosina + * Copyright (c) 2013 Colin Leitner + * Copyright (c) 2014-2016 Frank Praznik + * Copyright (c) 2010 Richard Nauber + * Copyright (c) 2016 Yuxuan Shui + * Copyright (c) 2018 William Whistler + */ + +#include +#include +#include + +#include "hid-ids.h" + +/* The original buggy USB descriptor */ +static u8 maltron_rdesc_o[] = { +0x05, 0x01, /* Usage Page (Generic Desktop Ctrls) */ +0x09, 0x80, /* Usage (Sys Control) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x02, /* Report ID (2) */ +0x75, 0x01, /* Report Size (1) */ +0x95, 0x01, /* Report Count (1) */ +0x15, 0x00, /* Logical Minimum (0) */ +0x25, 0x01, /* Logical Maximum (1) */ +0x09, 0x82, /* Usage (Sys Sleep) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x09, 0x82, /* Usage (Sys Sleep) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x09, 0x83, /* Usage (Sys Wake Up) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x75, 0x05, /* Report Size (5) */ +0x81, 0x01, /* Input (Const,Array,Abs) */ +0xC0, /* End Collection */ +0x05, 0x0C, /* Usage Page (Consumer) */ +0x09, 0x01, /* Usage (Consumer Control) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x03, /* Report ID (3) */ +0x95, 0x01, /* Report Count (1) */ +0x75, 0x10, /* Report Size (16) */ +0x19, 0x00, /* Usage Minimum (Unassigned) */ +0x2A, 0xFF, 0x7F, /* Usage Maximum (0x7FFF) */ +0x81, 0x00, /* Input (Data,Array,Abs) */ +0xC0, /* End Collection */ +0x06, 0x7F, 0xFF, /* Usage Page (Vendor Defined 0xFF7F) */ +0x09, 0x01, /* Usage (0x01) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x04, /* Report ID (4) */ +0x95, 0x01, /* Report Count (1) */ +0x75, 0x10, /* Report Size (16) */ +0x19, 0x00, /* Usage Minimum (0x00) */ +0x2A, 0xFF, 0x7F, /* Usage Maximum (0x7FFF) */ +0x81, 0x00, /* Input (Data,Array,Abs) */ +0x75, 0x02, /* Report Size (2) */ +0x25, 0x02, /* Logical Maximum (2) */ +0x09, 0x90, /* Usage (0x90) */ +0xB1, 0x02, /* Feature (Data,Var,Abs) */ +0x75, 0x06, /* Report Size (6) */ +0xB1, 0x01, /* Feature (Const,Array,Abs) */ +0x75, 0x01, /* Report Size (1) */ +0x25, 0x01, /* Logical Maximum (1) */ +0x05, 0x08, /* Usage Page (LEDs) */ +0x09, 0x2A, /* Usage (On-Line) */ +0x91, 0x02, /* Output (Data,Var,Abs) */ +0x09, 0x4B, /* Usage (Generic Indicator) */ +0x91, 0x02, /* Output (Data,Var,Abs) */ +0x75, 0x06, /* Report Size (6) */ +0x95, 0x01, /* Report Count (1) */ +0x91, 0x01, /* Output (Const,Array,Abs) */ +0xC0 /* End Collection */ +}; + +/* The patched descriptor, allowing media key events to be accepted as valid */ +static u8 maltron_rdesc[] = { +0x05, 0x01, /* Usage Page (Generic Desktop Ctrls) */ +0x09, 0x80, /* Usage (Sys Control) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x02, /* Report ID (2) */ +0x75, 0x01, /* Report Size (1) */ +0x95, 0x01, /* Report Count (1) */ +0x14, /* Logical Minimum */ +0x25, 0x01, /* Logical Maximum (1) */ +0x09, 0x82, /* Usage (Sys Sleep) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x09, 0x82, /* Usage (Sys Sleep) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x09, 0x83, /* Usage (Sys Wake Up) */ +0x81, 0x06, /* Input (Data,Var,Rel) */ +0x75, 0x05, /* Report Size (5) */ +0x81, 0x01, /* Input (Const,Array,Abs) */ +0xC0, /* End Collection */ +0x05, 0x0C, /* Usage Page (Consumer) */ +0x09, 0x01, /* Usage (Consumer Control) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x03, /* Report ID (3) */ +0x14, /* Logical Minimum */ +0x26, 0xFF, 0x7F, /* Logical Maximum (32767) */ +0x95, 0x01, /* Report Count (1) */ +0x75, 0x10, /* Report Size (16) */ +0x18, /* Usage Minimum */ +0x2A, 0xFF, 0x7F, /* Usage Maximum (0x7FFF) */ +0x80, /* Input */ +0xC0, /* End Collection */ +0x06, 0x7F, 0xFF, /* Usage Page (Vendor Defined 0xFF7F) */ +0x09, 0x01, /* Usage (0x01) */ +0xA1, 0x01, /* Collection (Application) */ +0x85, 0x04, /* Report ID (4) */ +0x95, 0x01, /* Report Count (1) */ +0x75, 0x10, /* Report Size (16) */ +0x18, /* Usage Minimum */ +0x2A, 0xFF, 0x7F, /* Usage Maximum (0x7FFF) */ +0x80, /* Input */ +0x75, 0x02, /* Report Size (2) */ +0x25, 0x02, /* Logical Maximum (2) */ +0x09, 0x90, /* Usage (0x90) */ +0xB1, 0x02, /* Feature (Data,Var,Abs) */ +0x75, 0x06, /* Report Size (6) */ +0xB1, 0x01, /* Feature (Const,Array,Abs) */ +0x75, 0x01, /* Report Size (1) */ +0x25, 0x01, /* Logical Maximum (1) */ +0x05, 0x08, /* Usage Page (LEDs) */ +0x09, 0x2A, /* Usage (On-Line) */ +0x91, 0x02, /* Output (Data,Var,Abs) */ +0x09, 0x4B, /* Usage (Generic Indicator) */ +0x91, 0x02, /* Output (Data,Var,Abs) */ +0x75, 0x06, /* Report Size (6) */ +0x95, 0x01, /* Report Count (1) */ +0x91, 0x01, /* Output (Const,Array,Abs) */ +0xC0 /* End Collection */ +}; + +static __u8 *maltron_report_fixup(struct hid_device *hdev, __u8 *rdesc, + unsigned int *rsize) +{ + if (*rsize == sizeof(maltron_rdesc_o)) { + if (memcmp(maltron_rdesc_o, rdesc, sizeof(maltron_rdesc_o)) == 0) { + hid_info(hdev, "Replacing Maltron L90 Keyboard report descriptor\n"); + *rsize = sizeof(maltron_rdesc); + return maltron_rdesc; + } + } + return rdesc; +} + +static const struct hid_device_id maltron_devices[] = { + { HID_USB_DEVICE(USB_VENDOR_ID_ALCOR, USB_DEVICE_ID_ALCOR_MALTRON_KB)}, + { } +}; +MODULE_DEVICE_TABLE(hid, maltron_devices); + +static struct hid_driver maltron_driver = { + .name = "maltron", + .id_table = maltron_devices, + .report_fixup = maltron_report_fixup +}; +module_hid_driver(maltron_driver); + +MODULE_LICENSE("GPL");