[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250226134922.GA1944823-robh@kernel.org>
Date: Wed, 26 Feb 2025 07:49:22 -0600
From: Rob Herring <robh@...nel.org>
To: Francesco Dolcini <francesco@...cini.it>
Cc: Jean Delvare <jdelvare@...e.com>, Guenter Roeck <linux@...ck-us.net>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Farouk Bouabid <farouk.bouabid@...rry.de>,
Quentin Schulz <quentin.schulz@...rry.de>,
Francesco Dolcini <francesco.dolcini@...adex.com>,
linux-hwmon@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] dt-bindings: hwmon: amc6821: add PWM polarity
On Mon, Feb 24, 2025 at 07:08:00PM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@...adex.com>
>
> Add property to describe the PWM-Out pin polarity.
Why doesn't the invert support in the pwm binding work for you? Yes, I
read the discussion, but don't remember the conclusion and you need to
justify it here.
>
> Link: https://www.ti.com/lit/gpn/amc6821
> Signed-off-by: Francesco Dolcini <francesco.dolcini@...adex.com>
> ---
> v2: no changes
> v1: https://lore.kernel.org/all/20250218165633.106867-2-francesco@dolcini.it/
> ---
> Documentation/devicetree/bindings/hwmon/ti,amc6821.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/ti,amc6821.yaml b/Documentation/devicetree/bindings/hwmon/ti,amc6821.yaml
> index 5d33f1a23d03..11604aa41b3e 100644
> --- a/Documentation/devicetree/bindings/hwmon/ti,amc6821.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ti,amc6821.yaml
> @@ -28,6 +28,14 @@ properties:
> i2c-mux:
> type: object
>
> + ti,pwm-inverted:
> + type: boolean
> + description:
> + Set to make the PWM-Out pin go high (with an external pull-up resistor)
> + for 100% duty cycle (suitable for driving the fan using a NMOS device),
> + when not set the PWM-Out pin goes low for 100% duty cycle (suitable for
> + driving the fan using a PMOS device).
> +
> required:
> - compatible
> - reg
> --
> 2.39.5
>
Powered by blists - more mailing lists