[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <bff5cb83-54fe-4e18-a0fe-32d90d90726f@gmail.com>
Date: Wed, 17 Jul 2024 01:27:41 +0800
From: Cryolitia PukNgae <cryolitia@...il.com>
To: Randy Dunlap <rdunlap@...radead.org>, 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
On 2024/7/16 22:36, Randy Dunlap wrote:
> 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
Thanks for review! v2 sent.
Powered by blists - more mailing lists