mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2024-12-28 16:53:49 +00:00
fe73965d07
By allowing the drivers to return a "const *" they can constify their static report arrays. This makes it clear to driver authors that the HID core will not modify those reports and they can be reused for multiple devices. Furthermore security is slightly improved as those reports are protected against accidental or malicious modifications. [bentiss: fixup hid-cougar.c and hid-multitouch.c for latest version of the master branch] Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> Link: https://patch.msgid.link/20240803-hid-const-fixup-v2-6-f53d7a7b29d8@weissschuh.net Signed-off-by: Benjamin Tissoires <bentiss@kernel.org>
141 lines
4.8 KiB
C
141 lines
4.8 KiB
C
// SPDX-License-Identifier: GPL-2.0-or-later
|
|
/*
|
|
* HID driver for ELECOM devices:
|
|
* - BM084 Bluetooth Mouse
|
|
* - EX-G Trackballs (M-XT3DRBK, M-XT3URBK, M-XT4DRBK)
|
|
* - DEFT Trackballs (M-DT1DRBK, M-DT1URBK, M-DT2DRBK, M-DT2URBK)
|
|
* - HUGE Trackballs (M-HT1DRBK, M-HT1URBK)
|
|
*
|
|
* Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com>
|
|
* Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com>
|
|
* Copyright (c) 2017 Diego Elio Pettenò <flameeyes@flameeyes.eu>
|
|
* Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org>
|
|
* Copyright (c) 2017 Tomasz Kramkowski <tk@the-tk.com>
|
|
* Copyright (c) 2020 YOSHIOKA Takuma <lo48576@hard-wi.red>
|
|
* Copyright (c) 2022 Takahiro Fujii <fujii@xaxxi.net>
|
|
*/
|
|
|
|
/*
|
|
*/
|
|
|
|
#include <linux/device.h>
|
|
#include <linux/hid.h>
|
|
#include <linux/module.h>
|
|
|
|
#include "hid-ids.h"
|
|
|
|
/*
|
|
* Certain ELECOM mice misreport their button count meaning that they only work
|
|
* correctly with the ELECOM mouse assistant software which is unavailable for
|
|
* Linux. A four extra INPUT reports and a FEATURE report are described by the
|
|
* report descriptor but it does not appear that these enable software to
|
|
* control what the extra buttons map to. The only simple and straightforward
|
|
* solution seems to involve fixing up the report descriptor.
|
|
*/
|
|
#define MOUSE_BUTTONS_MAX 8
|
|
static void mouse_button_fixup(struct hid_device *hdev,
|
|
__u8 *rdesc, unsigned int rsize,
|
|
unsigned int button_bit_count,
|
|
unsigned int padding_bit,
|
|
unsigned int button_report_size,
|
|
unsigned int button_usage_maximum,
|
|
int nbuttons)
|
|
{
|
|
if (rsize < 32 || rdesc[button_bit_count] != 0x95 ||
|
|
rdesc[button_report_size] != 0x75 ||
|
|
rdesc[button_report_size + 1] != 0x01 ||
|
|
rdesc[button_usage_maximum] != 0x29 || rdesc[padding_bit] != 0x75)
|
|
return;
|
|
hid_info(hdev, "Fixing up Elecom mouse button count\n");
|
|
nbuttons = clamp(nbuttons, 0, MOUSE_BUTTONS_MAX);
|
|
rdesc[button_bit_count + 1] = nbuttons;
|
|
rdesc[button_usage_maximum + 1] = nbuttons;
|
|
rdesc[padding_bit + 1] = MOUSE_BUTTONS_MAX - nbuttons;
|
|
}
|
|
|
|
static const __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc,
|
|
unsigned int *rsize)
|
|
{
|
|
switch (hdev->product) {
|
|
case USB_DEVICE_ID_ELECOM_BM084:
|
|
/* The BM084 Bluetooth mouse includes a non-existing horizontal
|
|
* wheel in the HID descriptor. */
|
|
if (*rsize >= 48 && rdesc[46] == 0x05 && rdesc[47] == 0x0c) {
|
|
hid_info(hdev, "Fixing up Elecom BM084 report descriptor\n");
|
|
rdesc[47] = 0x00;
|
|
}
|
|
break;
|
|
case USB_DEVICE_ID_ELECOM_M_XGL20DLBK:
|
|
/*
|
|
* Report descriptor format:
|
|
* 20: button bit count
|
|
* 28: padding bit count
|
|
* 22: button report size
|
|
* 14: button usage maximum
|
|
*/
|
|
mouse_button_fixup(hdev, rdesc, *rsize, 20, 28, 22, 14, 8);
|
|
break;
|
|
case USB_DEVICE_ID_ELECOM_M_XT3URBK:
|
|
case USB_DEVICE_ID_ELECOM_M_XT3DRBK:
|
|
case USB_DEVICE_ID_ELECOM_M_XT4DRBK:
|
|
/*
|
|
* Report descriptor format:
|
|
* 12: button bit count
|
|
* 30: padding bit count
|
|
* 14: button report size
|
|
* 20: button usage maximum
|
|
*/
|
|
mouse_button_fixup(hdev, rdesc, *rsize, 12, 30, 14, 20, 6);
|
|
break;
|
|
case USB_DEVICE_ID_ELECOM_M_DT1URBK:
|
|
case USB_DEVICE_ID_ELECOM_M_DT1DRBK:
|
|
case USB_DEVICE_ID_ELECOM_M_HT1URBK:
|
|
case USB_DEVICE_ID_ELECOM_M_HT1DRBK_010D:
|
|
/*
|
|
* Report descriptor format:
|
|
* 12: button bit count
|
|
* 30: padding bit count
|
|
* 14: button report size
|
|
* 20: button usage maximum
|
|
*/
|
|
mouse_button_fixup(hdev, rdesc, *rsize, 12, 30, 14, 20, 8);
|
|
break;
|
|
case USB_DEVICE_ID_ELECOM_M_HT1DRBK_011C:
|
|
/*
|
|
* Report descriptor format:
|
|
* 22: button bit count
|
|
* 30: padding bit count
|
|
* 24: button report size
|
|
* 16: button usage maximum
|
|
*/
|
|
mouse_button_fixup(hdev, rdesc, *rsize, 22, 30, 24, 16, 8);
|
|
break;
|
|
}
|
|
return rdesc;
|
|
}
|
|
|
|
static const struct hid_device_id elecom_devices[] = {
|
|
{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_XGL20DLBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_XT3URBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_XT3DRBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_XT4DRBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_DT1URBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_DT1DRBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_HT1URBK) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_HT1DRBK_010D) },
|
|
{ HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_M_HT1DRBK_011C) },
|
|
{ }
|
|
};
|
|
MODULE_DEVICE_TABLE(hid, elecom_devices);
|
|
|
|
static struct hid_driver elecom_driver = {
|
|
.name = "elecom",
|
|
.id_table = elecom_devices,
|
|
.report_fixup = elecom_report_fixup
|
|
};
|
|
module_hid_driver(elecom_driver);
|
|
|
|
MODULE_DESCRIPTION("HID driver for ELECOM devices");
|
|
MODULE_LICENSE("GPL");
|