From: Benjamin Tissoires <benjamin.tissoires@redhat.com>
To: Benjamin Tissoires <benjamin.tissoires@gmail.com>,
Jiri Kosina <jkosina@suse.cz>,
David Herrmann <dh.herrmann@gmail.com>,
Frank Praznik <frank.praznik@oh.rr.com>,
linux-input@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 00/11] HID: spring cleaning
Date: Sat, 1 Feb 2014 23:50:19 -0500 [thread overview]
Message-ID: <1391316630-29541-1-git-send-email-benjamin.tissoires@redhat.com> (raw)
Hi guys,
This is an attempt to complete the branch for-3.15/ll-driver-new-callbacks:
- try to implement as much as possible ll_driver callbacks (some are still
missing, but I did not had the time to complete it)
- add inliners hid_hw_* for all the ll_driver callbacks
- remove transport dependant callbacks in struct hid_device
- remove the so called "obsolete" hidinput_input_event handler which was used
in 2/4 transport drivers
Bonus point: 14 files changed, 213 insertions(+), 272 deletions(-)
Yay!
I made sure I kept the PS3 controller working and the LEDs (on logitech-dj and
on bluetooth keyboard) working. The rest do not mostly need further testing,
the code path did not changed. But still, a review (and some tests) would be a
good idea :)
Cheers,
Benjamin
PS: obviously, this goes on top on the branch for-3.15/ll-driver-new-callbacks
Benjamin Tissoires (11):
HID: uHID: implement .raw_request
HID: i2c-hid: implement ll_driver transport-layer callbacks
HID: add inliners for ll_driver transport-layer callbacks
HID: logitech-dj: remove hidinput_input_event
HID: HIDp: remove hidp_hidinput_event
HID: remove hidinput_input_event handler
HID: HIDp: remove duplicated coded
HID: usbhid: remove duplicated code
HID: remove hid_get_raw_report in struct hid_device
HID: introduce helper to access hid_output_raw_report()
HID: move hid_output_raw_report to hid_ll_driver
drivers/hid/hid-input.c | 12 ++---
drivers/hid/hid-lg.c | 6 ++-
drivers/hid/hid-logitech-dj.c | 101 +++++++++++++---------------------
drivers/hid/hid-magicmouse.c | 2 +-
drivers/hid/hid-sony.c | 19 +++++--
drivers/hid/hid-thingm.c | 4 +-
drivers/hid/hid-wacom.c | 16 +++---
drivers/hid/hid-wiimote-core.c | 4 +-
drivers/hid/hidraw.c | 11 ++--
drivers/hid/i2c-hid/i2c-hid.c | 27 +++++++++-
drivers/hid/uhid.c | 20 ++++++-
drivers/hid/usbhid/hid-core.c | 67 +++++------------------
include/linux/hid.h | 77 ++++++++++++++++++++++----
net/bluetooth/hidp/core.c | 119 +++++------------------------------------
14 files changed, 213 insertions(+), 272 deletions(-)
--
1.8.3.1
next reply other threads:[~2014-02-02 4:50 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-02 4:50 Benjamin Tissoires [this message]
2014-02-02 4:50 ` [PATCH 01/11] HID: uHID: implement .raw_request Benjamin Tissoires
2014-02-03 15:26 ` David Herrmann
2014-02-03 19:07 ` Benjamin Tissoires
2014-02-02 4:50 ` [PATCH 02/11] HID: i2c-hid: implement ll_driver transport-layer callbacks Benjamin Tissoires
2014-02-03 16:04 ` David Herrmann
2014-02-03 19:00 ` Benjamin Tissoires
2014-02-02 4:50 ` [PATCH 03/11] HID: add inliners for " Benjamin Tissoires
2014-02-03 15:29 ` David Herrmann
2014-02-02 4:50 ` [PATCH 04/11] HID: logitech-dj: remove hidinput_input_event Benjamin Tissoires
2014-02-03 16:06 ` David Herrmann
2014-02-03 16:21 ` Benjamin Tissoires
2014-02-03 17:07 ` David Herrmann
2014-02-02 4:50 ` [PATCH 05/11] HID: HIDp: remove hidp_hidinput_event Benjamin Tissoires
2014-02-03 15:10 ` David Herrmann
2014-02-03 15:28 ` Benjamin Tissoires
2014-02-02 4:50 ` [PATCH 06/11] HID: remove hidinput_input_event handler Benjamin Tissoires
2014-02-03 16:10 ` David Herrmann
2014-02-02 4:50 ` [PATCH 07/11] HID: HIDp: remove duplicated coded Benjamin Tissoires
2014-02-03 15:02 ` David Herrmann
2014-02-03 15:11 ` Benjamin Tissoires
2014-02-03 15:19 ` David Herrmann
2014-02-02 4:50 ` [PATCH 08/11] HID: usbhid: remove duplicated code Benjamin Tissoires
2014-02-03 16:09 ` David Herrmann
2014-02-02 4:50 ` [PATCH 09/11] HID: remove hid_get_raw_report in struct hid_device Benjamin Tissoires
2014-02-03 16:12 ` David Herrmann
2014-02-02 4:50 ` [PATCH 10/11] HID: introduce helper to access hid_output_raw_report() Benjamin Tissoires
2014-02-03 16:14 ` David Herrmann
2014-02-02 4:50 ` [PATCH 11/11] HID: move hid_output_raw_report to hid_ll_driver Benjamin Tissoires
2014-02-03 16:48 ` [PATCH 00/11] HID: spring cleaning David Herrmann
2014-02-03 19:19 ` 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=1391316630-29541-1-git-send-email-benjamin.tissoires@redhat.com \
--to=benjamin.tissoires@redhat.com \
--cc=benjamin.tissoires@gmail.com \
--cc=dh.herrmann@gmail.com \
--cc=frank.praznik@oh.rr.com \
--cc=jkosina@suse.cz \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).