diff mbox series

arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1

Message ID 20190309020642.15985-1-marek.vasut@gmail.com (mailing list archive)
State Accepted
Commit 4162aa9db3d44691c6ca67eacfe0099723684506
Delegated to: Simon Horman
Headers show
Series arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1 | expand

Commit Message

Marek Vasut March 9, 2019, 2:06 a.m. UTC 
From: Marek Vasut <marek.vasut+renesas@gmail.com>

Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
since they are available on connectors CN43, CN44 respectively.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
Cc: linux-renesas-soc@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
 .../arm64/boot/dts/renesas/r8a77995-draak.dts | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

Comments

Geert Uytterhoeven March 11, 2019, 9:46 a.m. UTC | #1 
Hi Marek,

On Sat, Mar 9, 2019 at 3:06 AM <marek.vasut@gmail.com> wrote:
> From: Marek Vasut <marek.vasut+renesas@gmail.com>
>
> Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
> since they are available on connectors CN43, CN44 respectively.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>

Thanks for your patch!
I assume you have tested this with your CAN-analyzer?

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
Marek Vasut March 11, 2019, 1:38 p.m. UTC | #2 
On 3/11/19 10:46 AM, Geert Uytterhoeven wrote:
> Hi Marek,
> 
> On Sat, Mar 9, 2019 at 3:06 AM <marek.vasut@gmail.com> wrote:
>> From: Marek Vasut <marek.vasut+renesas@gmail.com>
>>
>> Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
>> since they are available on connectors CN43, CN44 respectively.
>>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> 
> Thanks for your patch!
> I assume you have tested this with your CAN-analyzer?

Nope, only remotely between two CAN ports thus far.
Simon Horman March 13, 2019, 10:29 a.m. UTC | #3 
On Mon, Mar 11, 2019 at 02:38:26PM +0100, Marek Vasut wrote:
> On 3/11/19 10:46 AM, Geert Uytterhoeven wrote:
> > Hi Marek,
> > 
> > On Sat, Mar 9, 2019 at 3:06 AM <marek.vasut@gmail.com> wrote:
> >> From: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>
> >> Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
> >> since they are available on connectors CN43, CN44 respectively.
> >>
> >> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> > 
> > Thanks for your patch!
> > I assume you have tested this with your CAN-analyzer?
> 
> Nope, only remotely between two CAN ports thus far.

Thanks, applied for v5.2.
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
index db2bed1751b8..9a000a75fe8e 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
@@ -179,6 +179,18 @@ 
 	};
 };
 
+&can0 {
+	pinctrl-0 = <&can0_pins>;
+	pinctrl-names = "default";
+	status = "okay";
+};
+
+&can1 {
+	pinctrl-0 = <&can1_pins>;
+	pinctrl-names = "default";
+	status = "okay";
+};
+
 &du {
 	pinctrl-0 = <&du_pins>;
 	pinctrl-names = "default";
@@ -375,6 +387,16 @@ 
 		};
 	};
 
+	can0_pins: can0 {
+		groups = "can0_data_a";
+		function = "can0";
+	};
+
+	can1_pins: can1 {
+		groups = "can1_data_a";
+		function = "can1";
+	};
+
 	du_pins: du {
 		groups = "du_rgb888", "du_sync", "du_disp", "du_clk_out_0";
 		function = "du";