Re: [PATCH v11 0/4] Add MSI Claw HID Configuration Driver

[Jiri Kosina]

On Fri, 29 May 2026, Derek J. Clark wrote:

> This series adds an HID Configuration driver for the MSI Claw line of
> Handheld Gaming PC's. The MSI Claw HID interface provides multiple
> features, such as the ability to switch between xinput, dinput, and a
> desktop mode, RGB control, rumble intensity, and mapping of the rear "M"
> keys. There are additional gamepad modes that are not included in this
> driver as they appear to be used in assembly line testing or are
> incomplete in the firmware. During my testing I found them to be unstable.
> 
> The initial version of this driver was written by Denis Benato, which
> contained the initial reverse-engineering and implementation for the
> gamepad mode switching. This work was later expanded by Zhouwang Huang
> to include more gamepad modes and additional features. Finally, I
> refactored the entire driver, fixed multiple bugs, and refined the overall
> format to conform to kernel driver best practices and style guide.
> 
> Claude was used initially by Zhouwang Huang to quickly parse HID captures
> during the reverse-engineering of some of the features. Since Claude had
> already been used, as a test of its capabilities I had it implement the
> rumble intensity attribute after I had already rewritten most of the
> driver, which I then manually edited to fix some mistakes. I also used
> Claude to review the driver and these patches for any mistakes and bugs.
> 
> Assisted-by: Claude:claude-sonnet-4-6
> Co-developed-by: Denis Benato <denis.benato@xxxxxxxxx>
> Signed-off-by: Denis Benato <denis.benato@xxxxxxxxx>
> Co-developed-by: Zhouwang Huang <honjow311@xxxxxxxxx>
> Signed-off-by: Zhouwang Huang <honjow311@xxxxxxxxx>
> Signed-off-by: Derek J. Clark <derekjohn.clark@xxxxxxxxx>
> ---
> v11:
>   - Restore dropped changes from v10.

Thanks Derek.

Sashiko had quite a few insightful comments, I find especially the one 
with IRQ-vs-kernel context deadlock on drvdata->profile_lock valid.

Could you please go through those, and address it?

Thanks again for all the effort,

-- 
Jiri Kosina
SUSE Labs