Thread (5 messages) 5 messages, 4 authors, 2020-07-07

Re: [PATCH v7] hwmon: add Corsair Commander Pro driver

From: Benjamin Tissoires <hidden>
Date: 2020-07-07 12:41:38
Also in: linux-doc, linux-hwmon, lkml

On Tue, Jul 7, 2020 at 12:20 PM Jiri Kosina [off-list ref] wrote:
On Mon, 29 Jun 2020, Guenter Roeck wrote:
quoted
On Fri, Jun 26, 2020 at 07:59:36AM +0200, Marius Zachmann wrote:
quoted
This is v7 of a driver for the Corsair Commander Pro.
It provides sysfs attributes for:
- Reading fan speed
- Reading temp sensors
- Reading voltage values
- Writing pwm and reading last written pwm
- Reading fan and temp connection status

It is an usb driver, so it needs to be ignored by usbhid.
The Corsair Commander Pro is a fan controller and provides
no means for user interaction.
The two device numbers are there, because there is a slightly
different version of the same device. (Only difference
seem to be in some presets.)

This is based on the staging/hwmon tree.

Signed-off-by: Marius Zachmann <mail@mariuszachmann.de>
For my reference:

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Waiting for Ack from HID maintainer.
Acked-by: Jiri Kosina <redacted>
Sorry I missed this too:

Acked-by: Benjamin Tissoires <redacted>

for the HID hunk too.

Cheers,
Benjamin
for the drivers/hid/hid-quirks.c hunk. Thanks,

--
Jiri Kosina
SUSE Labs
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help