All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
@ 2019-03-09  2:06 ` marek.vasut
  0 siblings, 0 replies; 8+ messages in thread
From: marek.vasut @ 2019-03-09  2:06 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marek Vasut, Geert Uytterhoeven, Simon Horman, Wolfram Sang,
	linux-renesas-soc

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(+)

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


^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
@ 2019-03-09  2:06 ` marek.vasut
  0 siblings, 0 replies; 8+ messages in thread
From: marek.vasut @ 2019-03-09  2:06 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: linux-renesas-soc, Simon Horman, Wolfram Sang,
	Geert Uytterhoeven, Marek Vasut

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(+)

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


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
  2019-03-09  2:06 ` marek.vasut
@ 2019-03-11  9:46   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 8+ messages in thread
From: Geert Uytterhoeven @ 2019-03-11  9:46 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Linux ARM, Marek Vasut, Geert Uytterhoeven, Simon Horman,
	Wolfram Sang, Linux-Renesas

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

-- 
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

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
@ 2019-03-11  9:46   ` Geert Uytterhoeven
  0 siblings, 0 replies; 8+ messages in thread
From: Geert Uytterhoeven @ 2019-03-11  9:46 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Geert Uytterhoeven, Linux-Renesas, Wolfram Sang, Simon Horman,
	Linux ARM, Marek Vasut

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

-- 
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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
  2019-03-11  9:46   ` Geert Uytterhoeven
@ 2019-03-11 13:38     ` Marek Vasut
  -1 siblings, 0 replies; 8+ messages in thread
From: Marek Vasut @ 2019-03-11 13:38 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linux ARM, Marek Vasut, Geert Uytterhoeven, Simon Horman,
	Wolfram Sang, Linux-Renesas

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.

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
@ 2019-03-11 13:38     ` Marek Vasut
  0 siblings, 0 replies; 8+ messages in thread
From: Marek Vasut @ 2019-03-11 13:38 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Geert Uytterhoeven, Linux-Renesas, Wolfram Sang, Simon Horman,
	Linux ARM, Marek Vasut

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.

-- 
Best regards,
Marek Vasut

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
  2019-03-11 13:38     ` Marek Vasut
@ 2019-03-13 10:29       ` Simon Horman
  -1 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2019-03-13 10:29 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Geert Uytterhoeven, Linux ARM, Marek Vasut, Geert Uytterhoeven,
	Wolfram Sang, Linux-Renesas

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.

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1
@ 2019-03-13 10:29       ` Simon Horman
  0 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2019-03-13 10:29 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Geert Uytterhoeven, Linux-Renesas, Wolfram Sang,
	Geert Uytterhoeven, Linux ARM, Marek Vasut

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.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-03-13 10:30 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-09  2:06 [PATCH] arm64: dts: renesas: r8a77995: draak: Enable CAN0, CAN1 marek.vasut
2019-03-09  2:06 ` marek.vasut
2019-03-11  9:46 ` Geert Uytterhoeven
2019-03-11  9:46   ` Geert Uytterhoeven
2019-03-11 13:38   ` Marek Vasut
2019-03-11 13:38     ` Marek Vasut
2019-03-13 10:29     ` Simon Horman
2019-03-13 10:29       ` Simon Horman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.