Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1

[Simon Horman]

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  wrote:
> >> From: Marek Vasut 
> >>
> >> Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
> >> since they are available on connectors CN43, CN44 respectively.
> >>
> >> Signed-off-by: Marek Vasut 
> > 
> > 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.

Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1

[Marek Vasut]

On 3/11/19 10:46 AM, Geert Uytterhoeven wrote:
> Hi Marek,
> 
> On Sat, Mar 9, 2019 at 3:06 AM  wrote:
>> From: Marek Vasut 
>>
>> Enable both CAN0 and CAN1 controllers on R8A77995 Draak board,
>> since they are available on connectors CN43, CN44 respectively.
>>
>> Signed-off-by: Marek Vasut 
> 
> Thanks for your patch!
> I assume you have tested this with your CAN-analyzer?

Nope, only remotely between two CAN ports thus far.

-- 
Best regards,
Marek Vasut

Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1

[Geert Uytterhoeven]

Hi Marek,

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

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

Reviewed-by: Geert Uytterhoeven 

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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

[PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1

[marek . vasut]

From: Marek Vasut 

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

Signed-off-by: Marek Vasut 
Cc: Geert Uytterhoeven 
Cc: Simon Horman 
Cc: Wolfram Sang 
Cc: linux-renesas-soc@vger.kernel.org
To: linux-arm-ker...@lists.infradead.org
---
 .../arm64/boot/dts/renesas/r8a77995-draak.dts | 22 +++
 1 file changed, 22 insertions(+)

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";
-- 
2.20.1