<div dir="ltr">Hi,<div><br></div><div>Appreciate your help Jonathan and Rob,</div><div><br></div><div>Just seeing the following EVB device tree</div><div><div><a href="https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/aspeed-ast2500-evb.dts#L8" target="_blank">https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/aspeed-ast2500-evb.dts#L8</a><br></div><div><br></div></div><div>And not a EVB board.</div><div><a href="https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/aspeed-bmc-facebook-cmm.dts#L9">https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/aspeed-bmc-facebook-cmm.dts#L9</a><br></div><div><br></div><div>but still also option V2 is good for us.</div><div><br></div><div>Sorry it took that long.</div><div><br></div><div>Thanks a lot!</div><div><br></div><div>Tomer</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 15 Mar 2021 at 19:47, Jonathan Neuschäfer <<a href="mailto:j.neuschaefer@gmx.net" target="_blank">j.neuschaefer@gmx.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Mar 15, 2021 at 10:23:38AM -0600, Rob Herring wrote:<br>
> On Sat, Mar 13, 2021 at 06:53:20PM +0100, Jonathan Neuschäfer wrote:<br>
> > The general trend is to have devicetree bindings in YAML format, to<br>
> > allow automatic validation of bindings and devicetrees.<br>
> > <br>
> > Convert the NPCM SoC family's binding to YAML before it accumulates more<br>
> > entries.<br>
> > <br>
> > Signed-off-by: Jonathan Neuschäfer <<a href="mailto:j.neuschaefer@gmx.net" target="_blank">j.neuschaefer@gmx.net</a>><br>
> > ---<br>
> > <br>
> > If someone else wants to be listed as the maintainer, please let me<br>
> > know.<br>
> > <br>
> > <br>
> > v3:<br>
> > - In this version, I removed the nuvoton,npcm750-evb compatible string<br>
> >   again. I had previously introduced it to simplify the binding a little<br>
> >   bit, but Tomer Maimon suggested to keep /compatible of<br>
> >   nuvoton-npcm750-evb.dts as-is (i.e. only use "nuvoton,npcm750", no<br>
> >   board-specific string).<br>
> >   Because of this change, I am not including Rob Herring's R-b tag.<br>
> <br>
> I think you should add nuvoton,npcm750-evb. Not so much for that board, <br>
> but to ensure any additional boards get a board specific compatible.<br>
<br>
I'm inclined to agree and go back to v2, but I'd like to hear Tomer<br>
Maimon's opinion first.<br>
<br>
> > v2:<br>
> > - <a href="https://lore.kernel.org/lkml/20210116010907.3475405-1-j.neuschaefer@gmx.net/" rel="noreferrer" target="_blank">https://lore.kernel.org/lkml/20210116010907.3475405-1-j.neuschaefer@gmx.net/</a><br>
> >   <a href="https://lore.kernel.org/lkml/20210303154622.3018839-1-j.neuschaefer@gmx.net/" rel="noreferrer" target="_blank">https://lore.kernel.org/lkml/20210303154622.3018839-1-j.neuschaefer@gmx.net/</a><br>
> > - Fix indentation to satisfy yamllint<br>
> > - Fix $schema line<br>
<br>
<br>
Thanks,<br>
Jonathan Neuschäfer<br>
</blockquote></div>