linux-kernel - Re: [PATCH 1/2] dt-bindings: hwmon: pwm-fan: Document default-pwm property

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: <b3b8aa28-24d3-4952-a8d3-7019ad4c63de@kernel.org>
Date: Thu, 2 Jan 2025 19:22:57 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Peter Korsgaard <peter@...sgaard.com>, Guenter Roeck
 <linux@...ck-us.net>, devicetree@...r.kernel.org, linux-hwmon@...r.kernel.org
Cc: Jean Delvare <jdelvare@...e.com>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] dt-bindings: hwmon: pwm-fan: Document default-pwm
 property

On 02/01/2025 18:50, Peter Korsgaard wrote:
> The pwm-fan driver uses full PWM (255) duty cycle at startup, which may not
> always be desirable because of noise or power consumption peaks, so add an
> optional "default-pwm" property that can be used to specify a custom default
> PWM duty cycle.
> 
> Signed-off-by: Peter Korsgaard <peter@...sgaard.com>

That's v3, not v1. Also mention here shortly how Rob's comment is addressed.

Please always handle reviewers' feedback, either by implementing it or
by responding or by extending commit msg. All this is missing in all
your versions you sent.

Best regards,
Krzysztof