Convert serio-related drivers to use new cleanup facilities
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest24 patches 2 repliesIn review
Patches in v1 No applies recorded yet
Reviews trailers per patch
1 of 24 patches has a Reviewed-by or Acked-by.
| Patch | Pali Rohár |
|---|---|
| 1/24 | |
| 2/24 | |
| 3/24 | A-b |
| 4/24 | |
| 5/24 | |
| 6/24 | |
| 7/24 | |
| 8/24 | |
| 9/24 | |
| 10/24 | |
| 11/24 | |
| 12/24 | |
| 13/24 | |
| 14/24 | |
| 15/24 | |
| 16/24 | |
| 17/24 | |
| 18/24 | |
| 19/24 | |
| 20/24 | |
| 21/24 | |
| 22/24 | |
| 23/24 | |
| 24/24 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 00/24] Convert serio-related drivers to use new cleanup facilities
Hi,
This series converts drivers found in drivers/input/serio as well as a few of input drivers using serio to use new __free() and guard() cleanup facilities that simplify the code and ensure that all resources are released appropriately.
First patch introduces serio_pause_rx guard that pauses delivery of interrupts/data for a given serio port by calling serio_pause_rx() helper and automatically resumes it by calling serio_resume_rx() when needed.
The following 8 patches make use if this new guard.
The rest of the patches switch serio drivers to use guard(mutex), guard(spinlock*) and other cleanup functions to simplify the code.
Thanks!
Dmitry Torokhov (24):
Input: serio - define serio_pause_rx guard to pause and resume serio ports
Input: libps2 - use guard notation when temporarily pausing serio ports
Input: alps - use guard notation when pausing serio port
Input: byd - use guard notation when pausing serio port
Input: synaptics - use guard notation when pausing serio port
Input: atkbd - use guard notation when pausing serio port
Input: sunkbd - use guard notation when pausing serio port
Input: synaptics-rmi4 - use guard notation when pausing serio port in F03
Input: elo - use guard notation when pausing serio port
Input: gscps2 - use guard notation when acquiring spinlock
Input: hyperv-keyboard - use guard notation when acquiring spinlock
Input: i8042 - tease apart interrupt handler
Input: i8042 - use guard notation when acquiring spinlock
Input: ps2-gpio - use guard notation when acquiring mutex
Input: ps2mult - use guard notation when acquiring spinlock
Input: q40kbd - use guard notation when acquiring spinlock
Input: sa1111ps2 - use guard notation when acquiring spinlock
Input: serport - use guard notation when acquiring spinlock
Input: serio - use guard notation when acquiring mutexes and spinlocks
Input: serio_raw - use guard notation for locks and other resources
Input: serio-raw - fix potential serio port name truncation
Input: sun4i-ps2 - use guard notation when acquiring spinlock
Input: userio - switch to using cleanup functions
Input: xilinx_ps2 - use guard notation when acquiring spinlock
drivers/input/keyboard/atkbd.c | 8 +- drivers/input/keyboard/sunkbd.c | 5 +- drivers/input/mouse/alps.c | 4 +- drivers/input/mouse/byd.c | 5 +- drivers/input/mouse/synaptics.c | 6 +- drivers/input/rmi4/rmi_f03.c | 4 +- drivers/input/serio/gscps2.c | 114 +++++---- drivers/input/serio/hyperv-keyboard.c | 38 ++- drivers/input/serio/i8042.c | 327 +++++++++++++------------- drivers/input/serio/libps2.c | 28 ++- drivers/input/serio/ps2-gpio.c | 4 +- drivers/input/serio/ps2mult.c | 25 +- drivers/input/serio/q40kbd.c | 10 +- drivers/input/serio/sa1111ps2.c | 8 +- drivers/input/serio/serio.c | 164 +++++-------- drivers/input/serio/serio_raw.c | 125 ++++------ drivers/input/serio/serport.c | 27 +-- drivers/input/serio/sun4i-ps2.c | 8 +- drivers/input/serio/userio.c | 141 ++++++----- drivers/input/serio/xilinx_ps2.c | 15 +- drivers/input/touchscreen/elo.c | 8 +- include/linux/serio.h | 3 + 22 files changed, 487 insertions(+), 590 deletions(-)
Signature · 1 line
Dmitry