linux-kernel - Re: [PATCH 2/2] hwmon: document: add gpd-fan

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: <c81b25ec-3e6a-4b9b-8e3b-9991f1645d47@infradead.org>
Date: Tue, 16 Jul 2024 07:36:24 -0700
From: Randy Dunlap <rdunlap@...radead.org>
To: Cryolitia@...il.com, Jean Delvare <jdelvare@...e.com>,
 Guenter Roeck <linux@...ck-us.net>, Jonathan Corbet <corbet@....net>
Cc: linux-kernel@...r.kernel.org, linux-hwmon@...r.kernel.org,
 linux-doc@...r.kernel.org, Celeste Liu <CoelacanthusHex@...il.com>,
 Marcin Strągowski <marcin@...agowski.com>
Subject: Re: [PATCH 2/2] hwmon: document: add gpd-fan

Hi--

On 7/16/24 4:49 AM, Cryolitia PukNgae via B4 Relay wrote:
> From: Cryolitia PukNgae <Cryolitia@...il.com>
> 
> Add GPD fan driver document
> 
> Signed-off-by: Cryolitia PukNgae <Cryolitia@...il.com>
> ---
>  Documentation/hwmon/gpd-fan.rst | 68 +++++++++++++++++++++++++++++++++++++++++
>  Documentation/hwmon/index.rst   |  1 +
>  MAINTAINERS                     |  1 +
>  3 files changed, 70 insertions(+)
> 
> diff --git a/Documentation/hwmon/gpd-fan.rst b/Documentation/hwmon/gpd-fan.rst
> new file mode 100644
> index 000000000000..0e5bb8b5feed
> --- /dev/null
> +++ b/Documentation/hwmon/gpd-fan.rst
> @@ -0,0 +1,68 @@
> +.. SPDX-License-Identifier: GPL-2.0-or-later
> +
> +Kernel driver gpd-fan
> +=========================
> +
> +Authors:
> +    - Cryolitia PukNgae <Cryolitia@...il.com>
> +
> +Description:

No ending ':' above, please.

> +------------
> +
> +Handheld devices from GPD provide fan readings and fan control through
> +their embedded controllers.
> +
> +Supported devices
> +-----------------
> +
> +Currently the driver supports the following handhelds:
> +
> + - GPD Win Mini (7840U)
> + - GPD Win Mini (8840U)
> + - GPD Win Max 2
> + - GPD Win Max 2 2023 (7840U)
> + - GPD Win Max 2 2024 (8840U)
> + - GPD Win 4 (6800U)
> + - GPD Win 4 (7840U)
> +
> +Module parameters
> +-----------------
> +
> +gpd_fan_model
> +  Force specific which module quirk should be use.

                                                 used.

> +
> +   - wm2
> +       - GPD Win 4 (7840U)
> +       - GPD Win Max 2 (6800U)
> +       - GPD Win Max 2 2023 (7840U)
> +       - GPD Win Max 2 2024 (8840U)
> +   - win4
> +       - GPD Win 4 (6800U)
> +   - win_mini
> +       - GPD Win Mini (7840U)
> +       - GPD Win Mini (8840U)

so how is the module parameter used? like so?

   gpd_fan_model=wm2

> +
> +Sysfs entries
> +-------------
> +
> +The following attributes are supported:
> +
> +fan1_input
> +  Read Only. Reads current fan RMP.

Should that be                    RPM
?

> +
> +pwm1_enable
> +  Read Write. Enable manual fan control. Write "0" to disable control and run

     Read/Write
or
     Read-Write

> +  full speed. Write "1" to set to manual, write "2" to let the EC control decide
> +  fan speed. Read this attribute to see current status.
> +
> +pwm1
> +  Read Write. Read this attribute to see current duty cycle in the range [0-255].

     Read-Write
or
     Read/Write

> +  When pwm1_enable is set to "1" (manual) write any value in the range [0-255]
> +  to set fan speed.
> +
> +pwm1_mode
> +  Read Only. Should always be "1" by now to indicate the fan being under PWM mode.

                    Drop             by now

> +
> +update_interval
> +  Read Write. Set the interval in milliseconds to update fan speed. The default

     Read-Write
or
     Read/Write

> +  and minimum time is 1 second

-- 
~Randy