[0/7] DIGImend patches, part one

Message

José Expósito Feb. 10, 2022, 7:04 p.m. UTC

Hi everyone,

I'm working on the UC Logic driver to add support for my tablet.
After some work, I noticed that I needed some patches from the
DIGImend project [1].

Instead of cherry picking what I needed, I decided to go the hard
way and upstream every change present in DIGImend but missing in
the kernel with the intention of making development easier in the
future for everyone and providing better hardware support.

In a private conversation with DIGImend's maintainer, Nikolai
Kondrashov, I asked him for permission to upstream his changes and
he said it was fine :) All credit goes to him, I only fixed his
patches so they apply, fixed minor checkpatch errors, squashed
related changes and reviewed and tested them when my hardware
allowed me to.

This is the first series of patches from DIGImend. There are 37
patches in total [2], but I can imagine nobody wants to review such
a long series, so I'll be sending small groups of related patches...
Unless someone feels brave and tells me to send all of them ;)

Thank you very much in advance,
José Expósito

[1] https://github.com/DIGImend/digimend-kernel-drivers
[2] https://github.com/JoseExposito/linux/commits/patch-digimend-parblo-patches-mailing-list

Nikolai Kondrashov (7):
  HID: uclogic: Support Huion tilt reporting
  HID: uclogic: Rename Huion HS64 PID to Huion Tablet 2
  HID: uclogic: Support Huion 13th frame button
  HID: uclogic: Split pen and frame raw event handling
  HID: uclogic: Access pen/frame params directly in raw_event handling
  HID: uclogic: Skip non-input raw events earlier
  HID: uclogic: Handle virtual frame reports

 drivers/hid/hid-ids.h            |   2 +-
 drivers/hid/hid-uclogic-core.c   | 205 +++++++++++++++++++------------
 drivers/hid/hid-uclogic-params.c |   3 +-
 drivers/hid/hid-uclogic-params.h |   8 ++
 drivers/hid/hid-uclogic-rdesc.c  |  20 ++-
 5 files changed, 153 insertions(+), 85 deletions(-)

Comments

Jiri Kosina Feb. 16, 2022, 3:43 p.m. UTC | #1

On Thu, 10 Feb 2022, José Expósito wrote:

> I'm working on the UC Logic driver to add support for my tablet.
> After some work, I noticed that I needed some patches from the
> DIGImend project [1].
> 
> Instead of cherry picking what I needed, I decided to go the hard
> way and upstream every change present in DIGImend but missing in
> the kernel with the intention of making development easier in the
> future for everyone and providing better hardware support.
> 
> In a private conversation with DIGImend's maintainer, Nikolai
> Kondrashov, I asked him for permission to upstream his changes and
> he said it was fine :) All credit goes to him, I only fixed his
> patches so they apply, fixed minor checkpatch errors, squashed
> related changes and reviewed and tested them when my hardware
> allowed me to.
> 
> This is the first series of patches from DIGImend. There are 37
> patches in total [2], but I can imagine nobody wants to review such
> a long series, so I'll be sending small groups of related patches...

Thanks a lot for consideration :)

> Unless someone feels brave and tells me to send all of them ;)

Definitely not me :)

The patches look good to me, and I see Nikolai is CCed, so I've now queued 
them in hid.git#for-5.18/uclogic

Thanks,