| Message ID | 20201213183759.223246-18-aford173@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | arm64: dts: renesas: Cleanup Beacon Kit and support more SoC's | expand |
Hi Adam, On Sun, Dec 13, 2020 at 7:38 PM Adam Ford <aford173@gmail.com> wrote: > Beacon EmebeddedWorks is introducing a new kit based on the > RZ/G2N SoC from Renesas. > > The SOM supports eMMC, WiFi and Bluetooth, along with a Cat-M1 > cellular radio. > > The Baseboard has Ethernet, USB, HDMI, stereo audio in and out, > along with a variety of push buttons and LED's, and support for > a parallel RGB and an LVDS display. It uses the same baseboard > and SOM as the RZ/G2M. > > Signed-off-by: Adam Ford <aford173@gmail.com> Thanks for your patch! > --- /dev/null > +++ b/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts > @@ -0,0 +1,43 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright 2020, Compass Electronics Group, LLC > + */ > + > +/dts-v1/; > + > +#include "r8a774b1.dtsi" > +#include "beacon-renesom-som.dtsi" > +#include "beacon-renesom-baseboard.dtsi" > + > +/ { > + model = "Beacon Embedded Works RZ/G2N Development Kit"; > + compatible = "beacon,beacon-rzg2n", "renesas,r8a774b1"; > + > + aliases { > + serial0 = &scif2; > + serial1 = &hscif0; > + serial2 = &hscif1; > + serial3 = &scif0; > + serial4 = &hscif2; > + serial5 = &scif5; > + serial6 = &scif4; > + ethernet0 = &avb; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; No memory nodes? Are you relying on U-Boot to fill them in? If yes, why do you have them in the other board DTS files? > +}; > + > +&du { > + status = "okay"; Missing pinctrl properties? > + > + clocks = <&cpg CPG_MOD 724>, > + <&cpg CPG_MOD 723>, > + <&cpg CPG_MOD 721>, > + <&versaclock5 1>, > + <&x302_clk>, > + <&versaclock5 2>; > + clock-names = "du.0", "du.1", "du.3", > + "dclkin.0", "dclkin.1", "dclkin.3"; > +}; Gr{oetje,eeting}s, Geert
On Thu, Dec 17, 2020 at 5:49 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Adam, > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford <aford173@gmail.com> wrote: > > Beacon EmebeddedWorks is introducing a new kit based on the > > RZ/G2N SoC from Renesas. > > > > The SOM supports eMMC, WiFi and Bluetooth, along with a Cat-M1 > > cellular radio. > > > > The Baseboard has Ethernet, USB, HDMI, stereo audio in and out, > > along with a variety of push buttons and LED's, and support for > > a parallel RGB and an LVDS display. It uses the same baseboard > > and SOM as the RZ/G2M. > > > > Signed-off-by: Adam Ford <aford173@gmail.com> > > Thanks for your patch! Thank you for the review! > > > --- /dev/null > > +++ b/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts > > @@ -0,0 +1,43 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright 2020, Compass Electronics Group, LLC > > + */ > > + > > +/dts-v1/; > > + > > +#include "r8a774b1.dtsi" > > +#include "beacon-renesom-som.dtsi" > > +#include "beacon-renesom-baseboard.dtsi" > > + > > +/ { > > + model = "Beacon Embedded Works RZ/G2N Development Kit"; > > + compatible = "beacon,beacon-rzg2n", "renesas,r8a774b1"; > > + > > + aliases { > > + serial0 = &scif2; > > + serial1 = &hscif0; > > + serial2 = &hscif1; > > + serial3 = &scif0; > > + serial4 = &hscif2; > > + serial5 = &scif5; > > + serial6 = &scif4; > > + ethernet0 = &avb; > > + }; > > + > > + chosen { > > + stdout-path = "serial0:115200n8"; > > + }; > > No memory nodes? Are you relying on U-Boot to fill them in? > If yes, why do you have them in the other board DTS files? There is already a memory node included in the beacon-renesom.dtsi file which is defining memory@48000000 node which is common to the M, N and H. If I understand it correctly, the extra memory nodes defined in the higher-level dts for M and H variation are not applicable on the N. > > > +}; > > + > > +&du { > > + status = "okay"; > > Missing pinctrl properties? > oops. I'll fix. > > + > > + clocks = <&cpg CPG_MOD 724>, > > + <&cpg CPG_MOD 723>, > > + <&cpg CPG_MOD 721>, > > + <&versaclock5 1>, > > + <&x302_clk>, > > + <&versaclock5 2>; > > + clock-names = "du.0", "du.1", "du.3", > > + "dclkin.0", "dclkin.1", "dclkin.3"; > > +}; > > 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 Adam, > Subject: Re: [PATCH 17/18] arm64: dts: renesas: Introduce r8a774b1-beacon- > rzg2n-kit > > On Thu, Dec 17, 2020 at 5:49 AM Geert Uytterhoeven <geert@linux-m68k.org> > wrote: > > > > Hi Adam, > > > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford <aford173@gmail.com> wrote: > > > Beacon EmebeddedWorks is introducing a new kit based on the RZ/G2N > > > SoC from Renesas. > > > > > > The SOM supports eMMC, WiFi and Bluetooth, along with a Cat-M1 > > > cellular radio. > > > > > > The Baseboard has Ethernet, USB, HDMI, stereo audio in and out, > > > along with a variety of push buttons and LED's, and support for a > > > parallel RGB and an LVDS display. It uses the same baseboard and > > > SOM as the RZ/G2M. > > > > > > Signed-off-by: Adam Ford <aford173@gmail.com> > > > > Thanks for your patch! > > Thank you for the review! > > > > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts > > > @@ -0,0 +1,43 @@ > > > +// SPDX-License-Identifier: GPL-2.0 > > > +/* > > > + * Copyright 2020, Compass Electronics Group, LLC */ > > > + > > > +/dts-v1/; > > > + > > > +#include "r8a774b1.dtsi" > > > +#include "beacon-renesom-som.dtsi" > > > +#include "beacon-renesom-baseboard.dtsi" > > > + > > > +/ { > > > + model = "Beacon Embedded Works RZ/G2N Development Kit"; > > > + compatible = "beacon,beacon-rzg2n", "renesas,r8a774b1"; > > > + > > > + aliases { > > > + serial0 = &scif2; > > > + serial1 = &hscif0; > > > + serial2 = &hscif1; > > > + serial3 = &scif0; > > > + serial4 = &hscif2; > > > + serial5 = &scif5; > > > + serial6 = &scif4; > > > + ethernet0 = &avb; > > > + }; > > > + > > > + chosen { > > > + stdout-path = "serial0:115200n8"; > > > + }; > > > > No memory nodes? Are you relying on U-Boot to fill them in? > > If yes, why do you have them in the other board DTS files? > > There is already a memory node included in the beacon-renesom.dtsi file > which is defining memory@48000000 node which is common to the M, > N and H. If I understand it correctly, the extra memory nodes > defined in the higher-level dts for M and H variation are not applicable > on the N. If I am correct, It is not applicable, if it has only 2GB memory for the N variant. But If it has 4GB memory(1ch x4GB), then the memory is split between 32bit space(first 2GB) and 64 bit space(remaining 2GB). In this case you need to define extra DT node for memory in 64bit space. Cheers, Biju
diff --git a/arch/arm64/boot/dts/renesas/Makefile b/arch/arm64/boot/dts/renesas/Makefile index 3b8b03705917..ea3d7d9bc52e 100644 --- a/arch/arm64/boot/dts/renesas/Makefile +++ b/arch/arm64/boot/dts/renesas/Makefile @@ -8,6 +8,7 @@ dtb-$(CONFIG_ARCH_R8A774A1) += r8a774a1-hihope-rzg2m-rev2.dtb dtb-$(CONFIG_ARCH_R8A774A1) += r8a774a1-hihope-rzg2m-rev2-ex.dtb dtb-$(CONFIG_ARCH_R8A774A1) += r8a774a1-hihope-rzg2m-rev2-ex-idk-1110wr.dtb +dtb-$(CONFIG_ARCH_R8A774B1) += r8a774b1-beacon-rzg2n-kit.dtb dtb-$(CONFIG_ARCH_R8A774B1) += r8a774b1-hihope-rzg2n.dtb dtb-$(CONFIG_ARCH_R8A774B1) += r8a774b1-hihope-rzg2n-ex.dtb dtb-$(CONFIG_ARCH_R8A774B1) += r8a774b1-hihope-rzg2n-ex-idk-1110wr.dtb diff --git a/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts b/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts new file mode 100644 index 000000000000..01a233102976 --- /dev/null +++ b/arch/arm64/boot/dts/renesas/r8a774b1-beacon-rzg2n-kit.dts @@ -0,0 +1,43 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright 2020, Compass Electronics Group, LLC + */ + +/dts-v1/; + +#include "r8a774b1.dtsi" +#include "beacon-renesom-som.dtsi" +#include "beacon-renesom-baseboard.dtsi" + +/ { + model = "Beacon Embedded Works RZ/G2N Development Kit"; + compatible = "beacon,beacon-rzg2n", "renesas,r8a774b1"; + + aliases { + serial0 = &scif2; + serial1 = &hscif0; + serial2 = &hscif1; + serial3 = &scif0; + serial4 = &hscif2; + serial5 = &scif5; + serial6 = &scif4; + ethernet0 = &avb; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; +}; + +&du { + status = "okay"; + + clocks = <&cpg CPG_MOD 724>, + <&cpg CPG_MOD 723>, + <&cpg CPG_MOD 721>, + <&versaclock5 1>, + <&x302_clk>, + <&versaclock5 2>; + clock-names = "du.0", "du.1", "du.3", + "dclkin.0", "dclkin.1", "dclkin.3"; +};
Beacon EmebeddedWorks is introducing a new kit based on the RZ/G2N SoC from Renesas. The SOM supports eMMC, WiFi and Bluetooth, along with a Cat-M1 cellular radio. The Baseboard has Ethernet, USB, HDMI, stereo audio in and out, along with a variety of push buttons and LED's, and support for a parallel RGB and an LVDS display. It uses the same baseboard and SOM as the RZ/G2M. Signed-off-by: Adam Ford <aford173@gmail.com> --- arch/arm64/boot/dts/renesas/Makefile | 1 + .../dts/renesas/r8a774b1-beacon-rzg2n-kit.dts | 43 +++++++++++++++++++ 2 files changed, 44 insertions(+)