linux-kernel - Re: [PATCH v2 0/1] VRC-2 Car Controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAO-hwJLOcFkts2wuMvr79w81XENEb9XyAXESciOgUiqSwG2pBw@mail.gmail.com>
Date:   Tue, 20 Sep 2022 15:34:15 +0200
From:   Benjamin Tissoires <benjamin.tissoires@...hat.com>
To:     Marcus Folkesson <marcus.folkesson@...il.com>
Cc:     Jiri Kosina <jikos@...nel.org>,
        "open list:HID CORE LAYER" <linux-input@...r.kernel.org>,
        lkml <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/1] VRC-2 Car Controller

On Fri, Sep 2, 2022 at 10:20 AM Marcus Folkesson
<marcus.folkesson@...il.com> wrote:
>
> Hi,
>
> The device creates two USB endpoints where on of those represent the X/Y
> axis as 16 buttons which is not.. quite right.
> So get rid of that endpoint.
>
> Also fix the report descriptor for the other endpoint.
>
> Output from hid-recorder:
> #  X:     32 | Y:   1001 | #
> E: 000105.808008 7 20 00 e9 03 00 00 00
>
> Link to previous discussion:
> https://lore.kernel.org/all/CAO-hwJL-3vAzywjeUsopsRSJX6j-maC5R75ekxZg-W_oKDqYYw@mail.gmail.com/
>
>
> v2:
> - Remove .remove
> - Change help text in Kconfig
> - Remove redundant check for USB endpoint in vrc2_report_fixup
>
> Marcus Folkesson (1):
>   HID: Add driver for VRC-2 Car Controller
>
>  MAINTAINERS            |  6 +++
>  drivers/hid/Kconfig    | 10 +++++
>  drivers/hid/Makefile   |  1 +
>  drivers/hid/hid-vrc2.c | 91 ++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 108 insertions(+)
>  create mode 100644 drivers/hid/hid-vrc2.c
>

Applied to for-6.1/rc-controllers in hid.git

Thanks a lot for this work!

Cheers,
Benjamin