From: Marius Zachmann <mail@mariuszachmann.de>
To: Guenter Roeck <linux@roeck-us.net>,
Jean Delvare <jdelvare@suse.com>, Jiri Kosina <jikos@kernel.org>,
Benjamin Tissoires <benjamin.tissoires@redhat.com>
Cc: Marius Zachmann <mail@mariuszachmann.de>,
linux-hwmon@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-input@vger.kernel.org
Subject: [PATCH v4 0/3] hwmon: Add Corsair Commander Pro driver
Date: Wed, 24 Jun 2020 09:21:51 +0200 [thread overview]
Message-ID: <20200624072154.22486-1-mail@mariuszachmann.de> (raw)
This is v4 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. (patch 3/3)
This is based on the staging/hwmon tree.
Signed-off-by: Marius Zachmann <mail@mariuszachmann.de>
---
Changes from v3
- add device ids to hid_ignore_list in hid-quirks.h
- removed unused update_interval
- ccp_disconnect is static
Changes from v2
- add corsair-cpro to Documentation/hwmon/index.rst
- add SPDX license identifier to corsair-cpro.rst
- remove fanX_enable from Documentation and driver
- changed comment style
- clarified protocol in comments
- add get_temp_cnct for reading temperature sensor
connection status
- added temp_cnct and temp_label in ccp_device
- added tempX_label, showing connection status
- renamed get_fan_mode_label to get_fan_cnct
- get_temp_cnct and get_fan_cnct only called in probe
- send_usb_cmd checks errors in the device response
- send_usb_cmd clears the buffer and sets the
command
- inlined get_fan_rpm and get_voltage and removed
functions
- more reverse christmas tree
Marius Zachmann (3):
input: add to hid_ignore_list
add Documentation
add driver
Documentation/hwmon/corsair-cpro.rst | 40 +++
Documentation/hwmon/index.rst | 1 +
MAINTAINERS | 6 +
drivers/hid/hid-quirks.c | 2 +
drivers/hwmon/Kconfig | 10 +
drivers/hwmon/Makefile | 1 +
drivers/hwmon/corsair-cpro.c | 489 +++++++++++++++++++++++++++
7 files changed, 549 insertions(+)
create mode 100644 Documentation/hwmon/corsair-cpro.rst
create mode 100644 drivers/hwmon/corsair-cpro.c
--
2.27.0
next reply other threads:[~2020-06-24 7:23 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-24 7:21 Marius Zachmann [this message]
2020-06-24 7:21 ` [PATCH v4 1/3] input: add to hid_ignore_list Marius Zachmann
2020-06-24 8:45 ` Benjamin Tissoires
2020-06-24 7:21 ` [PATCH v4 2/3] hwmon: add Documentation Marius Zachmann
2020-06-24 7:21 ` [PATCH v4 3/3] hwmon: add driver Marius Zachmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200624072154.22486-1-mail@mariuszachmann.de \
--to=mail@mariuszachmann.de \
--cc=benjamin.tissoires@redhat.com \
--cc=jdelvare@suse.com \
--cc=jikos@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.