| Message ID | 20220915181558.354737-2-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Add support for Renesas RZ/Five SoC | expand |
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 <prabhakar.mahadev-lad.rj@bp.renesas.com> > > 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 <prabhakar.mahadev-lad.rj@bp.renesas.com> > --- > v3: > * New patch along with this series previously posted as a standalone > patch [0]. > > [0] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20220815111708.22302-1-prabhakar.mahadev-lad.rj@bp.renesas.com/ > --- > .../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 >
Hi Conor, Thank you for the review. On Thu, Sep 15, 2022 at 10:13 PM <Conor.Dooley@microchip.com> 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 <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > 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 <prabhakar.mahadev-lad.rj@bp.renesas.com> > > --- > > v3: > > * New patch along with this series previously posted as a standalone > > patch [0]. > > > > [0] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20220815111708.22302-1-prabhakar.mahadev-lad.rj@bp.renesas.com/ > > --- > > .../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
Hi Prabhakar, On Thu, Sep 15, 2022 at 8:17 PM Prabhakar <prabhakar.csengg@gmail.com> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > 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 <prabhakar.mahadev-lad.rj@bp.renesas.com> 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 <geert+renesas@glider.be> 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
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*