Re: [PATCH v4] dt-bindings: pwm: Convert NXP PCA9685 PWM controller to DT schema

["Uwe Kleine-König" <ukleinek@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1261 bytes --]

Hello Jeremy,

On Thu, Oct 22, 2026 at 12:00:28AM +0000, Jeremy Dean Clifton wrote:
> Convert the NXP PCA9685 PWM controller binding to DT schema format.
> 
> Changes from the text binding:
> 
> Document the 'reg' property and make it required. It was missing
> from the text binding, although the example used it and the device
> cannot work without the I2C address.
> 
> Rename the example node to the generic 'pwm', as required by
> pwm.yaml.
> 
> Describe both cells of the PWM specifier, and state that 'invert'
> and 'open-drain' apply to all outputs, as per the datasheet (INVRT
> and OUTDRV bits in MODE2).
> 
> Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
> Signed-off-by: Jeremy Dean Clifton <deaner92@yahoo.com>

Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-nexxt
as 7.5 material.

> +  "#pwm-cells":
> +    const: 2
> +    description:
> +      The first cell is the PWM channel index, from 0 to 15, or 16 for the
> +      ALLCALL channel, which sets all PWM channels at the same time. The
> +      second cell is the PWM period in nanoseconds.

If you want to work on that binding, you can deprecate #pwm-cells = <2>
and make <3> the right value. 

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]