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