Received: by 2002:a05:6358:1087:b0:cb:c9d3:cd90 with SMTP id j7csp2561725rwi; Fri, 28 Oct 2022 08:28:22 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4gVC1cpmrQMqDxU25q9/uWyl4exK5esY/vM/N/U/9iVhQ6elVlzXqw2JSLrJA28c0ar8Fx X-Received: by 2002:a17:90b:254a:b0:200:53f:891d with SMTP id nw10-20020a17090b254a00b00200053f891dmr17034050pjb.168.1666970891271; Fri, 28 Oct 2022 08:28:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666970891; cv=none; d=google.com; s=arc-20160816; b=yYgHNrIkCCen6DbMxRAqw3ZAeI6NAukAjA9Prchr52LtRIZwgG7OpasCM/m3oAV+Gu Y+yRwRE3NMl3LacGQ6Nl+NcbXc+p4T24PGnbt+kdO1FbBs0muX5JTkLzk0ePuFbLEoYg 4g02OhTAwmSsSTfZMobPh89geFn5CCPt9N9Z/IcfrOWr0+0mTX+EaBkSGH4sG98QNw5a 6JkM4a6l7qTE2QVv9Cy9TCOnkfLpmcqvbX9MO/CVSaxCbzeI4NCc1/9iG3LMj2USfO/9 yGhY+5iFcNdpwm05Wk/8KbSZclqEfgFPT+//mc2aN680qb8m1EWQUlK09glFWguH4wcR uV8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id; bh=kmtj+d0w8LfdGhhK6opc+lR9+jq3mx/XyE+RgTkJxUY=; b=zG/X4abIK3smhgFsGWQmU7L0AaXHGJZ4UWrq5g+OcapumyvjbgiM2wFV8B81ecZRq3 4PJN2WEpmuRoKsjGUfLtJHZtUHdaaYL4bI/LqRQNt9I/e/y31R6kf+Fe706RnAg0BZiT 8GNiIbLhR/QiCtV5VFZDl6c0B8BnYcvVogW7VOewNRWB0Vhm0OKOUar14aqtOVCpgnSg g3K6HibUHYT1nNNyS10PSnA+VRrkh4/sKKsRwarT9UoRvfA1ZMWTQ7V6RmQ93KECgwvi GcNPHH52IUwU+wSKetzwPIMo2rPufdlneVMxLtrDiOd3f7kSCVB676EFtIqRXVlamZSF E72A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h8-20020a056a00230800b00547a857b7b3si6618200pfh.375.2022.10.28.08.27.58; Fri, 28 Oct 2022 08:28:11 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229864AbiJ1PLc convert rfc822-to-8bit (ORCPT + 99 others); Fri, 28 Oct 2022 11:11:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33372 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229592AbiJ1PLa (ORCPT ); Fri, 28 Oct 2022 11:11:30 -0400 Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A59ED1958DC; Fri, 28 Oct 2022 08:11:28 -0700 (PDT) Received: from omf19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 855961605CC; Fri, 28 Oct 2022 15:11:27 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf19.hostedemail.com (Postfix) with ESMTPA id 353EC20026; Fri, 28 Oct 2022 15:11:08 +0000 (UTC) Message-ID: <4bc6b4d967ae65c9a5bfe0dbbacb2d0f963d29a0.camel@perches.com> Subject: Re: [PATCH] HID: input: Reduce object size by minimizing repeated calls From: Joe Perches To: Jiri Kosina , Benjamin Tissoires Cc: linux-input@vger.kernel.org, LKML Date: Fri, 28 Oct 2022 08:11:24 -0700 In-Reply-To: <083f14906ceb018af243df02d27e3217f74132f8.camel@perches.com> References: <083f14906ceb018af243df02d27e3217f74132f8.camel@perches.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 8BIT User-Agent: Evolution 3.44.4 (3.44.4-2.fc36) MIME-Version: 1.0 X-Spam-Status: No, score=-0.9 required=5.0 tests=BAYES_00,FORGED_SPF_HELO, RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS, SPF_NONE,UNPARSEABLE_RELAY autolearn=no autolearn_force=no version=3.4.6 X-Stat-Signature: ruxsoq1rh8bu7gfd83377e8ks8ito35p X-Rspamd-Server: rspamout03 X-Rspamd-Queue-Id: 353EC20026 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX1/teTDUCkVCMWD3L/VFAP2dSI3jS3PEFKA= X-HE-Tag: 1666969868-86172 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2022-05-18 at 13:26 -0700, Joe Perches wrote: > On Mon, 2022-05-09 at 23:56 -0700, Joe Perches wrote: > > Reduce object size by using an automatic for various repeated calls > > to map_abs, map_key_clear, and map_led in switch/case statements. > > ping? Second ping. IMO: It's a non-trivial amount. An entire page of code. > > $ size drivers/hid/hid-input.o* (defconfig x86-64) > > text data bss dec hex filename > > 22042 88 0 22130 5672 drivers/hid/hid-input.o.new > > 27589 88 0 27677 6c1d drivers/hid/hid-input.o.old > > > > Signed-off-by: Joe Perches > > --- > > drivers/hid/hid-input.c | 522 ++++++++++++++++++++++++++---------------------- > > 1 file changed, 278 insertions(+), 244 deletions(-) > > > > diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c > > index c6b27aab90414..21d67866efb06 100644 > > --- a/drivers/hid/hid-input.c > > +++ b/drivers/hid/hid-input.c > > @@ -743,37 +743,44 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel > > map_key(code); > > break; > > > > - case HID_UP_SIMULATION: > > + case HID_UP_SIMULATION: { > > + int abs; > > + > > switch (usage->hid & 0xffff) { > > - case 0xba: map_abs(ABS_RUDDER); break; > > - case 0xbb: map_abs(ABS_THROTTLE); break; > > - case 0xc4: map_abs(ABS_GAS); break; > > - case 0xc5: map_abs(ABS_BRAKE); break; > > - case 0xc8: map_abs(ABS_WHEEL); break; > > + case 0xba: abs = ABS_RUDDER; break; > > + case 0xbb: abs = ABS_THROTTLE; break; > > + case 0xc4: abs = ABS_GAS; break; > > + case 0xc5: abs = ABS_BRAKE; break; > > + case 0xc8: abs = ABS_WHEEL; break; > > default: goto ignore; > > } > > + map_abs(abs); > > break; > > + } > > > > case HID_UP_GENDESK: > > if ((usage->hid & 0xf0) == 0x80) { /* SystemControl */ > > + int key; > > + > > switch (usage->hid & 0xf) { > > - case 0x1: map_key_clear(KEY_POWER); break; > > - case 0x2: map_key_clear(KEY_SLEEP); break; > > - case 0x3: map_key_clear(KEY_WAKEUP); break; > > - case 0x4: map_key_clear(KEY_CONTEXT_MENU); break; > > - case 0x5: map_key_clear(KEY_MENU); break; > > - case 0x6: map_key_clear(KEY_PROG1); break; > > - case 0x7: map_key_clear(KEY_HELP); break; > > - case 0x8: map_key_clear(KEY_EXIT); break; > > - case 0x9: map_key_clear(KEY_SELECT); break; > > - case 0xa: map_key_clear(KEY_RIGHT); break; > > - case 0xb: map_key_clear(KEY_LEFT); break; > > - case 0xc: map_key_clear(KEY_UP); break; > > - case 0xd: map_key_clear(KEY_DOWN); break; > > - case 0xe: map_key_clear(KEY_POWER2); break; > > - case 0xf: map_key_clear(KEY_RESTART); break; > > + case 0x1: key = KEY_POWER; break; > > + case 0x2: key = KEY_SLEEP; break; > > + case 0x3: key = KEY_WAKEUP; break; > > + case 0x4: key = KEY_CONTEXT_MENU; break; > > + case 0x5: key = KEY_MENU; break; > > + case 0x6: key = KEY_PROG1; break; > > + case 0x7: key = KEY_HELP; break; > > + case 0x8: key = KEY_EXIT; break; > > + case 0x9: key = KEY_SELECT; break; > > + case 0xa: key = KEY_RIGHT; break; > > + case 0xb: key = KEY_LEFT; break; > > + case 0xc: key = KEY_UP; break; > > + case 0xd: key = KEY_DOWN; break; > > + case 0xe: key = KEY_POWER2; break; > > + case 0xf: key = KEY_RESTART; break; > > default: goto unknown; > > } > > + map_key_clear(key); > > break; > > } > > > > @@ -859,23 +866,27 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel > > > > break; > > > > - case HID_UP_LED: > > - switch (usage->hid & 0xffff) { /* HID-Value: */ > > - case 0x01: map_led (LED_NUML); break; /* "Num Lock" */ > > - case 0x02: map_led (LED_CAPSL); break; /* "Caps Lock" */ > > - case 0x03: map_led (LED_SCROLLL); break; /* "Scroll Lock" */ > > - case 0x04: map_led (LED_COMPOSE); break; /* "Compose" */ > > - case 0x05: map_led (LED_KANA); break; /* "Kana" */ > > - case 0x27: map_led (LED_SLEEP); break; /* "Stand-By" */ > > - case 0x4c: map_led (LED_SUSPEND); break; /* "System Suspend" */ > > - case 0x09: map_led (LED_MUTE); break; /* "Mute" */ > > - case 0x4b: map_led (LED_MISC); break; /* "Generic Indicator" */ > > - case 0x19: map_led (LED_MAIL); break; /* "Message Waiting" */ > > - case 0x4d: map_led (LED_CHARGING); break; /* "External Power Connected" */ > > + case HID_UP_LED: { > > + int led; > > + > > + switch (usage->hid & 0xffff) { /* HID-Value: */ > > + case 0x01: led = LED_NUML; break; /* Num Lock */ > > + case 0x02: led = LED_CAPSL; break; /* Caps Lock */ > > + case 0x03: led = LED_SCROLLL; break; /* Scroll Lock */ > > + case 0x04: led = LED_COMPOSE; break; /* Compose */ > > + case 0x05: led = LED_KANA; break; /* Kana */ > > + case 0x27: led = LED_SLEEP; break; /* Stand-By */ > > + case 0x4c: led = LED_SUSPEND; break; /* System Suspend */ > > + case 0x09: led = LED_MUTE; break; /* Mute */ > > + case 0x4b: led = LED_MISC; break; /* Generic Indicator */ > > + case 0x19: led = LED_MAIL; break; /* Message Waiting */ > > + case 0x4d: led = LED_CHARGING; break; /* External Power Connected */ > > > > default: goto ignore; > > } > > + map_led(led); > > break; > > + } > > > > case HID_UP_DIGITIZER: > > if ((field->application & 0xff) == 0x01) /* Digitizer */ > > @@ -976,218 +987,233 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel > > } > > break; > > > > - case HID_UP_TELEPHONY: > > + case HID_UP_TELEPHONY: { > > + int key; > > + > > switch (usage->hid & HID_USAGE) { > > - case 0x2f: map_key_clear(KEY_MICMUTE); break; > > - case 0xb0: map_key_clear(KEY_NUMERIC_0); break; > > - case 0xb1: map_key_clear(KEY_NUMERIC_1); break; > > - case 0xb2: map_key_clear(KEY_NUMERIC_2); break; > > - case 0xb3: map_key_clear(KEY_NUMERIC_3); break; > > - case 0xb4: map_key_clear(KEY_NUMERIC_4); break; > > - case 0xb5: map_key_clear(KEY_NUMERIC_5); break; > > - case 0xb6: map_key_clear(KEY_NUMERIC_6); break; > > - case 0xb7: map_key_clear(KEY_NUMERIC_7); break; > > - case 0xb8: map_key_clear(KEY_NUMERIC_8); break; > > - case 0xb9: map_key_clear(KEY_NUMERIC_9); break; > > - case 0xba: map_key_clear(KEY_NUMERIC_STAR); break; > > - case 0xbb: map_key_clear(KEY_NUMERIC_POUND); break; > > - case 0xbc: map_key_clear(KEY_NUMERIC_A); break; > > - case 0xbd: map_key_clear(KEY_NUMERIC_B); break; > > - case 0xbe: map_key_clear(KEY_NUMERIC_C); break; > > - case 0xbf: map_key_clear(KEY_NUMERIC_D); break; > > + case 0x2f: key = KEY_MICMUTE; break; > > + case 0xb0: key = KEY_NUMERIC_0; break; > > + case 0xb1: key = KEY_NUMERIC_1; break; > > + case 0xb2: key = KEY_NUMERIC_2; break; > > + case 0xb3: key = KEY_NUMERIC_3; break; > > + case 0xb4: key = KEY_NUMERIC_4; break; > > + case 0xb5: key = KEY_NUMERIC_5; break; > > + case 0xb6: key = KEY_NUMERIC_6; break; > > + case 0xb7: key = KEY_NUMERIC_7; break; > > + case 0xb8: key = KEY_NUMERIC_8; break; > > + case 0xb9: key = KEY_NUMERIC_9; break; > > + case 0xba: key = KEY_NUMERIC_STAR; break; > > + case 0xbb: key = KEY_NUMERIC_POUND; break; > > + case 0xbc: key = KEY_NUMERIC_A; break; > > + case 0xbd: key = KEY_NUMERIC_B; break; > > + case 0xbe: key = KEY_NUMERIC_C; break; > > + case 0xbf: key = KEY_NUMERIC_D; break; > > default: goto ignore; > > } > > + map_key_clear(key); > > break; > > + } > > + > > + case HID_UP_CONSUMER: { /* USB HUT v1.12, pages 75-84 */ > > + int key; > > + bool clear = true; > > > > - case HID_UP_CONSUMER: /* USB HUT v1.12, pages 75-84 */ > > switch (usage->hid & HID_USAGE) { > > case 0x000: goto ignore; > > - case 0x030: map_key_clear(KEY_POWER); break; > > - case 0x031: map_key_clear(KEY_RESTART); break; > > - case 0x032: map_key_clear(KEY_SLEEP); break; > > - case 0x034: map_key_clear(KEY_SLEEP); break; > > - case 0x035: map_key_clear(KEY_KBDILLUMTOGGLE); break; > > - case 0x036: map_key_clear(BTN_MISC); break; > > - > > - case 0x040: map_key_clear(KEY_MENU); break; /* Menu */ > > - case 0x041: map_key_clear(KEY_SELECT); break; /* Menu Pick */ > > - case 0x042: map_key_clear(KEY_UP); break; /* Menu Up */ > > - case 0x043: map_key_clear(KEY_DOWN); break; /* Menu Down */ > > - case 0x044: map_key_clear(KEY_LEFT); break; /* Menu Left */ > > - case 0x045: map_key_clear(KEY_RIGHT); break; /* Menu Right */ > > - case 0x046: map_key_clear(KEY_ESC); break; /* Menu Escape */ > > - case 0x047: map_key_clear(KEY_KPPLUS); break; /* Menu Value Increase */ > > - case 0x048: map_key_clear(KEY_KPMINUS); break; /* Menu Value Decrease */ > > - > > - case 0x060: map_key_clear(KEY_INFO); break; /* Data On Screen */ > > - case 0x061: map_key_clear(KEY_SUBTITLE); break; /* Closed Caption */ > > - case 0x063: map_key_clear(KEY_VCR); break; /* VCR/TV */ > > - case 0x065: map_key_clear(KEY_CAMERA); break; /* Snapshot */ > > - case 0x069: map_key_clear(KEY_RED); break; > > - case 0x06a: map_key_clear(KEY_GREEN); break; > > - case 0x06b: map_key_clear(KEY_BLUE); break; > > - case 0x06c: map_key_clear(KEY_YELLOW); break; > > - case 0x06d: map_key_clear(KEY_ASPECT_RATIO); break; > > - > > - case 0x06f: map_key_clear(KEY_BRIGHTNESSUP); break; > > - case 0x070: map_key_clear(KEY_BRIGHTNESSDOWN); break; > > - case 0x072: map_key_clear(KEY_BRIGHTNESS_TOGGLE); break; > > - case 0x073: map_key_clear(KEY_BRIGHTNESS_MIN); break; > > - case 0x074: map_key_clear(KEY_BRIGHTNESS_MAX); break; > > - case 0x075: map_key_clear(KEY_BRIGHTNESS_AUTO); break; > > - > > - case 0x079: map_key_clear(KEY_KBDILLUMUP); break; > > - case 0x07a: map_key_clear(KEY_KBDILLUMDOWN); break; > > - case 0x07c: map_key_clear(KEY_KBDILLUMTOGGLE); break; > > - > > - case 0x082: map_key_clear(KEY_VIDEO_NEXT); break; > > - case 0x083: map_key_clear(KEY_LAST); break; > > - case 0x084: map_key_clear(KEY_ENTER); break; > > - case 0x088: map_key_clear(KEY_PC); break; > > - case 0x089: map_key_clear(KEY_TV); break; > > - case 0x08a: map_key_clear(KEY_WWW); break; > > - case 0x08b: map_key_clear(KEY_DVD); break; > > - case 0x08c: map_key_clear(KEY_PHONE); break; > > - case 0x08d: map_key_clear(KEY_PROGRAM); break; > > - case 0x08e: map_key_clear(KEY_VIDEOPHONE); break; > > - case 0x08f: map_key_clear(KEY_GAMES); break; > > - case 0x090: map_key_clear(KEY_MEMO); break; > > - case 0x091: map_key_clear(KEY_CD); break; > > - case 0x092: map_key_clear(KEY_VCR); break; > > - case 0x093: map_key_clear(KEY_TUNER); break; > > - case 0x094: map_key_clear(KEY_EXIT); break; > > - case 0x095: map_key_clear(KEY_HELP); break; > > - case 0x096: map_key_clear(KEY_TAPE); break; > > - case 0x097: map_key_clear(KEY_TV2); break; > > - case 0x098: map_key_clear(KEY_SAT); break; > > - case 0x09a: map_key_clear(KEY_PVR); break; > > - > > - case 0x09c: map_key_clear(KEY_CHANNELUP); break; > > - case 0x09d: map_key_clear(KEY_CHANNELDOWN); break; > > - case 0x0a0: map_key_clear(KEY_VCR2); break; > > - > > - case 0x0b0: map_key_clear(KEY_PLAY); break; > > - case 0x0b1: map_key_clear(KEY_PAUSE); break; > > - case 0x0b2: map_key_clear(KEY_RECORD); break; > > - case 0x0b3: map_key_clear(KEY_FASTFORWARD); break; > > - case 0x0b4: map_key_clear(KEY_REWIND); break; > > - case 0x0b5: map_key_clear(KEY_NEXTSONG); break; > > - case 0x0b6: map_key_clear(KEY_PREVIOUSSONG); break; > > - case 0x0b7: map_key_clear(KEY_STOPCD); break; > > - case 0x0b8: map_key_clear(KEY_EJECTCD); break; > > - case 0x0bc: map_key_clear(KEY_MEDIA_REPEAT); break; > > - case 0x0b9: map_key_clear(KEY_SHUFFLE); break; > > - case 0x0bf: map_key_clear(KEY_SLOW); break; > > - > > - case 0x0cd: map_key_clear(KEY_PLAYPAUSE); break; > > - case 0x0cf: map_key_clear(KEY_VOICECOMMAND); break; > > - > > - case 0x0d8: map_key_clear(KEY_DICTATE); break; > > - case 0x0d9: map_key_clear(KEY_EMOJI_PICKER); break; > > - > > - case 0x0e0: map_abs_clear(ABS_VOLUME); break; > > - case 0x0e2: map_key_clear(KEY_MUTE); break; > > - case 0x0e5: map_key_clear(KEY_BASSBOOST); break; > > - case 0x0e9: map_key_clear(KEY_VOLUMEUP); break; > > - case 0x0ea: map_key_clear(KEY_VOLUMEDOWN); break; > > - case 0x0f5: map_key_clear(KEY_SLOW); break; > > - > > - case 0x181: map_key_clear(KEY_BUTTONCONFIG); break; > > - case 0x182: map_key_clear(KEY_BOOKMARKS); break; > > - case 0x183: map_key_clear(KEY_CONFIG); break; > > - case 0x184: map_key_clear(KEY_WORDPROCESSOR); break; > > - case 0x185: map_key_clear(KEY_EDITOR); break; > > - case 0x186: map_key_clear(KEY_SPREADSHEET); break; > > - case 0x187: map_key_clear(KEY_GRAPHICSEDITOR); break; > > - case 0x188: map_key_clear(KEY_PRESENTATION); break; > > - case 0x189: map_key_clear(KEY_DATABASE); break; > > - case 0x18a: map_key_clear(KEY_MAIL); break; > > - case 0x18b: map_key_clear(KEY_NEWS); break; > > - case 0x18c: map_key_clear(KEY_VOICEMAIL); break; > > - case 0x18d: map_key_clear(KEY_ADDRESSBOOK); break; > > - case 0x18e: map_key_clear(KEY_CALENDAR); break; > > - case 0x18f: map_key_clear(KEY_TASKMANAGER); break; > > - case 0x190: map_key_clear(KEY_JOURNAL); break; > > - case 0x191: map_key_clear(KEY_FINANCE); break; > > - case 0x192: map_key_clear(KEY_CALC); break; > > - case 0x193: map_key_clear(KEY_PLAYER); break; > > - case 0x194: map_key_clear(KEY_FILE); break; > > - case 0x196: map_key_clear(KEY_WWW); break; > > - case 0x199: map_key_clear(KEY_CHAT); break; > > - case 0x19c: map_key_clear(KEY_LOGOFF); break; > > - case 0x19e: map_key_clear(KEY_COFFEE); break; > > - case 0x19f: map_key_clear(KEY_CONTROLPANEL); break; > > - case 0x1a2: map_key_clear(KEY_APPSELECT); break; > > - case 0x1a3: map_key_clear(KEY_NEXT); break; > > - case 0x1a4: map_key_clear(KEY_PREVIOUS); break; > > - case 0x1a6: map_key_clear(KEY_HELP); break; > > - case 0x1a7: map_key_clear(KEY_DOCUMENTS); break; > > - case 0x1ab: map_key_clear(KEY_SPELLCHECK); break; > > - case 0x1ae: map_key_clear(KEY_KEYBOARD); break; > > - case 0x1b1: map_key_clear(KEY_SCREENSAVER); break; > > - case 0x1b4: map_key_clear(KEY_FILE); break; > > - case 0x1b6: map_key_clear(KEY_IMAGES); break; > > - case 0x1b7: map_key_clear(KEY_AUDIO); break; > > - case 0x1b8: map_key_clear(KEY_VIDEO); break; > > - case 0x1bc: map_key_clear(KEY_MESSENGER); break; > > - case 0x1bd: map_key_clear(KEY_INFO); break; > > - case 0x1cb: map_key_clear(KEY_ASSISTANT); break; > > - case 0x201: map_key_clear(KEY_NEW); break; > > - case 0x202: map_key_clear(KEY_OPEN); break; > > - case 0x203: map_key_clear(KEY_CLOSE); break; > > - case 0x204: map_key_clear(KEY_EXIT); break; > > - case 0x207: map_key_clear(KEY_SAVE); break; > > - case 0x208: map_key_clear(KEY_PRINT); break; > > - case 0x209: map_key_clear(KEY_PROPS); break; > > - case 0x21a: map_key_clear(KEY_UNDO); break; > > - case 0x21b: map_key_clear(KEY_COPY); break; > > - case 0x21c: map_key_clear(KEY_CUT); break; > > - case 0x21d: map_key_clear(KEY_PASTE); break; > > - case 0x21f: map_key_clear(KEY_FIND); break; > > - case 0x221: map_key_clear(KEY_SEARCH); break; > > - case 0x222: map_key_clear(KEY_GOTO); break; > > - case 0x223: map_key_clear(KEY_HOMEPAGE); break; > > - case 0x224: map_key_clear(KEY_BACK); break; > > - case 0x225: map_key_clear(KEY_FORWARD); break; > > - case 0x226: map_key_clear(KEY_STOP); break; > > - case 0x227: map_key_clear(KEY_REFRESH); break; > > - case 0x22a: map_key_clear(KEY_BOOKMARKS); break; > > - case 0x22d: map_key_clear(KEY_ZOOMIN); break; > > - case 0x22e: map_key_clear(KEY_ZOOMOUT); break; > > - case 0x22f: map_key_clear(KEY_ZOOMRESET); break; > > - case 0x232: map_key_clear(KEY_FULL_SCREEN); break; > > - case 0x233: map_key_clear(KEY_SCROLLUP); break; > > - case 0x234: map_key_clear(KEY_SCROLLDOWN); break; > > + case 0x030: key = KEY_POWER; break; > > + case 0x031: key = KEY_RESTART; break; > > + case 0x032: key = KEY_SLEEP; break; > > + case 0x034: key = KEY_SLEEP; break; > > + case 0x035: key = KEY_KBDILLUMTOGGLE; break; > > + case 0x036: key = BTN_MISC; break; > > + > > + case 0x040: key = KEY_MENU; break; /* Menu */ > > + case 0x041: key = KEY_SELECT; break; /* Menu Pick */ > > + case 0x042: key = KEY_UP; break; /* Menu Up */ > > + case 0x043: key = KEY_DOWN; break; /* Menu Down */ > > + case 0x044: key = KEY_LEFT; break; /* Menu Left */ > > + case 0x045: key = KEY_RIGHT; break; /* Menu Right */ > > + case 0x046: key = KEY_ESC; break; /* Menu Escape */ > > + case 0x047: key = KEY_KPPLUS; break; /* Menu Value Increase */ > > + case 0x048: key = KEY_KPMINUS; break; /* Menu Value Decrease */ > > + > > + case 0x060: key = KEY_INFO; break; /* Data On Screen */ > > + case 0x061: key = KEY_SUBTITLE; break; /* Closed Caption */ > > + case 0x063: key = KEY_VCR; break; /* VCR/TV */ > > + case 0x065: key = KEY_CAMERA; break; /* Snapshot */ > > + case 0x069: key = KEY_RED; break; > > + case 0x06a: key = KEY_GREEN; break; > > + case 0x06b: key = KEY_BLUE; break; > > + case 0x06c: key = KEY_YELLOW; break; > > + case 0x06d: key = KEY_ASPECT_RATIO; break; > > + > > + case 0x06f: key = KEY_BRIGHTNESSUP; break; > > + case 0x070: key = KEY_BRIGHTNESSDOWN; break; > > + case 0x072: key = KEY_BRIGHTNESS_TOGGLE; break; > > + case 0x073: key = KEY_BRIGHTNESS_MIN; break; > > + case 0x074: key = KEY_BRIGHTNESS_MAX; break; > > + case 0x075: key = KEY_BRIGHTNESS_AUTO; break; > > + > > + case 0x079: key = KEY_KBDILLUMUP; break; > > + case 0x07a: key = KEY_KBDILLUMDOWN; break; > > + case 0x07c: key = KEY_KBDILLUMTOGGLE; break; > > + > > + case 0x082: key = KEY_VIDEO_NEXT; break; > > + case 0x083: key = KEY_LAST; break; > > + case 0x084: key = KEY_ENTER; break; > > + case 0x088: key = KEY_PC; break; > > + case 0x089: key = KEY_TV; break; > > + case 0x08a: key = KEY_WWW; break; > > + case 0x08b: key = KEY_DVD; break; > > + case 0x08c: key = KEY_PHONE; break; > > + case 0x08d: key = KEY_PROGRAM; break; > > + case 0x08e: key = KEY_VIDEOPHONE; break; > > + case 0x08f: key = KEY_GAMES; break; > > + case 0x090: key = KEY_MEMO; break; > > + case 0x091: key = KEY_CD; break; > > + case 0x092: key = KEY_VCR; break; > > + case 0x093: key = KEY_TUNER; break; > > + case 0x094: key = KEY_EXIT; break; > > + case 0x095: key = KEY_HELP; break; > > + case 0x096: key = KEY_TAPE; break; > > + case 0x097: key = KEY_TV2; break; > > + case 0x098: key = KEY_SAT; break; > > + case 0x09a: key = KEY_PVR; break; > > + > > + case 0x09c: key = KEY_CHANNELUP; break; > > + case 0x09d: key = KEY_CHANNELDOWN; break; > > + case 0x0a0: key = KEY_VCR2; break; > > + > > + case 0x0b0: key = KEY_PLAY; break; > > + case 0x0b1: key = KEY_PAUSE; break; > > + case 0x0b2: key = KEY_RECORD; break; > > + case 0x0b3: key = KEY_FASTFORWARD; break; > > + case 0x0b4: key = KEY_REWIND; break; > > + case 0x0b5: key = KEY_NEXTSONG; break; > > + case 0x0b6: key = KEY_PREVIOUSSONG; break; > > + case 0x0b7: key = KEY_STOPCD; break; > > + case 0x0b8: key = KEY_EJECTCD; break; > > + case 0x0bc: key = KEY_MEDIA_REPEAT; break; > > + case 0x0b9: key = KEY_SHUFFLE; break; > > + case 0x0bf: key = KEY_SLOW; break; > > + > > + case 0x0cd: key = KEY_PLAYPAUSE; break; > > + case 0x0cf: key = KEY_VOICECOMMAND; break; > > + > > + case 0x0d8: key = KEY_DICTATE; break; > > + case 0x0d9: key = KEY_EMOJI_PICKER; break; > > + > > + case 0x0e0: > > + map_abs_clear(ABS_VOLUME); > > + clear = false; > > + break; > > + case 0x0e2: key = KEY_MUTE; break; > > + case 0x0e5: key = KEY_BASSBOOST; break; > > + case 0x0e9: key = KEY_VOLUMEUP; break; > > + case 0x0ea: key = KEY_VOLUMEDOWN; break; > > + case 0x0f5: key = KEY_SLOW; break; > > + > > + case 0x181: key = KEY_BUTTONCONFIG; break; > > + case 0x182: key = KEY_BOOKMARKS; break; > > + case 0x183: key = KEY_CONFIG; break; > > + case 0x184: key = KEY_WORDPROCESSOR; break; > > + case 0x185: key = KEY_EDITOR; break; > > + case 0x186: key = KEY_SPREADSHEET; break; > > + case 0x187: key = KEY_GRAPHICSEDITOR; break; > > + case 0x188: key = KEY_PRESENTATION; break; > > + case 0x189: key = KEY_DATABASE; break; > > + case 0x18a: key = KEY_MAIL; break; > > + case 0x18b: key = KEY_NEWS; break; > > + case 0x18c: key = KEY_VOICEMAIL; break; > > + case 0x18d: key = KEY_ADDRESSBOOK; break; > > + case 0x18e: key = KEY_CALENDAR; break; > > + case 0x18f: key = KEY_TASKMANAGER; break; > > + case 0x190: key = KEY_JOURNAL; break; > > + case 0x191: key = KEY_FINANCE; break; > > + case 0x192: key = KEY_CALC; break; > > + case 0x193: key = KEY_PLAYER; break; > > + case 0x194: key = KEY_FILE; break; > > + case 0x196: key = KEY_WWW; break; > > + case 0x199: key = KEY_CHAT; break; > > + case 0x19c: key = KEY_LOGOFF; break; > > + case 0x19e: key = KEY_COFFEE; break; > > + case 0x19f: key = KEY_CONTROLPANEL; break; > > + case 0x1a2: key = KEY_APPSELECT; break; > > + case 0x1a3: key = KEY_NEXT; break; > > + case 0x1a4: key = KEY_PREVIOUS; break; > > + case 0x1a6: key = KEY_HELP; break; > > + case 0x1a7: key = KEY_DOCUMENTS; break; > > + case 0x1ab: key = KEY_SPELLCHECK; break; > > + case 0x1ae: key = KEY_KEYBOARD; break; > > + case 0x1b1: key = KEY_SCREENSAVER; break; > > + case 0x1b4: key = KEY_FILE; break; > > + case 0x1b6: key = KEY_IMAGES; break; > > + case 0x1b7: key = KEY_AUDIO; break; > > + case 0x1b8: key = KEY_VIDEO; break; > > + case 0x1bc: key = KEY_MESSENGER; break; > > + case 0x1bd: key = KEY_INFO; break; > > + case 0x1cb: key = KEY_ASSISTANT; break; > > + case 0x201: key = KEY_NEW; break; > > + case 0x202: key = KEY_OPEN; break; > > + case 0x203: key = KEY_CLOSE; break; > > + case 0x204: key = KEY_EXIT; break; > > + case 0x207: key = KEY_SAVE; break; > > + case 0x208: key = KEY_PRINT; break; > > + case 0x209: key = KEY_PROPS; break; > > + case 0x21a: key = KEY_UNDO; break; > > + case 0x21b: key = KEY_COPY; break; > > + case 0x21c: key = KEY_CUT; break; > > + case 0x21d: key = KEY_PASTE; break; > > + case 0x21f: key = KEY_FIND; break; > > + case 0x221: key = KEY_SEARCH; break; > > + case 0x222: key = KEY_GOTO; break; > > + case 0x223: key = KEY_HOMEPAGE; break; > > + case 0x224: key = KEY_BACK; break; > > + case 0x225: key = KEY_FORWARD; break; > > + case 0x226: key = KEY_STOP; break; > > + case 0x227: key = KEY_REFRESH; break; > > + case 0x22a: key = KEY_BOOKMARKS; break; > > + case 0x22d: key = KEY_ZOOMIN; break; > > + case 0x22e: key = KEY_ZOOMOUT; break; > > + case 0x22f: key = KEY_ZOOMRESET; break; > > + case 0x232: key = KEY_FULL_SCREEN; break; > > + case 0x233: key = KEY_SCROLLUP; break; > > + case 0x234: key = KEY_SCROLLDOWN; break; > > case 0x238: /* AC Pan */ > > set_bit(REL_HWHEEL, input->relbit); > > map_rel(REL_HWHEEL_HI_RES); > > + clear = false; > > break; > > - case 0x23d: map_key_clear(KEY_EDIT); break; > > - case 0x25f: map_key_clear(KEY_CANCEL); break; > > - case 0x269: map_key_clear(KEY_INSERT); break; > > - case 0x26a: map_key_clear(KEY_DELETE); break; > > - case 0x279: map_key_clear(KEY_REDO); break; > > + case 0x23d: key = KEY_EDIT; break; > > + case 0x25f: key = KEY_CANCEL; break; > > + case 0x269: key = KEY_INSERT; break; > > + case 0x26a: key = KEY_DELETE; break; > > + case 0x279: key = KEY_REDO; break; > > > > - case 0x289: map_key_clear(KEY_REPLY); break; > > - case 0x28b: map_key_clear(KEY_FORWARDMAIL); break; > > - case 0x28c: map_key_clear(KEY_SEND); break; > > + case 0x289: key = KEY_REPLY; break; > > + case 0x28b: key = KEY_FORWARDMAIL; break; > > + case 0x28c: key = KEY_SEND; break; > > > > - case 0x29d: map_key_clear(KEY_KBD_LAYOUT_NEXT); break; > > + case 0x29d: key = KEY_KBD_LAYOUT_NEXT; break; > > > > - case 0x2a2: map_key_clear(KEY_ALL_APPLICATIONS); break; > > + case 0x2a2: key = KEY_ALL_APPLICATIONS; break; > > > > - case 0x2c7: map_key_clear(KEY_KBDINPUTASSIST_PREV); break; > > - case 0x2c8: map_key_clear(KEY_KBDINPUTASSIST_NEXT); break; > > - case 0x2c9: map_key_clear(KEY_KBDINPUTASSIST_PREVGROUP); break; > > - case 0x2ca: map_key_clear(KEY_KBDINPUTASSIST_NEXTGROUP); break; > > - case 0x2cb: map_key_clear(KEY_KBDINPUTASSIST_ACCEPT); break; > > - case 0x2cc: map_key_clear(KEY_KBDINPUTASSIST_CANCEL); break; > > + case 0x2c7: key = KEY_KBDINPUTASSIST_PREV; break; > > + case 0x2c8: key = KEY_KBDINPUTASSIST_NEXT; break; > > + case 0x2c9: key = KEY_KBDINPUTASSIST_PREVGROUP; break; > > + case 0x2ca: key = KEY_KBDINPUTASSIST_NEXTGROUP; break; > > + case 0x2cb: key = KEY_KBDINPUTASSIST_ACCEPT; break; > > + case 0x2cc: key = KEY_KBDINPUTASSIST_CANCEL; break; > > > > - case 0x29f: map_key_clear(KEY_SCALE); break; > > + case 0x29f: key = KEY_SCALE; break; > > > > - default: map_key_clear(KEY_UNKNOWN); > > + default: key = KEY_UNKNOWN; break; > > } > > + > > + if (clear) > > + map_key_clear(key); > > break; > > + } > > > > case HID_UP_GENDEVCTRLS: > > switch (usage->hid) { > > @@ -1207,34 +1233,42 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel > > } > > goto unknown; > > > > - case HID_UP_HPVENDOR: /* Reported on a Dutch layout HP5308 */ > > + case HID_UP_HPVENDOR: { /* Reported on a Dutch layout HP5308 */ > > + int key; > > + > > set_bit(EV_REP, input->evbit); > > switch (usage->hid & HID_USAGE) { > > - case 0x021: map_key_clear(KEY_PRINT); break; > > - case 0x070: map_key_clear(KEY_HP); break; > > - case 0x071: map_key_clear(KEY_CAMERA); break; > > - case 0x072: map_key_clear(KEY_SOUND); break; > > - case 0x073: map_key_clear(KEY_QUESTION); break; > > - case 0x080: map_key_clear(KEY_EMAIL); break; > > - case 0x081: map_key_clear(KEY_CHAT); break; > > - case 0x082: map_key_clear(KEY_SEARCH); break; > > - case 0x083: map_key_clear(KEY_CONNECT); break; > > - case 0x084: map_key_clear(KEY_FINANCE); break; > > - case 0x085: map_key_clear(KEY_SPORT); break; > > - case 0x086: map_key_clear(KEY_SHOP); break; > > + case 0x021: key = KEY_PRINT; break; > > + case 0x070: key = KEY_HP; break; > > + case 0x071: key = KEY_CAMERA; break; > > + case 0x072: key = KEY_SOUND; break; > > + case 0x073: key = KEY_QUESTION; break; > > + case 0x080: key = KEY_EMAIL; break; > > + case 0x081: key = KEY_CHAT; break; > > + case 0x082: key = KEY_SEARCH; break; > > + case 0x083: key = KEY_CONNECT; break; > > + case 0x084: key = KEY_FINANCE; break; > > + case 0x085: key = KEY_SPORT; break; > > + case 0x086: key = KEY_SHOP; break; > > default: goto ignore; > > } > > + map_key_clear(key); > > break; > > + } > > + > > + case HID_UP_HPVENDOR2: { > > + int key; > > > > - case HID_UP_HPVENDOR2: > > set_bit(EV_REP, input->evbit); > > switch (usage->hid & HID_USAGE) { > > - case 0x001: map_key_clear(KEY_MICMUTE); break; > > - case 0x003: map_key_clear(KEY_BRIGHTNESSDOWN); break; > > - case 0x004: map_key_clear(KEY_BRIGHTNESSUP); break; > > + case 0x001: key = KEY_MICMUTE; break; > > + case 0x003: key = KEY_BRIGHTNESSDOWN; break; > > + case 0x004: key = KEY_BRIGHTNESSUP; break; > > default: goto ignore; > > } > > + map_key_clear(key); > > break; > > + } > > > > case HID_UP_MSVENDOR: > > goto ignore; > > > >