From patchwork Wed Jul 13 16:06:06 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Benjamin Tissoires X-Patchwork-Id: 9227921 X-Patchwork-Delegate: jikos@jikos.cz Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 1C5276088F for ; Wed, 13 Jul 2016 16:13:45 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 0D9FD27F9E for ; Wed, 13 Jul 2016 16:13:45 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 01F3827FA7; Wed, 13 Jul 2016 16:13:44 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8E6882621F for ; Wed, 13 Jul 2016 16:13:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752455AbcGMQHR (ORCPT ); Wed, 13 Jul 2016 12:07:17 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44251 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752410AbcGMQHM (ORCPT ); Wed, 13 Jul 2016 12:07:12 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id F2FEF7DD02; Wed, 13 Jul 2016 16:06:57 +0000 (UTC) Received: from plouf.banquise.eu.com (ovpn-116-194.phx2.redhat.com [10.3.116.194]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u6DG6JwO004508; Wed, 13 Jul 2016 12:06:56 -0400 From: Benjamin Tissoires To: Jiri Kosina , Ping Cheng , Jason Gerecke , Aaron Skomra , Peter Hutterer Cc: linux-kernel@vger.kernel.org, linux-input@vger.kernel.org Subject: [PATCH v2 19/30] HID: wacom: EKR: have one array of struct remotes instead of many arrays Date: Wed, 13 Jul 2016 18:06:06 +0200 Message-Id: <1468425977-4819-20-git-send-email-benjamin.tissoires@redhat.com> In-Reply-To: <1468425977-4819-1-git-send-email-benjamin.tissoires@redhat.com> References: <1468425977-4819-1-git-send-email-benjamin.tissoires@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Wed, 13 Jul 2016 16:06:58 +0000 (UTC) Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP No functional changes, just a prep patch for the one after. Signed-off-by: Benjamin Tissoires --- No changes in v2 --- drivers/hid/wacom.h | 6 ++++-- drivers/hid/wacom_sys.c | 53 +++++++++++++++++++++++++------------------------ drivers/hid/wacom_wac.c | 2 +- 3 files changed, 32 insertions(+), 29 deletions(-) diff --git a/drivers/hid/wacom.h b/drivers/hid/wacom.h index 110ea67..6b8df67 100644 --- a/drivers/hid/wacom.h +++ b/drivers/hid/wacom.h @@ -120,8 +120,10 @@ struct wacom_remote { spinlock_t remote_lock; struct kfifo remote_fifo; struct kobject *remote_dir; - struct attribute_group remote_group[WACOM_MAX_REMOTES]; - __u32 serial[WACOM_MAX_REMOTES]; + struct { + struct attribute_group group; + u32 serial; + } remotes[WACOM_MAX_REMOTES]; }; struct wacom { diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c index 9121e72..c3b2692 100644 --- a/drivers/hid/wacom_sys.c +++ b/drivers/hid/wacom_sys.c @@ -1308,16 +1308,16 @@ static int wacom_remote_create_attr_group(struct wacom *wacom, __u32 serial, int error = 0; struct wacom_remote *remote = wacom->remote; - remote->remote_group[index].name = devm_kasprintf(&wacom->hdev->dev, + remote->remotes[index].group.name = devm_kasprintf(&wacom->hdev->dev, GFP_KERNEL, "%d", serial); - if (!remote->remote_group[index].name) + if (!remote->remotes[index].group.name) return -ENOMEM; error = __wacom_devm_sysfs_create_group(wacom, remote->remote_dir, - &remote->remote_group[index]); + &remote->remotes[index].group); if (error) { - remote->remote_group[index].name = NULL; + remote->remotes[index].group.name = NULL; hid_err(wacom->hdev, "cannot create sysfs group err: %d\n", error); return error; @@ -1421,11 +1421,11 @@ static int wacom_initialize_remotes(struct wacom *wacom) return -ENOMEM; } - remote->remote_group[0] = remote0_serial_group; - remote->remote_group[1] = remote1_serial_group; - remote->remote_group[2] = remote2_serial_group; - remote->remote_group[3] = remote3_serial_group; - remote->remote_group[4] = remote4_serial_group; + remote->remotes[0].group = remote0_serial_group; + remote->remotes[1].group = remote1_serial_group; + remote->remotes[2].group = remote2_serial_group; + remote->remotes[3].group = remote3_serial_group; + remote->remotes[4].group = remote4_serial_group; remote->remote_dir = kobject_create_and_add("wacom_remote", &wacom->hdev->dev.kobj); @@ -1442,7 +1442,7 @@ static int wacom_initialize_remotes(struct wacom *wacom) for (i = 0; i < WACOM_MAX_REMOTES; i++) { wacom->led.groups[i].select = WACOM_STATUS_UNKNOWN; - remote->serial[i] = 0; + remote->remotes[i].serial = 0; } error = devm_add_action_or_reset(&wacom->hdev->dev, @@ -1903,16 +1903,17 @@ fail: static void wacom_remote_destroy_one(struct wacom *wacom, unsigned int index) { struct wacom_remote *remote = wacom->remote; - u32 serial = remote->serial[index]; + u32 serial = remote->remotes[index].serial; int i; - if (remote->remote_group[index].name) - devres_release_group(&wacom->hdev->dev, &remote->serial[index]); + if (remote->remotes[index].group.name) + devres_release_group(&wacom->hdev->dev, + &remote->remotes[index]); for (i = 0; i < WACOM_MAX_REMOTES; i++) { - if (remote->serial[i] == serial) { - remote->serial[i] = 0; - remote->remote_group[i].name = NULL; + if (remote->remotes[i].serial == serial) { + remote->remotes[i].serial = 0; + remote->remotes[i].group.name = NULL; wacom->led.groups[i].select = WACOM_STATUS_UNKNOWN; } } @@ -1929,30 +1930,30 @@ static int wacom_remote_create_one(struct wacom *wacom, u32 serial, * check to make sure this serial isn't already paired. */ for (k = 0; k < WACOM_MAX_REMOTES; k++) { - if (remote->serial[k] == serial) + if (remote->remotes[k].serial == serial) break; } if (k < WACOM_MAX_REMOTES) { - remote->serial[index] = serial; + remote->remotes[index].serial = serial; return 0; } - if (!devres_open_group(dev, &remote->serial[index], GFP_KERNEL)) + if (!devres_open_group(dev, &remote->remotes[index], GFP_KERNEL)) return -ENOMEM; error = wacom_remote_create_attr_group(wacom, serial, index); if (error) goto fail; - remote->serial[index] = serial; + remote->remotes[index].serial = serial; - devres_close_group(dev, &remote->serial[index]); + devres_close_group(dev, &remote->remotes[index]); return 0; fail: - devres_release_group(dev, &remote->serial[index]); - remote->serial[index] = 0; + devres_release_group(dev, &remote->remotes[index]); + remote->remotes[index].serial = 0; return error; } @@ -1986,15 +1987,15 @@ static void wacom_remote_work(struct work_struct *work) serial = data.remote[i].serial; if (data.remote[i].connected) { - if (remote->serial[i] == serial) + if (remote->remotes[i].serial == serial) continue; - if (remote->serial[i]) + if (remote->remotes[i].serial) wacom_remote_destroy_one(wacom, i); wacom_remote_create_one(wacom, serial, i); - } else if (remote->serial[i]) { + } else if (remote->remotes[i].serial) { wacom_remote_destroy_one(wacom, i); } } diff --git a/drivers/hid/wacom_wac.c b/drivers/hid/wacom_wac.c index ce1089f..aee0761 100644 --- a/drivers/hid/wacom_wac.c +++ b/drivers/hid/wacom_wac.c @@ -808,7 +808,7 @@ static int wacom_remote_irq(struct wacom_wac *wacom_wac, size_t len) touch_ring_mode = (data[11] & 0xC0) >> 6; for (i = 0; i < WACOM_MAX_REMOTES; i++) { - if (remote->serial[i] == serial) + if (remote->remotes[i].serial == serial) wacom->led.groups[i].select = touch_ring_mode; }