Re: [PATCH v3] dt-bindings: arm: Convert nuvoton,npcm750 binding to YAML

["Jonathan Neuschäfer" <j.neuschaefer@gmx.net>]

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

On Mon, Mar 15, 2021 at 10:23:38AM -0600, Rob Herring wrote:
> On Sat, Mar 13, 2021 at 06:53:20PM +0100, Jonathan Neuschäfer wrote:
> > The general trend is to have devicetree bindings in YAML format, to
> > allow automatic validation of bindings and devicetrees.
> > 
> > Convert the NPCM SoC family's binding to YAML before it accumulates more
> > entries.
> > 
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> > ---
> > 
> > If someone else wants to be listed as the maintainer, please let me
> > know.
> > 
> > 
> > v3:
> > - In this version, I removed the nuvoton,npcm750-evb compatible string
> >   again. I had previously introduced it to simplify the binding a little
> >   bit, but Tomer Maimon suggested to keep /compatible of
> >   nuvoton-npcm750-evb.dts as-is (i.e. only use "nuvoton,npcm750", no
> >   board-specific string).
> >   Because of this change, I am not including Rob Herring's R-b tag.
> 
> I think you should add nuvoton,npcm750-evb. Not so much for that board, 
> but to ensure any additional boards get a board specific compatible.

I'm inclined to agree and go back to v2, but I'd like to hear Tomer
Maimon's opinion first.

> > v2:
> > - https://lore.kernel.org/lkml/20210116010907.3475405-1-j.neuschaefer@gmx.net/
> >   https://lore.kernel.org/lkml/20210303154622.3018839-1-j.neuschaefer@gmx.net/
> > - Fix indentation to satisfy yamllint
> > - Fix $schema line


Thanks,
Jonathan Neuschäfer

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