| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20210529071808.5268-1-navin@linumiz.com>
Date: Sat, 29 May 2021 12:48:08 +0530
From: Navin Sankar Velliangiri <navin@...umiz.com>
To: linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Cc: jikos@...nel.org, benjamin.tissoires@...hat.com, navin@...umiz.com
Subject: [PATCH] hid: hid-chicony: fix switch case indentation
fixed switch case indentation.
Block comments should not use a trailing */ on a separate line.
Reported by checkpatch.
Signed-off-by: Navin Sankar Velliangiri <navin@...umiz.com>
---
drivers/hid/hid-chicony.c | 74 +++++++++++++++++++++++++++++----------
1 file changed, 55 insertions(+), 19 deletions(-)
diff --git a/drivers/hid/hid-chicony.c b/drivers/hid/hid-chicony.c
index ca556d39da2a..7dd4cfd256d9 100644
--- a/drivers/hid/hid-chicony.c
+++ b/drivers/hid/hid-chicony.c
@@ -65,26 +65,61 @@ static int ch_input_mapping(struct hid_device *hdev, struct hid_input *hi,
set_bit(EV_REP, hi->input->evbit);
switch (usage->hid & HID_USAGE) {
- case 0xff01: ch_map_key_clear(BTN_1); break;
- case 0xff02: ch_map_key_clear(BTN_2); break;
- case 0xff03: ch_map_key_clear(BTN_3); break;
- case 0xff04: ch_map_key_clear(BTN_4); break;
- case 0xff05: ch_map_key_clear(BTN_5); break;
- case 0xff06: ch_map_key_clear(BTN_6); break;
- case 0xff07: ch_map_key_clear(BTN_7); break;
- case 0xff08: ch_map_key_clear(BTN_8); break;
- case 0xff09: ch_map_key_clear(BTN_9); break;
- case 0xff0a: ch_map_key_clear(BTN_A); break;
- case 0xff0b: ch_map_key_clear(BTN_B); break;
- case 0x00f1: ch_map_key_clear(KEY_WLAN); break;
- case 0x00f2: ch_map_key_clear(KEY_BRIGHTNESSDOWN); break;
- case 0x00f3: ch_map_key_clear(KEY_BRIGHTNESSUP); break;
- case 0x00f4: ch_map_key_clear(KEY_DISPLAY_OFF); break;
- case 0x00f7: ch_map_key_clear(KEY_CAMERA); break;
- case 0x00f8: ch_map_key_clear(KEY_PROG1); break;
+ case 0xff01:
+ ch_map_key_clear(BTN_1);
+ break;
+ case 0xff02:
+ ch_map_key_clear(BTN_2);
+ break;
+ case 0xff03:
+ ch_map_key_clear(BTN_3);
+ break;
+ case 0xff04:
+ ch_map_key_clear(BTN_4);
+ break;
+ case 0xff05:
+ ch_map_key_clear(BTN_5);
+ break;
+ case 0xff06:
+ ch_map_key_clear(BTN_6);
+ break;
+ case 0xff07:
+ ch_map_key_clear(BTN_7);
+ break;
+ case 0xff08:
+ ch_map_key_clear(BTN_8);
+ break;
+ case 0xff09:
+ ch_map_key_clear(BTN_9);
+ break;
+ case 0xff0a:
+ ch_map_key_clear(BTN_A);
+ break;
+ case 0xff0b:
+ ch_map_key_clear(BTN_B);
+ break;
+ case 0x00f1:
+ ch_map_key_clear(KEY_WLAN);
+ break;
+ case 0x00f2:
+ ch_map_key_clear(KEY_BRIGHTNESSDOWN);
+ break;
+ case 0x00f3:
+ ch_map_key_clear(KEY_BRIGHTNESSUP);
+ break;
+ case 0x00f4:
+ ch_map_key_clear(KEY_DISPLAY_OFF);
+ break;
+ case 0x00f7:
+ ch_map_key_clear(KEY_CAMERA);
+ break;
+ case 0x00f8:
+ ch_map_key_clear(KEY_PROG1);
+ break;
default:
return 0;
}
+
return 1;
}
@@ -92,10 +127,11 @@ static __u8 *ch_switch12_report_fixup(struct hid_device *hdev, __u8 *rdesc,
unsigned int *rsize)
{
struct usb_interface *intf = to_usb_interface(hdev->dev.parent);
-
+
if (intf->cur_altsetting->desc.bInterfaceNumber == 1) {
/* Change usage maximum and logical maximum from 0x7fff to
- * 0x2fff, so they don't exceed HID_MAX_USAGES */
+ * 0x2fff, so they don't exceed HID_MAX_USAGES
+ */
switch (hdev->product) {
case USB_DEVICE_ID_CHICONY_ACER_SWITCH12:
if (*rsize >= 128 && rdesc[64] == 0xff && rdesc[65] == 0x7f
--
2.31.1
Powered by blists - more mailing lists