Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Simon Horman
On Thu, Feb 15, 2018 at 08:38:20AM +, Kieran Bingham wrote: > From: Kieran Bingham > > The r8a77995 D3 platform has 2 LVDS channels connected to the DU. > > Signed-off-by: Kieran Bingham It looks like there has been some review of this patch that needs addressing, I've marked it as "Change

Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Kieran Bingham
On 15/02/18 14:10, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Thursday, 15 February 2018 10:38:20 EET Kieran Bingham wrote: >> From: Kieran Bingham >> >> The r8a77995 D3 platform has 2 LVDS channels connected to the DU. >> >> Signed-off-by: Kieran Bingham >> --- >>

Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Thursday, 15 February 2018 10:38:20 EET Kieran Bingham wrote: > From: Kieran Bingham > > The r8a77995 D3 platform has 2 LVDS channels connected to the DU. > > Signed-off-by: Kieran Bingham > --- > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 52 +

Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Geert Uytterhoeven
Hi Kieran, On Thu, Feb 15, 2018 at 1:41 PM, Kieran Bingham wrote: > On 15/02/18 09:07, Geert Uytterhoeven wrote: >> On Thu, Feb 15, 2018 at 9:38 AM, Kieran Bingham wrote: >>> From: Kieran Bingham >>> >>> The r8a77995 D3 platform has 2 LVDS channels connected to the DU. >>> >>> Signed-off-by: Ki

Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Kieran Bingham
Hi Geert, On 15/02/18 09:07, Geert Uytterhoeven wrote: > Hi Kieran, > > On Thu, Feb 15, 2018 at 9:38 AM, Kieran Bingham wrote: >> From: Kieran Bingham >> >> The r8a77995 D3 platform has 2 LVDS channels connected to the DU. >> >> Signed-off-by: Kieran Bingham > > I love your patch! > Yet somet

Re: [PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Geert Uytterhoeven
Hi Kieran, On Thu, Feb 15, 2018 at 9:38 AM, Kieran Bingham wrote: > From: Kieran Bingham > > The r8a77995 D3 platform has 2 LVDS channels connected to the DU. > > Signed-off-by: Kieran Bingham I love your patch! Yet something to improve... > --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi > +

[PATCH 5/8] arm64: dts: renesas: r8a77995: Add LVDS support

2018-02-15 Thread Kieran Bingham
From: Kieran Bingham The r8a77995 D3 platform has 2 LVDS channels connected to the DU. Signed-off-by: Kieran Bingham --- arch/arm64/boot/dts/renesas/r8a77995.dtsi | 52 +++ 1 file changed, 52 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/a