linux-kernel - Re: [PATCH v4] HID: cmedia: add support for HS-100B mute button

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <nycvar.YFH.7.76.2107281151280.8253@cbobk.fhfr.pm>
Date:   Wed, 28 Jul 2021 11:51:36 +0200 (CEST)
From:   Jiri Kosina <jikos@...nel.org>
To:     Thomas Weißschuh <linux@...ssschuh.net>
cc:     Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        linux-input@...r.kernel.org, Ben Chen <ben_chen@...linktech.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4] HID: cmedia: add support for HS-100B mute button

On Tue, 20 Jul 2021, Thomas Weißschuh wrote:

> These chips report mute button events in bit 4 of their report without it
> being part of the report descriptor.
> Use a custom descriptor that maps this bit.
> 
> Signed-off-by: Thomas Weißschuh <linux@...ssschuh.net>
> ---
> 
> v1: https://lore.kernel.org/linux-input/a769ae40-6d0c-47c4-803f-2c8dbc362f24@t-8ch.de/
> 
> v1 -> v2:
>  * Merged into the existing cmedia driver instead of creating a dedicated
>    driver.
> 
> v2: https://lore.kernel.org/linux-input/20210716160659.154779-1-linux@weissschuh.net/
> 
> v2 -> v3:
>  * Fixed cosmetic checkpatch warnings
>  * CC-ed Ben Chen who is the original author of hid-cmedia.c
> 
> v3: https://lore.kernel.org/linux-input/20210719110911.310701-1-linux@weissschuh.net/
> 
> v3 -> v4:
>  * Create a dedicated struct hid_driver, leaving the existing intact
>  * Drop entry in hid-quirks.c

Applied, thanks Thomas.

-- 
Jiri Kosina
SUSE Labs