From: Benjamin Tissoires <benjamin.tissoires@redhat.com>
To: Jiri Kosina <jkosina@suse.cz>,
Nestor Lopez Casado <nlopezcasad@logitech.com>,
Andrew de los Reyes <andrew-vger@gizmolabs.org>
Cc: linux-kernel@vger.kernel.org, linux-input@vger.kernel.org
Subject: [PATCH 00/13] HID: add support of Logitech touchpads and special devices
Date: Tue, 30 Sep 2014 13:18:22 -0400 [thread overview]
Message-ID: <1412097515-17241-1-git-send-email-benjamin.tissoires@redhat.com> (raw)
Hi,
Well this patch series is really an old guy:
- I started on this back in 2011 during my short appearance at Logitech
- it has been updated a few time since
- part of it is already shipped in ChromeOS
- now is the time to push it upstream (hopefully)
So, basically, this patch series aims at supporting the touchpads made by
Logitech (i.e. report the raw multi touch events), plus it also gives a driver
to enable special "features" of other various Logitech devices as reported
here[1].
There are couple of changes I introduced in this submission compared to what
Logitech and ChromeOS used to see:
- there will be only one driver for HID++ devices: hid-logitech-hidpp (this way,
we are trying to stay in the line of the HID subsystem, one HID driver per
vendor)
- hid-logitech-dj only deals with the transport layer, so it does not depend on
hid-logitech-hidpp
- there is no more callbacks mechanism in hidpp and the specific drivers, all
is handled in one place
- there is no need of a blacklist in hid-logitech-dj to decide if the device has
a subdriver or not.
- there is no support of the mice with raw touch. This can be added in a future
patch series.
I am finally finding this ready to be sent upstream, so I hope some of you would
be kind enough to review the code.
Regarding the features, here is what end user will expect/see:
- all the DJ devices (connected through the unifying wirelees receiver) have a
proper name now (T400, M325, ...). This name is retrieved from the receiver
which has a short buffer for it, thus the short name.
- the touchpads input devices will be created once the device is actually
connected. For bluetooth device, it's a normal HID process. For DJ devices,
the receiver is created, the hidraw device of the touchpad is created, but the
input device, with the full name (asked from the device) appears on the first
connection (or events).
- The TK820 - keyboard/touchpad combo presents now 2 input devices. One for the
keyboard part, one for the touchpad. So X and libinput can deal with it
easily.
Jiri, This patch series can be split in 2 if you would like:
- patches 1 to 4 can be scheduled easily for 3.18 IMO
- patches 5 to 13 might need some more reviews, so either 3.18 or 3.19
Cheers,
Benjamin
[1] https://www.mail-archive.com/linux-input@vger.kernel.org/msg11831.html
Benjamin Tissoires (13):
HID: fix merge from wacom into the HID tree
HID: core: do not scan reports if the group is already set
HID: logitech-dj: rely on hid groups to separate receivers from dj
devices
HID: logitech-dj: merge header file into the source
HID: Introduce hidpp, a module to handle Logitech hid++ devices
HID: logitech: move dj devices to the HID++ module
HID: logitech-dj: allow transfer of HID++ reports from/to the correct
dj device
HID: logitech: allow the DJ device to request the unifying name
HID: logitech-dj: enable notifications on connect/disconnect
HID: logitech-hidpp: late bind the input device on wireless connection
HID: logitech-hidpp: Add Wireless Touchpad T650 support
HID: logitech-hidpp: add support of the first Logitech Wireless
Touchpad
HID: logitech-hidpp: support combo keyboard touchpad TK820
drivers/hid/Kconfig | 12 +
drivers/hid/Makefile | 1 +
drivers/hid/hid-core.c | 21 +-
drivers/hid/hid-ids.h | 1 +
drivers/hid/hid-logitech-dj.c | 397 ++++++++++---
drivers/hid/hid-logitech-dj.h | 125 ----
drivers/hid/hid-logitech-hidpp.c | 1201 ++++++++++++++++++++++++++++++++++++++
include/linux/hid.h | 5 +-
8 files changed, 1526 insertions(+), 237 deletions(-)
delete mode 100644 drivers/hid/hid-logitech-dj.h
create mode 100644 drivers/hid/hid-logitech-hidpp.c
--
2.1.0
next reply other threads:[~2014-09-30 17:18 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-30 17:18 Benjamin Tissoires [this message]
2014-09-30 17:18 ` [PATCH 01/13] HID: fix merge from wacom into the HID tree Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 02/13] HID: core: do not scan reports if the group is already set Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 03/13] HID: logitech-dj: rely on hid groups to separate receivers from dj devices Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 04/13] HID: logitech-dj: merge header file into the source Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 05/13] HID: Introduce hidpp, a module to handle Logitech hid++ devices Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 06/13] HID: logitech: move dj devices to the HID++ module Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 07/13] HID: logitech-dj: allow transfer of HID++ reports from/to the correct dj device Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 08/13] HID: logitech: allow the DJ device to request the unifying name Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 09/13] HID: logitech-dj: enable notifications on connect/disconnect Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 10/13] HID: logitech-hidpp: late bind the input device on wireless connection Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 11/13] HID: logitech-hidpp: Add Wireless Touchpad T650 support Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 12/13] HID: logitech-hidpp: add support of the first Logitech Wireless Touchpad Benjamin Tissoires
2014-09-30 17:18 ` [PATCH 13/13] HID: logitech-hidpp: support combo keyboard touchpad TK820 Benjamin Tissoires
2014-10-01 8:17 ` [PATCH 00/13] HID: add support of Logitech touchpads and special devices Jiri Kosina
2014-10-01 13:10 ` Benjamin Tissoires
2014-10-02 18:19 ` Andrew de los Reyes
2014-10-03 11:09 ` Jiri Kosina
2014-10-03 14:44 ` Andrew de los Reyes
2014-10-28 20:05 ` Benjamin Tissoires
2014-10-29 9:49 ` Jiri Kosina
2014-10-29 14:41 ` Benjamin Tissoires
2014-10-29 16:14 ` Jiri Kosina
2014-10-29 18:14 ` Benjamin Tissoires
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1412097515-17241-1-git-send-email-benjamin.tissoires@redhat.com \
--to=benjamin.tissoires@redhat.com \
--cc=andrew-vger@gizmolabs.org \
--cc=jkosina@suse.cz \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nlopezcasad@logitech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®