diff mbox

[05/13,v4,RFC] arm64: renesas: Add SCIF2 support on r8a7795

Message ID 87pp30yhby.wl%kuninori.morimoto.gx@renesas.com (mailing list archive)
State RFC
Delegated to: Simon Horman
Headers show

Commit Message

Kuninori Morimoto Aug. 6, 2015, 8:06 a.m. UTC
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

This patch enables SCIF2

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
v3 -> v4

 - new patch

 .../bindings/serial/renesas,sci-serial.txt          |  4 ++++
 arch/arm64/boot/dts/renesas/r8a7795.dtsi            | 21 +++++++++++++++++++++
 include/dt-bindings/clock/r8a7795-clock.h           |  1 +
 3 files changed, 26 insertions(+)

Comments

Yoshihiro Shimoda Aug. 6, 2015, 8:43 a.m. UTC | #1
Hi Morimoto-san,


> Sent: Thursday, August 06, 2015 5:07 PM
> 
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> 
> This patch enables SCIF2
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
< snip >
> diff --git a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
> b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
> index e84b13a..8c97599 100644
> --- a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
> +++ b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
> @@ -23,6 +23,10 @@ Required properties:
< snip >
> +    - "renesas,scifa-r8a7795" for R8A7795 (R-Car H3) SCIFA compatible UART.
> +    - "renesas,scifb-r8a7795" for R8A7795 (R-Car H3) SCIFB compatible UART.

The R8A7795 doesn't have SCIFA and SCIFB.
(In other words, it has SCIF and HSCIF.)

Best regards,
Yoshihiro Shimoda

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kuninori Morimoto Aug. 6, 2015, 8:46 a.m. UTC | #2
Hi Shimoda-san

> > +    - "renesas,scifa-r8a7795" for R8A7795 (R-Car H3) SCIFA compatible UART.
> > +    - "renesas,scifb-r8a7795" for R8A7795 (R-Car H3) SCIFB compatible UART.
> 
> The R8A7795 doesn't have SCIFA and SCIFB.
> (In other words, it has SCIF and HSCIF.)

Grr !!
Thank you for pointing it. will fix in v5

Best regards
---
Kuninori Morimoto
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Magnus Damm Aug. 6, 2015, 11:03 p.m. UTC | #3
Hi Morimoto-san,

On Thu, Aug 6, 2015 at 5:46 PM, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>
> Hi Shimoda-san
>
>> > +    - "renesas,scifa-r8a7795" for R8A7795 (R-Car H3) SCIFA compatible UART.
>> > +    - "renesas,scifb-r8a7795" for R8A7795 (R-Car H3) SCIFB compatible UART.
>>
>> The R8A7795 doesn't have SCIFA and SCIFB.
>> (In other words, it has SCIF and HSCIF.)
>
> Grr !!
> Thank you for pointing it. will fix in v5

Thanks, at the same time can you break out the DT binding patch from
the DTS changes? They most likely go through different trees.

Best,

/ magnus
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kuninori Morimoto Aug. 7, 2015, 4:21 a.m. UTC | #4
Hi Magnus

> > Grr !!
> > Thank you for pointing it. will fix in v5
> 
> Thanks, at the same time can you break out the DT binding patch from
> the DTS changes? They most likely go through different trees.

Thanks !
I believe I did it on v5 patchset


Best regards
---
Kuninori Morimoto
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
index e84b13a..8c97599 100644
--- a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
+++ b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
@@ -23,6 +23,10 @@  Required properties:
     - "renesas,scifa-r8a7794" for R8A7794 (R-Car E2) SCIFA compatible UART.
     - "renesas,scifb-r8a7794" for R8A7794 (R-Car E2) SCIFB compatible UART.
     - "renesas,hscif-r8a7794" for R8A7794 (R-Car E2) HSCIF compatible UART.
+    - "renesas,scif-r8a7795" for R8A7795 (R-Car H3) SCIF compatible UART.
+    - "renesas,scifa-r8a7795" for R8A7795 (R-Car H3) SCIFA compatible UART.
+    - "renesas,scifb-r8a7795" for R8A7795 (R-Car H3) SCIFB compatible UART.
+    - "renesas,hscif-r8a7795" for R8A7795 (R-Car H3) HSCIF compatible UART.
     - "renesas,scifa-sh73a0" for SH73A0 (SH-Mobile AG5) SCIFA compatible UART.
     - "renesas,scifb-sh73a0" for SH73A0 (SH-Mobile AG5) SCIFB compatible UART.
     - "renesas,scif" for generic SCIF compatible UART.
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 2375b2d..db0828a 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -80,6 +80,10 @@ 
 			};
 
 			cpg_clocks: cpg_clocks@e6150000 {
+				#address-cells = <2>;
+				#size-cells = <2>;
+				ranges;
+
 				compatible = "renesas,r8a7795-cpg-clocks",
 					     "renesas,rcar-gen3-cpg-clocks";
 				reg = <0 0xe6150000 0 0x1000>;
@@ -87,7 +91,24 @@ 
 				#clock-cells = <1>;
 				clock-output-names = "main", "pll0", "pll1","pll2",
 						     "pll3", "pll4";
+
+				mstp3_clks: mstp3_clks@e615013c {
+					compatible = "renesas,r8a7795-mstp-clocks",
+						     "renesas,cpg-mstp-clocks";
+					reg = <0 0xe615013c 0 4>, <0 0xe6150048 0 4>;
+					clocks =  <&p_clk>;
+					#clock-cells = <1>;
+					renesas,clock-indices = <RCAR_R8A7795_CLK_SCIF2>;
+					clock-output-names = "scif2";
+				};
 			};
 		};
+		scif2: serial@e6e88000 {
+			compatible = "renesas,scif-r8a7795", "renesas,scif";
+			reg = <0 0xe6e88000 0 64>;
+			interrupts = <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&mstp3_clks RCAR_R8A7795_CLK_SCIF2>;
+			clock-names = "sci_ick";
+		};
 	};
 };
diff --git a/include/dt-bindings/clock/r8a7795-clock.h b/include/dt-bindings/clock/r8a7795-clock.h
index 4097b0b..43e942a 100644
--- a/include/dt-bindings/clock/r8a7795-clock.h
+++ b/include/dt-bindings/clock/r8a7795-clock.h
@@ -16,6 +16,7 @@ 
 /* MSTP2 */
 
 /* MSTP3 */
+#define RCAR_R8A7795_CLK_SCIF2		10
 
 /* MSTP5 */