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@xxxxxxxxxxx> --- 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