Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-30 Thread Simon Horman
On Mon, Aug 27, 2018 at 09:28:09PM +0200, Eugeniu Rosca wrote: > Hi Simon, hi Geert, > > On Mon, Aug 27, 2018 at 02:44:47PM +0200, Simon Horman wrote: > > On Thu, Aug 23, 2018 at 10:52:09AM +0200, Geert Uytterhoeven wrote: > > > On Fri, Aug 17, 2018 at 3:53 PM Kieran Bingham > > > wrote: > > > >

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-27 Thread Eugeniu Rosca
Hi Simon, hi Geert, On Mon, Aug 27, 2018 at 02:44:47PM +0200, Simon Horman wrote: > On Thu, Aug 23, 2018 at 10:52:09AM +0200, Geert Uytterhoeven wrote: > > On Fri, Aug 17, 2018 at 3:53 PM Kieran Bingham > > wrote: > > > On 12/08/18 14:31, Eugeniu Rosca wrote: > > > > According to R-Car Gen3 HW ma

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-27 Thread Eugeniu Rosca
Hi Kieran, I appreciate the detailed reply zooming in into many aspects (both technical and related to process/workflow) of contributing/reviewing patches. I take it as is. I could elaborate on specific parts of it, like applying the "undefined behavior" term (which comes from the world of compile

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-27 Thread Simon Horman
On Thu, Aug 23, 2018 at 10:52:09AM +0200, Geert Uytterhoeven wrote: > On Fri, Aug 17, 2018 at 3:53 PM Kieran Bingham > wrote: > > On 12/08/18 14:31, Eugeniu Rosca wrote: > > > According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > > > interfaces, similar to H3, M3-W and other SoCs fro

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-23 Thread Kieran Bingham
Hi Eugeniu, On 23/08/18 18:14, Eugeniu Rosca wrote: > Dear reviewers, > > On Thu, Aug 23, 2018 at 11:01:46AM +0200, Geert Uytterhoeven wrote: >> Hi Sergei, >> >> On Thu, Aug 23, 2018 at 10:56 AM Sergei Shtylyov >> wrote: >>> On 8/23/2018 11:52 AM, Geert Uytterhoeven wrote: >> According to R-

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-23 Thread Eugeniu Rosca
Dear reviewers, On Thu, Aug 23, 2018 at 11:01:46AM +0200, Geert Uytterhoeven wrote: > Hi Sergei, > > On Thu, Aug 23, 2018 at 10:56 AM Sergei Shtylyov > wrote: > > On 8/23/2018 11:52 AM, Geert Uytterhoeven wrote: > > >>> According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > > >>> in

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-23 Thread Geert Uytterhoeven
Hi Sergei, On Thu, Aug 23, 2018 at 10:56 AM Sergei Shtylyov wrote: > On 8/23/2018 11:52 AM, Geert Uytterhoeven wrote: > >>> According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > >>> interfaces, similar to H3, M3-W and other SoCs from the same family. > >>> > >>> Add CAN placeholder

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-23 Thread Sergei Shtylyov
On 8/23/2018 11:52 AM, Geert Uytterhoeven wrote: According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN interfaces, similar to H3, M3-W and other SoCs from the same family. Add CAN placeholder nodes to avoid below DTC errors: Error: arch/arm64/boot/dts/renesas/ulcb-kf.dtsi:19.1-6 Lab

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-23 Thread Geert Uytterhoeven
On Fri, Aug 17, 2018 at 3:53 PM Kieran Bingham wrote: > On 12/08/18 14:31, Eugeniu Rosca wrote: > > According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > > interfaces, similar to H3, M3-W and other SoCs from the same family. > > > > Add CAN placeholder nodes to avoid below DTC errors

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-22 Thread Simon Horman
On Fri, Aug 17, 2018 at 02:53:37PM +0100, Kieran Bingham wrote: > Hi Eugeniu, > > Thank you for the patch, > > On 12/08/18 14:31, Eugeniu Rosca wrote: > > According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > > interfaces, similar to H3, M3-W and other SoCs from the same family. > >

Re: [PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-17 Thread Kieran Bingham
Hi Eugeniu, Thank you for the patch, On 12/08/18 14:31, Eugeniu Rosca wrote: > According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN > interfaces, similar to H3, M3-W and other SoCs from the same family. > > Add CAN placeholder nodes to avoid below DTC errors: > Error: arch/arm64/boo

[PATCH v2 5/7] arm64: dts: renesas: r8a77965: Add CAN{0,1} placeholder nodes

2018-08-12 Thread Eugeniu Rosca
According to R-Car Gen3 HW manual rev1.00, R-Car M3-N has two CAN interfaces, similar to H3, M3-W and other SoCs from the same family. Add CAN placeholder nodes to avoid below DTC errors: Error: arch/arm64/boot/dts/renesas/ulcb-kf.dtsi:19.1-6 Label or path can0 not found Error: arch/arm64/boot/dt