Re: [PATCH v7 1/4] dt-bindings: hwmon: fan: Add fan binding to schema

From: Krzysztof Kozlowski
Date: Tue Nov 29 2022 - 03:12:16 EST


On 21/11/2022 13:29, Naresh Solanki wrote:

> + pulses-per-revolution:
> + description:
> + The number of pulse from fan sensor per revolution.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> + target-rpm:
> + description:
> + Target RPM the fan should be configured during driver probe.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> + pwms:
> + description:
> + PWM provider.

Ah, so it is not a PWM provider by this FAN controller? A bit confusing
description. Instead maybe:
PWM signal for the fan

and do you expect more than one PWM for one fan?

Best regards,
Krzysztof