| Message ID | 20220503115557.53370-13-phil.edworthy@renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Add new Renesas RZ/V2M SoC and Renesas RZ/V2M EVK support | expand |
Hi Phil, On Tue, May 3, 2022 at 2:02 PM Phil Edworthy <phil.edworthy@renesas.com> wrote: > Add basic support for RZ/V2M EVK (based on R9A09G011): > - memory > - External input clock > - UART > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > v3: > - Add comment about mem reserved for ISP Firmware > - Remove sys node Thanks for the update! > --- /dev/null > +++ b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts > @@ -0,0 +1,44 @@ > +// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +/* > + * Device Tree Source for the RZ/V2M (r9a09g011) Evaluation Kit Board > + * > + * Copyright (C) 2022 Renesas Electronics Corp. > + */ > + > +/dts-v1/; > +#include "r9a09g011.dtsi" > + > +/ { > + memory@58000000 { > + device_type = "memory"; > + /* > + * first 2GiB is reserved for ISP Firmware, "2GiB" (0x80000000) doesn't look right to me. Perhaps 1.25 GIB? If you confirm, I can fix up (and the previous patch), and queue in renesas-devel for v5.19. > + * next 128MiB is reserved for secure area. > + */ > + reg = <0x0 0x58000000 0x0 0x28000000>; > + }; The rest LGTM, so with the above fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Thanks! 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
Hi Geert, On 04 May 2022 09:52 Geert Uytterhoeven wrote: > On Tue, May 3, 2022 at 2:02 PM Phil Edworthy wrote: > > Add basic support for RZ/V2M EVK (based on R9A09G011): > > - memory > > - External input clock > > - UART > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > v3: > > - Add comment about mem reserved for ISP Firmware > > - Remove sys node > > Thanks for the update! > > > --- /dev/null > > +++ b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts > > @@ -0,0 +1,44 @@ > > +// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +/* > > + * Device Tree Source for the RZ/V2M (r9a09g011) Evaluation Kit Board > > + * > > + * Copyright (C) 2022 Renesas Electronics Corp. > > + */ > > + > > +/dts-v1/; > > +#include "r9a09g011.dtsi" > > + > > +/ { > > > + memory@58000000 { > > + device_type = "memory"; > > + /* > > + * first 2GiB is reserved for ISP Firmware, > > "2GiB" (0x80000000) doesn't look right to me. > Perhaps 1.25 GIB? How on earth did I do that? Yes, 1.25 GiB is correct. > If you confirm, I can fix up (and the previous patch), and queue in > renesas-devel for v5.19. > > > + * next 128MiB is reserved for secure area. > > + */ > > + reg = <0x0 0x58000000 0x0 0x28000000>; > > + }; > > The rest LGTM, so with the above fixed: > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Thanks! Phil
diff --git a/arch/arm64/boot/dts/renesas/Makefile b/arch/arm64/boot/dts/renesas/Makefile index 15309309a2e1..e66d76d42e52 100644 --- a/arch/arm64/boot/dts/renesas/Makefile +++ b/arch/arm64/boot/dts/renesas/Makefile @@ -83,3 +83,5 @@ dtb-$(CONFIG_ARCH_R9A07G044) += r9a07g044c2-smarc.dtb dtb-$(CONFIG_ARCH_R9A07G044) += r9a07g044l2-smarc.dtb dtb-$(CONFIG_ARCH_R9A07G054) += r9a07g054l2-smarc.dtb + +dtb-$(CONFIG_ARCH_R9A09G011) += r9a09g011-v2mevk2.dtb diff --git a/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts new file mode 100644 index 000000000000..41cba82c2252 --- /dev/null +++ b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts @@ -0,0 +1,44 @@ +// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +/* + * Device Tree Source for the RZ/V2M (r9a09g011) Evaluation Kit Board + * + * Copyright (C) 2022 Renesas Electronics Corp. + */ + +/dts-v1/; +#include "r9a09g011.dtsi" + +/ { + model = "RZ/V2M Evaluation Kit 2.0"; + compatible = "renesas,rzv2mevk2", "renesas,r9a09g011"; + + aliases { + serial0 = &uart0; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; + + memory@58000000 { + device_type = "memory"; + /* + * first 2GiB is reserved for ISP Firmware, + * next 128MiB is reserved for secure area. + */ + reg = <0x0 0x58000000 0x0 0x28000000>; + }; + + memory@180000000 { + device_type = "memory"; + reg = <0x1 0x80000000 0x0 0x80000000>; + }; +}; + +&extal_clk { + clock-frequency = <48000000>; +}; + +&uart0 { + status = "okay"; +};