linux-kernel - Re: [PATCH v5] HID: corsair-void: Add Corsair Void headset family driver

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: <nycvar.YFH.7.76.2410111311431.20286@cbobk.fhfr.pm>
Date: Fri, 11 Oct 2024 13:12:00 +0200 (CEST)
From: Jiri Kosina <jikos@...nel.org>
To: Stuart Hayhurst <stuart.a.hayhurst@...il.com>
cc: linux-input@...r.kernel.org, Benjamin Tissoires <bentiss@...nel.org>, 
    Christophe JAILLET <christophe.jaillet@...adoo.fr>, 
    linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5] HID: corsair-void: Add Corsair Void headset family
 driver

On Wed, 9 Oct 2024, Stuart Hayhurst wrote:

> Introduce a driver for the Corsair Void family of headsets, supporting:
>  - Battery reporting (power_supply)
>  - Sidetone setting support
>  - Physical microphone location reporting
>  - Headset and receiver firmware version reporting
>  - Built-in alert triggering
>  - USB wireless_status
> 
> Tested with a Void Pro Wireless, Void Elite Wireless and a Void Elite Wired
> 
> Signed-off-by: Stuart Hayhurst <stuart.a.hayhurst@...il.com>
> ---
> 
> v4 -> v5:
>  - Changed kernel version in sysfs docs to 6.13
>  - Use CONFIG_HID_CORSAIR instead of CONFIG_HID_CORSAIR_VOID
> 
> v3 -> v4:
>  - Added missing <linux/device.h> include
>  - Added missing newline, fix indent
>  - Use hid_hw_raw_request return code directly for wireless sidetone
>  - Use battery status enum for remaining values
>  - Use of devm_kasprintf() to create battery string
>  - Moved corsair_void_request_status error reporting to callers
>  - Simplified corsair_void_request_status
>  - Removed battery_struct_size, use sizeof(*battery_data) instead
>  - Removed unnecessary curly brackets in send_alert_store
> 
> v2 -> v3:
>  - Use __free(kfree) helper to free allocations
>  - Combined similar condition checks
>  - Check power_supply was created successfully before saving it
>  - Removed explicit initialisation of return value in corsair_void_probe
>  - Use preferred syntax for drvdata's struct allocation size
>  - Removed named success exit point from driver probe
>  - Removed unused driver probe exit point
> 
> v1 -> v2:
>  - Added a module author email address
>  - Removed unused variable (psy_cfg)
>  - Corrected status request buffer size from 12 bytes to 2 bytes
>  - Use enums for battery status
>  - Use kmalloc instead of kzalloc where applicable
>  - Use __le16 type for cpu_to_le16
>  - Use DEVICE_ATTR_RO and DEVICE_ATTR_WO to create attributes
>  - Simplified corsair_void_set_sidetone_wired
>  - Cleaned up patch description
> 
>  .../ABI/testing/sysfs-driver-hid-corsair-void |  38 +
>  drivers/hid/Kconfig                           |   3 +
>  drivers/hid/Makefile                          |   2 +-
>  drivers/hid/hid-corsair-void.c                | 829 ++++++++++++++++++
>  4 files changed, 871 insertions(+), 1 deletion(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-driver-hid-corsair-void
>  create mode 100644 drivers/hid/hid-corsair-void.c

Applied to hid.git#for-6.13/corsair. Thanks,

-- 
Jiri Kosina
SUSE Labs