2022-09-15 18:21:00

by Lad, Prabhakar

[permalink] [raw]
Subject: [PATCH v3 01/10] dt-bindings: soc: renesas: Move renesas.yaml from arm to soc

From: Lad Prabhakar <[email protected]>

renesas.yaml lists out all the Renesas SoC's and the platforms/EVK's which
is either ARM32/ARM64. It would rather make sense if we move renesas.yaml
to the soc/renesas folder instead. This is in preparation for adding a new
SoC (RZ/Five) from Renesas which is based on RISC-V.

While at it drop the old entry for renesas.yaml from MAINTAINERS file and
there is no need to update the new file path of renesas.yaml as we already
have an entry for Documentation/devicetree/bindings/soc/renesas/ folder.

Signed-off-by: Lad Prabhakar <[email protected]>
---
v3:
* New patch along with this series previously posted as a standalone
patch [0].

[0] https://patchwork.kernel.org/project/linux-renesas-soc/patch/[email protected]/
---
.../devicetree/bindings/{arm => soc/renesas}/renesas.yaml | 0
MAINTAINERS | 1 -
2 files changed, 1 deletion(-)
rename Documentation/devicetree/bindings/{arm => soc/renesas}/renesas.yaml (100%)

diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
similarity index 100%
rename from Documentation/devicetree/bindings/arm/renesas.yaml
rename to Documentation/devicetree/bindings/soc/renesas/renesas.yaml
diff --git a/MAINTAINERS b/MAINTAINERS
index c26a5c573a5d..f5f17c50dac3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2673,7 +2673,6 @@ S: Supported
Q: http://patchwork.kernel.org/project/linux-renesas-soc/list/
C: irc://irc.libera.chat/renesas-soc
T: git git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git next
-F: Documentation/devicetree/bindings/arm/renesas.yaml
F: Documentation/devicetree/bindings/hwinfo/renesas,prr.yaml
F: Documentation/devicetree/bindings/soc/renesas/
F: arch/arm/boot/dts/emev2*
--
2.25.1


2022-09-15 21:57:35

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v3 01/10] dt-bindings: soc: renesas: Move renesas.yaml from arm to soc

On 15/09/2022 19:15, Prabhakar wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> From: Lad Prabhakar <[email protected]>
>
> renesas.yaml lists out all the Renesas SoC's and the platforms/EVK's which
> is either ARM32/ARM64. It would rather make sense if we move renesas.yaml
> to the soc/renesas folder instead. This is in preparation for adding a new
> SoC (RZ/Five) from Renesas which is based on RISC-V.
>
> While at it drop the old entry for renesas.yaml from MAINTAINERS file and
> there is no need to update the new file path of renesas.yaml as we already
> have an entry for Documentation/devicetree/bindings/soc/renesas/ folder.
>
> Signed-off-by: Lad Prabhakar <[email protected]>
> ---
> v3:
> * New patch along with this series previously posted as a standalone
> patch [0].
>
> [0] https://patchwork.kernel.org/project/linux-renesas-soc/patch/[email protected]/
> ---
> .../devicetree/bindings/{arm => soc/renesas}/renesas.yaml | 0
> MAINTAINERS | 1 -
> 2 files changed, 1 deletion(-)
> rename Documentation/devicetree/bindings/{arm => soc/renesas}/renesas.yaml (100%)

Got one error from the rename:
/stuff/linux/Documentation/devicetree/bindings/soc/renesas/renesas.yaml: $id: relative path/filename doesn't match actual path or filename
expected: http://devicetree.org/schemas/soc/renesas/renesas.yaml#

>
> diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> similarity index 100%
> rename from Documentation/devicetree/bindings/arm/renesas.yaml
> rename to Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c26a5c573a5d..f5f17c50dac3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2673,7 +2673,6 @@ S: Supported
> Q: http://patchwork.kernel.org/project/linux-renesas-soc/list/
> C: irc://irc.libera.chat/renesas-soc
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git next
> -F: Documentation/devicetree/bindings/arm/renesas.yaml
> F: Documentation/devicetree/bindings/hwinfo/renesas,prr.yaml
> F: Documentation/devicetree/bindings/soc/renesas/
> F: arch/arm/boot/dts/emev2*
> --
> 2.25.1
>

2022-09-15 22:10:18

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3 01/10] dt-bindings: soc: renesas: Move renesas.yaml from arm to soc

Hi Conor,

Thank you for the review.

On Thu, Sep 15, 2022 at 10:13 PM <[email protected]> wrote:
>
> On 15/09/2022 19:15, Prabhakar wrote:
> > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >
> > From: Lad Prabhakar <[email protected]>
> >
> > renesas.yaml lists out all the Renesas SoC's and the platforms/EVK's which
> > is either ARM32/ARM64. It would rather make sense if we move renesas.yaml
> > to the soc/renesas folder instead. This is in preparation for adding a new
> > SoC (RZ/Five) from Renesas which is based on RISC-V.
> >
> > While at it drop the old entry for renesas.yaml from MAINTAINERS file and
> > there is no need to update the new file path of renesas.yaml as we already
> > have an entry for Documentation/devicetree/bindings/soc/renesas/ folder.
> >
> > Signed-off-by: Lad Prabhakar <[email protected]>
> > ---
> > v3:
> > * New patch along with this series previously posted as a standalone
> > patch [0].
> >
> > [0] https://patchwork.kernel.org/project/linux-renesas-soc/patch/[email protected]/
> > ---
> > .../devicetree/bindings/{arm => soc/renesas}/renesas.yaml | 0
> > MAINTAINERS | 1 -
> > 2 files changed, 1 deletion(-)
> > rename Documentation/devicetree/bindings/{arm => soc/renesas}/renesas.yaml (100%)
>
> Got one error from the rename:
> /stuff/linux/Documentation/devicetree/bindings/soc/renesas/renesas.yaml: $id: relative path/filename doesn't match actual path or filename
> expected: http://devicetree.org/schemas/soc/renesas/renesas.yaml#
>
Argh, how did I miss that!

Cheers,
Prabhakar

2022-09-20 12:54:09

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH v3 01/10] dt-bindings: soc: renesas: Move renesas.yaml from arm to soc

Hi Prabhakar,

On Thu, Sep 15, 2022 at 8:17 PM Prabhakar <[email protected]> wrote:
> From: Lad Prabhakar <[email protected]>
>
> renesas.yaml lists out all the Renesas SoC's and the platforms/EVK's which
> is either ARM32/ARM64. It would rather make sense if we move renesas.yaml
> to the soc/renesas folder instead. This is in preparation for adding a new
> SoC (RZ/Five) from Renesas which is based on RISC-V.
>
> While at it drop the old entry for renesas.yaml from MAINTAINERS file and
> there is no need to update the new file path of renesas.yaml as we already
> have an entry for Documentation/devicetree/bindings/soc/renesas/ folder.
>
> Signed-off-by: Lad Prabhakar <[email protected]>

Thanks for your patch!

> rename from Documentation/devicetree/bindings/arm/renesas.yaml
> rename to Documentation/devicetree/bindings/soc/renesas/renesas.yaml

Fine for me. With the yaml path inside fixed:
Reviewed-by: Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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