Thread (17 messages) 17 messages, 3 authors, 2017-01-23

Re: [PATCH 7/7] arm64: dts: r8a7796: salvator-x: add SCIF1 (DEBUG1)

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2016-12-14 14:10:08
Also in: linux-renesas-soc

Hi Uli,

On Fri, Dec 9, 2016 at 1:36 PM, Ulrich Hecht
[off-list ref] wrote:
quoted hunk ↗ jump to hunk
Enables the SCIF hooked up to the DEBUG1 connector.

Signed-off-by: Ulrich Hecht <redacted>
---
 arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
index da46ac7..9204691 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
@@ -18,6 +18,7 @@

        aliases {
                serial0 = &scif2;
+               serial1 = &scif1;
Is there any specific reason you chose scif1 over hscif1?
quoted hunk ↗ jump to hunk
                ethernet0 = &avb;
        };
@@ -140,6 +141,11 @@
        pinctrl-0 = <&scif_clk_pins>;
        pinctrl-names = "default";

+       scif1_pins: scif1 {
+               groups = "scif1_data_a";
Please add "scif1_ctrl_a".
quoted hunk ↗ jump to hunk
+               function = "scif1";
+       };
+
        scif2_pins: scif2 {
                groups = "scif2_data_a";
                function = "scif2";
@@ -280,6 +286,12 @@
        status = "okay";
 };

+&scif1 {
+       pinctrl-0 = <&scif1_pins>;
+       pinctrl-names = "default";
Please add "uart-has-rtscts".
+       status = "okay";
+};
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help