Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Sekhar Nori
On Thursday 20 October 2016 03:51 PM, Tomi Valkeinen wrote: > On 20/10/16 13:07, Sekhar Nori wrote: > >> Per me, compatible property is an ordered list precisely for the reason >> that things should continue to "work" with as closely matched driver as >> possible. So even if someone is running a

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Sekhar Nori
On Thursday 20 October 2016 03:51 PM, Tomi Valkeinen wrote: > On 20/10/16 13:07, Sekhar Nori wrote: > >> Per me, compatible property is an ordered list precisely for the reason >> that things should continue to "work" with as closely matched driver as >> possible. So even if someone is running a

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Tomi Valkeinen
On 20/10/16 13:07, Sekhar Nori wrote: > Per me, compatible property is an ordered list precisely for the reason > that things should continue to "work" with as closely matched driver as > possible. So even if someone is running a kernel which does not > recognize "ti,da850-tilcdc", it should

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Tomi Valkeinen
On 20/10/16 13:07, Sekhar Nori wrote: > Per me, compatible property is an ordered list precisely for the reason > that things should continue to "work" with as closely matched driver as > possible. So even if someone is running a kernel which does not > recognize "ti,da850-tilcdc", it should

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Sekhar Nori
On Monday 17 October 2016 07:31 PM, Bartosz Golaszewski wrote: > 2016-10-17 14:29 GMT+02:00 Tomi Valkeinen : >> On 17/10/16 14:40, Laurent Pinchart wrote: >>> Hello, >>> >>> On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: On 17/10/16 10:12, Sekhar Nori wrote: >

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-20 Thread Sekhar Nori
On Monday 17 October 2016 07:31 PM, Bartosz Golaszewski wrote: > 2016-10-17 14:29 GMT+02:00 Tomi Valkeinen : >> On 17/10/16 14:40, Laurent Pinchart wrote: >>> Hello, >>> >>> On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: On 17/10/16 10:12, Sekhar Nori wrote: > On Monday 17 October

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Bartosz Golaszewski
2016-10-17 14:29 GMT+02:00 Tomi Valkeinen : > On 17/10/16 14:40, Laurent Pinchart wrote: >> Hello, >> >> On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: >>> On 17/10/16 10:12, Sekhar Nori wrote: On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Bartosz Golaszewski
2016-10-17 14:29 GMT+02:00 Tomi Valkeinen : > On 17/10/16 14:40, Laurent Pinchart wrote: >> Hello, >> >> On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: >>> On 17/10/16 10:12, Sekhar Nori wrote: On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On 15/10/16 20:42, Sekhar

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Laurent Pinchart
Hi Tomi, On Monday 17 Oct 2016 15:29:23 Tomi Valkeinen wrote: > On 17/10/16 14:40, Laurent Pinchart wrote: > > On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: > >> On 17/10/16 10:12, Sekhar Nori wrote: > >>> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On 15/10/16 20:42,

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Laurent Pinchart
Hi Tomi, On Monday 17 Oct 2016 15:29:23 Tomi Valkeinen wrote: > On 17/10/16 14:40, Laurent Pinchart wrote: > > On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: > >> On 17/10/16 10:12, Sekhar Nori wrote: > >>> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On 15/10/16 20:42,

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Tomi Valkeinen
On 17/10/16 14:40, Laurent Pinchart wrote: > Hello, > > On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: >> On 17/10/16 10:12, Sekhar Nori wrote: >>> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Tomi Valkeinen
On 17/10/16 14:40, Laurent Pinchart wrote: > Hello, > > On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: >> On 17/10/16 10:12, Sekhar Nori wrote: >>> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Laurent Pinchart
Hello, On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: > On 17/10/16 10:12, Sekhar Nori wrote: >> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >>> On 15/10/16 20:42, Sekhar Nori wrote: > diff --git a/arch/arm/boot/dts/da850.dtsi > b/arch/arm/boot/dts/da850.dtsi >

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Laurent Pinchart
Hello, On Monday 17 Oct 2016 10:33:58 Tomi Valkeinen wrote: > On 17/10/16 10:12, Sekhar Nori wrote: >> On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >>> On 15/10/16 20:42, Sekhar Nori wrote: > diff --git a/arch/arm/boot/dts/da850.dtsi > b/arch/arm/boot/dts/da850.dtsi >

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Tomi Valkeinen
On 17/10/16 10:12, Sekhar Nori wrote: > On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >> On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi index f79e1b9..32908ae 100644 --- a/arch/arm/boot/dts/da850.dtsi

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Tomi Valkeinen
On 17/10/16 10:12, Sekhar Nori wrote: > On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >> On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi index f79e1b9..32908ae 100644 --- a/arch/arm/boot/dts/da850.dtsi

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Bartosz Golaszewski
2016-10-17 9:12 GMT+02:00 Sekhar Nori : > On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >> On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi index f79e1b9..32908ae 100644 ---

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Bartosz Golaszewski
2016-10-17 9:12 GMT+02:00 Sekhar Nori : > On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: >> On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi index f79e1b9..32908ae 100644 --- a/arch/arm/boot/dts/da850.dtsi

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Sekhar Nori
On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On 15/10/16 20:42, Sekhar Nori wrote: > >>> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi >>> index f79e1b9..32908ae 100644 >>> --- a/arch/arm/boot/dts/da850.dtsi >>> +++ b/arch/arm/boot/dts/da850.dtsi >> >>>

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-17 Thread Sekhar Nori
On Monday 17 October 2016 11:26 AM, Tomi Valkeinen wrote: > On 15/10/16 20:42, Sekhar Nori wrote: > >>> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi >>> index f79e1b9..32908ae 100644 >>> --- a/arch/arm/boot/dts/da850.dtsi >>> +++ b/arch/arm/boot/dts/da850.dtsi >> >>>

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-16 Thread Tomi Valkeinen
On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi >> index f79e1b9..32908ae 100644 >> --- a/arch/arm/boot/dts/da850.dtsi >> +++ b/arch/arm/boot/dts/da850.dtsi > >> @@ -399,6 +420,14 @@ >> < 0 1>; >>

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-16 Thread Tomi Valkeinen
On 15/10/16 20:42, Sekhar Nori wrote: >> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi >> index f79e1b9..32908ae 100644 >> --- a/arch/arm/boot/dts/da850.dtsi >> +++ b/arch/arm/boot/dts/da850.dtsi > >> @@ -399,6 +420,14 @@ >> < 0 1>; >>

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-15 Thread Bartosz Golaszewski
2016-10-15 19:42 GMT+02:00 Sekhar Nori : > On Wednesday 05 October 2016 06:35 PM, Bartosz Golaszewski wrote: >> From: Karl Beldan >> >> Add pins used by the LCD controller and a disabled LCDC node to be >> reused in device trees including da850.dtsi. >> >>

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-15 Thread Bartosz Golaszewski
2016-10-15 19:42 GMT+02:00 Sekhar Nori : > On Wednesday 05 October 2016 06:35 PM, Bartosz Golaszewski wrote: >> From: Karl Beldan >> >> Add pins used by the LCD controller and a disabled LCDC node to be >> reused in device trees including da850.dtsi. >> >> Signed-off-by: Karl Beldan >> [Bartosz:

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-15 Thread Sekhar Nori
On Wednesday 05 October 2016 06:35 PM, Bartosz Golaszewski wrote: > From: Karl Beldan > > Add pins used by the LCD controller and a disabled LCDC node to be > reused in device trees including da850.dtsi. > > Signed-off-by: Karl Beldan > [Bartosz: >

Re: [PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-15 Thread Sekhar Nori
On Wednesday 05 October 2016 06:35 PM, Bartosz Golaszewski wrote: > From: Karl Beldan > > Add pins used by the LCD controller and a disabled LCDC node to be > reused in device trees including da850.dtsi. > > Signed-off-by: Karl Beldan > [Bartosz: > - added the commit description > -

[PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-05 Thread Bartosz Golaszewski
From: Karl Beldan Add pins used by the LCD controller and a disabled LCDC node to be reused in device trees including da850.dtsi. Signed-off-by: Karl Beldan [Bartosz: - added the commit description - changed the dt node name to a generic one -

[PATCH 2/2] ARM: dts: da850: add a node for the LCD controller

2016-10-05 Thread Bartosz Golaszewski
From: Karl Beldan Add pins used by the LCD controller and a disabled LCDC node to be reused in device trees including da850.dtsi. Signed-off-by: Karl Beldan [Bartosz: - added the commit description - changed the dt node name to a generic one - added a da850-specific compatible string -