Re: [PATCH v2 0/6] Convert adi,adv7511.txt DT bindings to yaml

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Ricardo,

On Mon, May 11, 2020 at 1:06 PM Ricardo Cañuelo
<ricardo.canuelo@collabora.com> wrote:
> This series convert the adi,adv7511.txt DT bindings to json-schema. As a
> result of the conversion some dts files needed to be updated.
>
> The changes to the dts files are of three types:
>
>   - Reordering of the I2C slave addresses list of the ADV75xx node. The
>     addresses in the 'reg' property and the matching names in
>     'reg-names' for an I2C slave don't need to be in any particular
>     order, but the DT schema defines these properties as a cell array
>     and a string array respectively, which are ordered, so the
>     definitions in the dts files must match the order in the binding.
>
>   - Filling the minimum binding requirements. Most of the time this
>     means creating a 'ports' node in the boards that don't define
>     them. Note, however, that the purpose of this is simply to make the
>     definition compliant with the binding. I didn't define any endpoints
>     for the ports.
>
>   - Removing unneeded properties.
>
> About the binding conversion:
>
>   - The original binding covered five different devices: ADV7511,
>     ADV7511W, ADV7513, ADV7533 and ADV7535. They all share a common set
>     of properties but ADV7533 and ADV7535 have enough differences from
>     the rest to warrant their own binding file. In v1 I modelled all the
>     properties constraints for all five devices in a single file but it
>     turned out a bit too complex. Splitting the binding into one for
>     ADV7511/11W/13 and another for ADV7533/35 makes them much easier to
>     read and maintain.

Thanks for your series!

> Patches 1/6 to 5/6 contain the dts changes. Patch 6/6 contains the
> binding conversion.

If the binding conversion is accepted, I can queue the below in
renesas-fix-for-v5.7, to avoid the conversion introducing a regression.

>   arm64: dts: renesas: make hdmi encoder nodes compliant with DT
>     bindings
>   ARM: dts: renesas: make hdmi encoder nodes compliant with DT bindings
>   ARM: dts: iwg20d-q7-dbcm-ca: remove unneeded properties in hdmi@39

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds