Re: [PATCH v6 1/2] dt-bindings: pwm: Add pwm-gpio

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

Some buzz around the patch made me notice this:

On Sun, Jun 2, 2024 at 10:33 PM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:

> +  "#pwm-cells":
> +    const: 3

I guess we should document these three cells:
- First cell must be 0 - just the one PWM on the one GPIO pin
- Second cell should be the default period that can be changed by software
- Third cell is polarity, 0 or PWM_POLARITY_INVERTED

I guess this is 3 not 2 because the maintainers previously said they wanted
it like this? (I haven't read all old mail, nor do I remember...)

The #pwm-cells are currently not properly specified in the bindings: for example
pwm-tiecap.yaml says "See pwm.yaml in this directory for a description
of the cells format."
and that file says nothing about the cells and what they are for, should
I send a separate patch for that?

Yours,
Linus Walleij