Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp920119iob; Wed, 18 May 2022 16:32:53 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy4/7fKe/HlgWMYdC4HSX7ARzmQf73QcTIT9tYhv0X78e8eS5f/sIdZTYi07Ho8S2wKf0WM X-Received: by 2002:a63:190c:0:b0:3f4:8e84:56aa with SMTP id z12-20020a63190c000000b003f48e8456aamr1545942pgl.568.1652916773363; Wed, 18 May 2022 16:32:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1652916773; cv=none; d=google.com; s=arc-20160816; b=babiWMICtxU2ktgLXhJ4Xter4VNfvStcC+FgvHdklXX8ogNpuXDAqwdo4kAm7WPQr8 bMe+VymxAp1Y0yOFHWzZuxcGzJxqnn6a7DcWScmoScfpawzhv0rsKMNkZxGMtBoVLzd8 eng3mPdJ9he3VmVaHjHvcMDdK1SRfpKP36bw5U60niH1Je7zYvjDvdLQ08qhSwvRMgnt jG6JRDs38kEoV5bE4VpPaknStJUuFSmCy1+MQYUfbWZAyaLVsYBYi6R9WBMg+cx26+Yf jJxOtK58xIh9f2nROBF9Uu0m4ktFoEGQyfM0/EP0Bk3R4QwmWdOPI/tVvgO+86wzuUbA WegQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=3PjQuKMkTY1kyPsdXkRWcy11RqWiSJraBoDtsiZIFcs=; b=j5wXUQtU3WmAecavaIBWyKRWwqiRVBwwSOOqihgvqtncweQy9B+0H46Hk0AUs4IMCw cRtrB6DK2ikOndAfhfwHMi/ND8ULpee81Un1mYtvaEURIOfDC4DPaz7s47PFE041msEK qy5qm6Ym3qjMO4mNImY50BIkx3RwMEeNTCLinRcCXcUsM672ZzGcOcQVHcwc38gGLapc ryJ0xPi3Fx0lRgruJYj4IWf3tSJ0n3tenu3pBXhUVAI4frzBnk2WHAKZbISFXpU3suXW MpIViW0QZbM8r+ykUDy+EaDznV2ms/YVbdV1RKyt7dTAaEHwA0Yc5smUR2XhuIIViWJC LT3g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b="qhYeSW3/"; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f13-20020a63554d000000b003c6d5d28d7esi4132443pgm.176.2022.05.18.16.32.41; Wed, 18 May 2022 16:32:53 -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; dkim=pass header.i=@gmail.com header.s=20210112 header.b="qhYeSW3/"; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230084AbiERWrQ (ORCPT + 99 others); Wed, 18 May 2022 18:47:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34366 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230171AbiERWrN (ORCPT ); Wed, 18 May 2022 18:47:13 -0400 Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DAE4714AF5E; Wed, 18 May 2022 15:47:11 -0700 (PDT) Received: by mail-wr1-x42d.google.com with SMTP id w4so4641597wrg.12; Wed, 18 May 2022 15:47:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=3PjQuKMkTY1kyPsdXkRWcy11RqWiSJraBoDtsiZIFcs=; b=qhYeSW3/QJqh9nsZ+2HKtdpdAxJMptXHaJINqDeoNxIadeGNmM6ybUoYv95RC3c1Hn vMNUFzfQllU7Qxdx09C0m5+EEI2wvZyYZihrf0VhO++tZci7dK/TtyaVTUFH5OGlYfwO Y1uCV1x7O+ofv4zLmpiUjcWFmsGgcaf2wqfinIFj0+bcjvCn/YjbD6aXhlMrXOcaMSgi 70Cb5Q3yksfegz715aegGKK3LOE6CW5GZwMEkQMqZJmOAYxd4YOvnIK32hDM9syw+wx2 Id3J5UjDGcM2obX04kPMGafGWD2FACVBSgfPFKUfiYZ+pqE+KTf7765l6Y5vAEZK9FZw 3zVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=3PjQuKMkTY1kyPsdXkRWcy11RqWiSJraBoDtsiZIFcs=; b=wSgpT6XYqv9MF9y/tKUQYiUL60Um9/9sWGFzv7HiCY6OPHgxXCuve8vy5s3LFogSKJ ageCM2IJWWxgZT0HWi51ImQGXe2reGsOAaq9nml12iXx+anQDrtEBIG2L1xOjlEVoFVl MRfd4egX+Cz/ICBMvItByPLCuOQa6+yb12M4qbfDq+kmAOcg/LE+3jjcTCCLpVo3MEUs w04Z6Y0hkEA4wmSyvotmlGDihwITf+9sgKeO15oZobVQVjzrfyDNTQXs/VsncwLZtZyJ 7c/nvNUbEeWqw1xDPZXSfQ3jUS0zhpdHomYeeJsi9fFgYNCbcjT5+PMWlL+3zgi60++x AEwQ== X-Gm-Message-State: AOAM53105WKVWgk72J4ZRr8uU+bQdKpwAOQaCEpXTPQ1J8rU0HJJR1qJ Fafmy5Q/LIgxnqa43dqv/AA= X-Received: by 2002:a05:6000:1787:b0:20c:64d4:d564 with SMTP id e7-20020a056000178700b0020c64d4d564mr1435987wrg.569.1652914030398; Wed, 18 May 2022 15:47:10 -0700 (PDT) Received: from localhost.localdomain ([94.73.36.128]) by smtp.gmail.com with ESMTPSA id bi22-20020a05600c3d9600b003942a244f33sm5223419wmb.12.2022.05.18.15.47.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 May 2022 15:47:10 -0700 (PDT) From: =?UTF-8?q?Jos=C3=A9=20Exp=C3=B3sito?= To: jikos@kernel.org Cc: benjamin.tissoires@redhat.com, spbnick@gmail.com, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, stefanberzl@gmail.com, =?UTF-8?q?Jos=C3=A9=20Exp=C3=B3sito?= Subject: [PATCH for-5.19/uclogic v2 1/4] HID: uclogic: Make template placeholder IDs generic Date: Thu, 19 May 2022 00:46:59 +0200 Message-Id: <20220518224702.1409327-2-jose.exposito89@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220518224702.1409327-1-jose.exposito89@gmail.com> References: <20220518224702.1409327-1-jose.exposito89@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 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 Up until now, the report descriptor template parameter IDs were only used with pen report descriptors and they were named accordingly. Rename the enum and the total number of IDs to make them interface agnostic. Refactor, no functional changes. Signed-off-by: José Expósito --- drivers/hid/hid-uclogic-params.c | 4 ++-- drivers/hid/hid-uclogic-rdesc.c | 14 +++++++------- drivers/hid/hid-uclogic-rdesc.h | 10 +++++----- 3 files changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/hid/hid-uclogic-params.c b/drivers/hid/hid-uclogic-params.c index db838f16282d..b43142f98a8b 100644 --- a/drivers/hid/hid-uclogic-params.c +++ b/drivers/hid/hid-uclogic-params.c @@ -234,7 +234,7 @@ static int uclogic_params_pen_init_v1(struct uclogic_params_pen *pen, const int len = 12; s32 resolution; /* Pen report descriptor template parameters */ - s32 desc_params[UCLOGIC_RDESC_PEN_PH_ID_NUM]; + s32 desc_params[UCLOGIC_RDESC_PH_ID_NUM]; __u8 *desc_ptr = NULL; /* Check arguments */ @@ -379,7 +379,7 @@ static int uclogic_params_pen_init_v2(struct uclogic_params_pen *pen, size_t i; s32 resolution; /* Pen report descriptor template parameters */ - s32 desc_params[UCLOGIC_RDESC_PEN_PH_ID_NUM]; + s32 desc_params[UCLOGIC_RDESC_PH_ID_NUM]; __u8 *desc_ptr = NULL; /* Check arguments */ diff --git a/drivers/hid/hid-uclogic-rdesc.c b/drivers/hid/hid-uclogic-rdesc.c index 13f9ce73f1b1..7126fba80968 100644 --- a/drivers/hid/hid-uclogic-rdesc.c +++ b/drivers/hid/hid-uclogic-rdesc.c @@ -979,7 +979,7 @@ const size_t uclogic_rdesc_xppen_deco01_frame_size = * uclogic_rdesc_template_apply() - apply report descriptor parameters to a * report descriptor template, creating a report descriptor. Copies the * template over to the new report descriptor and replaces every occurrence of - * UCLOGIC_RDESC_PH_HEAD, followed by an index byte, with the value from the + * UCLOGIC_RDESC_PEN_PH_HEAD, followed by an index byte, with the value from the * parameter list at that index. * * @template_ptr: Pointer to the template buffer. @@ -996,7 +996,7 @@ __u8 *uclogic_rdesc_template_apply(const __u8 *template_ptr, const s32 *param_list, size_t param_num) { - static const __u8 head[] = {UCLOGIC_RDESC_PH_HEAD}; + static const __u8 pen_head[] = {UCLOGIC_RDESC_PEN_PH_HEAD}; __u8 *rdesc_ptr; __u8 *p; s32 v; @@ -1005,12 +1005,12 @@ __u8 *uclogic_rdesc_template_apply(const __u8 *template_ptr, if (rdesc_ptr == NULL) return NULL; - for (p = rdesc_ptr; p + sizeof(head) < rdesc_ptr + template_size;) { - if (memcmp(p, head, sizeof(head)) == 0 && - p[sizeof(head)] < param_num) { - v = param_list[p[sizeof(head)]]; + for (p = rdesc_ptr; p + sizeof(pen_head) < rdesc_ptr + template_size;) { + if (memcmp(p, pen_head, sizeof(pen_head)) == 0 && + p[sizeof(pen_head)] < param_num) { + v = param_list[p[sizeof(pen_head)]]; put_unaligned(cpu_to_le32(v), (s32 *)p); - p += sizeof(head) + 1; + p += sizeof(pen_head) + 1; } else { p++; } diff --git a/drivers/hid/hid-uclogic-rdesc.h b/drivers/hid/hid-uclogic-rdesc.h index 0c6e95e8bde7..9d37090c39d1 100644 --- a/drivers/hid/hid-uclogic-rdesc.h +++ b/drivers/hid/hid-uclogic-rdesc.h @@ -81,7 +81,7 @@ extern __u8 uclogic_rdesc_twha60_fixed1_arr[]; extern const size_t uclogic_rdesc_twha60_fixed1_size; /* Report descriptor template placeholder head */ -#define UCLOGIC_RDESC_PH_HEAD 0xFE, 0xED, 0x1D +#define UCLOGIC_RDESC_PEN_PH_HEAD 0xFE, 0xED, 0x1D /* Apply report descriptor parameters to a report descriptor template */ extern __u8 *uclogic_rdesc_template_apply(const __u8 *template_ptr, @@ -89,19 +89,19 @@ extern __u8 *uclogic_rdesc_template_apply(const __u8 *template_ptr, const s32 *param_list, size_t param_num); -/* Pen report descriptor template placeholder IDs */ -enum uclogic_rdesc_pen_ph_id { +/* Report descriptor template placeholder IDs */ +enum uclogic_rdesc_ph_id { UCLOGIC_RDESC_PEN_PH_ID_X_LM, UCLOGIC_RDESC_PEN_PH_ID_X_PM, UCLOGIC_RDESC_PEN_PH_ID_Y_LM, UCLOGIC_RDESC_PEN_PH_ID_Y_PM, UCLOGIC_RDESC_PEN_PH_ID_PRESSURE_LM, - UCLOGIC_RDESC_PEN_PH_ID_NUM + UCLOGIC_RDESC_PH_ID_NUM }; /* Report descriptor pen template placeholder */ #define UCLOGIC_RDESC_PEN_PH(_ID) \ - UCLOGIC_RDESC_PH_HEAD, UCLOGIC_RDESC_PEN_PH_ID_##_ID + UCLOGIC_RDESC_PEN_PH_HEAD, UCLOGIC_RDESC_PEN_PH_ID_##_ID /* Report ID for v1 pen reports */ #define UCLOGIC_RDESC_V1_PEN_ID 0x07 -- 2.25.1