mbox series

[v4,0/8] XP-PEN Deco Pro S support

Message ID 20220815142956.19180-1-jose.exposito89@gmail.com (mailing list archive)
Headers show
Series XP-PEN Deco Pro S support | expand

Message

José Expósito Aug. 15, 2022, 2:29 p.m. UTC 
Hello everyone,

This is v4 of [1]. Check the description in the link for more
information, please.

v1 -> v2:

 - First patch acked-by Daniel Latypov

v2 -> v3:

 - Fix bug in "HID: uclogic: Add support for UGEE v2 mouse frames".
   Reported-by: kernel test robot <lkp@intel.com>

   I copy-pasted the wrong template size.
   Thanks to Nathan Chancellor for looking into it. As he mentioned [2],
   CONFIG_FORTIFY_SOURCE doesn't catch this error without LTO enabled.

v3 -> v4:

 - Rebase on hid/master after the merge window.

 - Add and extra patch to add support for Parblo A610 PRO. The tablet
   is similar to the XP-PEN Deco Pro S and adding its ID is enough to
   support it.

 - Minimal cleanup in the return branch of uclogic_params_ugee_v2_init_frame_mouse
   and uclogic_params_ugee_v2_init_frame_buttons.

[1] https://lore.kernel.org/linux-input/20220717144333.251190-1-jose.exposito89@gmail.com/T/
[2] https://lore.kernel.org/linux-input/20220717144333.251190-1-jose.exposito89@gmail.com/T/#m796ac6c8f7484b0bafc1f1>

José Expósito (8):
  HID: uclogic: KUnit best practices and naming conventions
  HID: uclogic: Refactor UGEE v2 string descriptor parsing
  HID: uclogic: Refactor UGEE v2 frame initialization
  HID: uclogic: Parse the UGEE v2 frame type
  HID: uclogic: Add support for UGEE v2 dial frames
  HID: uclogic: Add support for UGEE v2 mouse frames
  HID: uclogic: Add support for XP-PEN Deco Pro S
  HID: uclogic: Add support for Parblo A610 PRO

 drivers/hid/Kconfig                   |   2 +-
 drivers/hid/Makefile                  |   4 +-
 drivers/hid/hid-ids.h                 |   2 +
 drivers/hid/hid-uclogic-core.c        |   4 +
 drivers/hid/hid-uclogic-params-test.c | 192 +++++++++++++++++++++
 drivers/hid/hid-uclogic-params.c      | 230 +++++++++++++++++++++-----
 drivers/hid/hid-uclogic-params.h      |  10 ++
 drivers/hid/hid-uclogic-rdesc-test.c  |  22 +--
 drivers/hid/hid-uclogic-rdesc.c       |  74 +++++++++
 drivers/hid/hid-uclogic-rdesc.h       |   8 +
 10 files changed, 498 insertions(+), 50 deletions(-)
 create mode 100644 drivers/hid/hid-uclogic-params-test.c

Comments

Jiri Kosina Aug. 25, 2022, 8:28 a.m. UTC | #1 
On Mon, 15 Aug 2022, José Expósito wrote:

> Hello everyone,
> 
> This is v4 of [1]. Check the description in the link for more
> information, please.
> 
> v1 -> v2:
> 
>  - First patch acked-by Daniel Latypov
> 
> v2 -> v3:
> 
>  - Fix bug in "HID: uclogic: Add support for UGEE v2 mouse frames".
>    Reported-by: kernel test robot <lkp@intel.com>
> 
>    I copy-pasted the wrong template size.
>    Thanks to Nathan Chancellor for looking into it. As he mentioned [2],
>    CONFIG_FORTIFY_SOURCE doesn't catch this error without LTO enabled.
> 
> v3 -> v4:
> 
>  - Rebase on hid/master after the merge window.
> 
>  - Add and extra patch to add support for Parblo A610 PRO. The tablet
>    is similar to the XP-PEN Deco Pro S and adding its ID is enough to
>    support it.
> 
>  - Minimal cleanup in the return branch of uclogic_params_ugee_v2_init_frame_mouse
>    and uclogic_params_ugee_v2_init_frame_buttons.
> 
> [1] https://lore.kernel.org/linux-input/20220717144333.251190-1-jose.exposito89@gmail.com/T/
> [2] https://lore.kernel.org/linux-input/20220717144333.251190-1-jose.exposito89@gmail.com/T/#m796ac6c8f7484b0bafc1f1>
> 
> José Expósito (8):
>   HID: uclogic: KUnit best practices and naming conventions
>   HID: uclogic: Refactor UGEE v2 string descriptor parsing
>   HID: uclogic: Refactor UGEE v2 frame initialization
>   HID: uclogic: Parse the UGEE v2 frame type
>   HID: uclogic: Add support for UGEE v2 dial frames
>   HID: uclogic: Add support for UGEE v2 mouse frames
>   HID: uclogic: Add support for XP-PEN Deco Pro S
>   HID: uclogic: Add support for Parblo A610 PRO

Now in hid.git#for-6.1/uclogic. Thanks,