Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-13 Thread Linus Walleij
On Thu, Mar 6, 2014 at 11:21 AM, Tomi Valkeinen tomi.valkei...@ti.com wrote: Hi Pawel, On 05/03/14 18:23, Pawel Moll wrote: This patch adds basic DT bindings for the PL11x CLCD cells and make their fbdev driver use them. Is this an old HW, and presumably there won't be new users for it?

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-13 Thread Pawel Moll
On Thu, 2014-03-13 at 09:30 +, Linus Walleij wrote: On Thu, Mar 6, 2014 at 11:21 AM, Tomi Valkeinen tomi.valkei...@ti.com wrote: On 05/03/14 18:23, Pawel Moll wrote: This patch adds basic DT bindings for the PL11x CLCD cells and make their fbdev driver use them. Is this an old HW,

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-13 Thread Russell King - ARM Linux
On Mon, Mar 10, 2014 at 02:23:21PM +0200, Tomi Valkeinen wrote: On 10/03/14 14:09, Pawel Moll wrote: Note that the same DT bindings you add here should also work for the DRM driver in the future. So, in fact, the question is extended to: will the fbdev No, I don't think so. It's a

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-13 Thread Pawel Moll
On Mon, 2014-03-10 at 12:23 +, Tomi Valkeinen wrote: The DT describes the hardware, and there should be only one description of it, no matter what the used SW is. I see compatible = arm,pl111 in the dts. PL111 is the LCD controller, right? Correct. What will the DRM driver use, then,

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-10 Thread Pawel Moll
On Fri, 2014-03-07 at 11:35 +, Tomi Valkeinen wrote: ... and this makes me try to get out of its way. In other words, I fully agree with you, but I don't think this applies to this particular patch, as I'm not even trying to represent the display pipeline. The (optional) Right, you

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-10 Thread Tomi Valkeinen
On 10/03/14 14:09, Pawel Moll wrote: Note that the same DT bindings you add here should also work for the DRM driver in the future. So, in fact, the question is extended to: will the fbdev No, I don't think so. It's a end of line for it, I believe. From my personal point of view the main

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-07 Thread Tomi Valkeinen
On 06/03/14 17:23, Pawel Moll wrote: On Thu, 2014-03-06 at 10:21 +, Tomi Valkeinen wrote: Is this an old HW, and presumably there won't be new users for it? If yes, this is probably fine. There is a DRM driver in the works, actually... If not, you might want to look at the video

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-06 Thread Tomi Valkeinen
Hi Pawel, On 05/03/14 18:23, Pawel Moll wrote: This patch adds basic DT bindings for the PL11x CLCD cells and make their fbdev driver use them. Is this an old HW, and presumably there won't be new users for it? If yes, this is probably fine. If not, you might want to look at the video ports

Re: [PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-06 Thread Pawel Moll
On Thu, 2014-03-06 at 10:21 +, Tomi Valkeinen wrote: Is this an old HW, and presumably there won't be new users for it? If yes, this is probably fine. There is a DRM driver in the works, actually... If not, you might want to look at the video ports and endpoints, which is used by at

[PATCH v5 1/2] video: ARM CLCD: Add DT support

2014-03-05 Thread Pawel Moll
This patch adds basic DT bindings for the PL11x CLCD cells and make their fbdev driver use them. Signed-off-by: Pawel Moll pawel.m...@arm.com --- Changes since v4: - simplified the pads description property and made it optional Changes since v3: - changed wording and order of interrupt-names and