Re: [PATCH v6 06/12] dt-bindings: arm: Add initial bindings for Nuvoton platform

From: Rob Herring
Date: Mon Apr 03 2023 - 16:33:57 EST


On Fri, Mar 31, 2023 at 10:15:16AM +0800, Jacky Huang wrote:
>
>
> On 2023/3/30 下午 09:25, Krzysztof Kozlowski wrote:
> > On 30/03/2023 12:41, Jacky Huang wrote:
> > > Dear Rob,
> > >
> > >
> > > On 2023/3/29 下午 09:07, Rob Herring wrote:
> > > > On Mon, Mar 27, 2023 at 9:19 PM Jacky Huang <ychuang570808@xxxxxxxxx> wrote:
> > > > > From: Jacky Huang <ychuang3@xxxxxxxxxxx>
> > > > >
> > > > > Rename the bindings/arm/npcm directory as nuvoton.
> > > > > Add binding for ARMv8 based Nuvotn SoCs and platform boards.
> > > > > Add initial bindings for ma35d1 series development boards.
> > > > >
> > > > > Signed-off-by: Jacky Huang <ychuang3@xxxxxxxxxxx>
> > > > > ---
> > > > > .../bindings/arm/nuvoton/nuvoton,ma35d1.yaml | 30 +++++++++++++++++++
> > > > > .../nuvoton,npcm-gcr.yaml} | 2 +-
> > > > > .../npcm.yaml => nuvoton/nuvoton,npcm.yaml} | 2 +-
> > > > > 3 files changed, 32 insertions(+), 2 deletions(-)
> > > > > create mode 100644 Documentation/devicetree/bindings/arm/nuvoton/nuvoton,ma35d1.yaml
> > > > > rename Documentation/devicetree/bindings/arm/{npcm/nuvoton,gcr.yaml => nuvoton/nuvoton,npcm-gcr.yaml} (93%)
> > > > Since you are moving it, this one should be moved to bindings/soc/nuvoton/
> > > >
> > > > > rename Documentation/devicetree/bindings/arm/{npcm/npcm.yaml => nuvoton/nuvoton,npcm.yaml} (93%)
> > > OK, I will move "Documentation/devicetree/bindings/arm/npcm" to
> > > "Documentation/devicetree/bindings/soc/nuvoton".
> > To clarify - Rob asked *this one*. Not all.

nuvoton,gcr.yaml is what should be moved to bindings/soc/... The rest
are the correct locations.

> >
> >
> >
> > Best regards,
> > Krzysztof
> >
>
> Dear Krzysztof,
>
> I am sorry that still cannot get it exactly.
>
> 1. Just only submit  the nuvoton,ma35d1.yaml to
> Documentation/devicetree/bindings/soc/nuvoton/nuvoton,ma35d1.yaml
>     and keep Documentation/devicetree/bindings/arm/npcm unchanged
>
> 2. Move the directory "Documentation/devicetree/bindings/arm/npcm" to
> "Documentation/devicetree/bindings/soc/nuvoton"
>    and submit  the nuvoton,ma35d1.yaml to
> "Documentation/devicetree/bindings/soc/nuvoton"
>
> I should do 1 or 2?  Thank you very much.

Neither.

Arm board and soc root node bindings go in bindings/arm/...

Miscellaneous system registers that don't fit into any other bindings/*
subsystem directory go in bindings/soc/

Rob