Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-03 Thread Liviu Dudau
On Thu, Dec 03, 2015 at 02:20:26PM +, Daniel Stone wrote: > Hi Liviu, > > On 3 December 2015 at 10:00, Liviu Dudau wrote: > > On Wed, Dec 02, 2015 at 05:21:44PM +, Daniel Stone wrote: > >> On 2 December 2015 at 12:23, Liviu Dudau wrote: > >> > + if (irq_status & HDLCD_INTERRUPT_VSY

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-03 Thread Daniel Stone
Hi Liviu, On 3 December 2015 at 10:00, Liviu Dudau wrote: > On Wed, Dec 02, 2015 at 05:21:44PM +, Daniel Stone wrote: >> On 2 December 2015 at 12:23, Liviu Dudau wrote: >> > + if (irq_status & HDLCD_INTERRUPT_VSYNC) { >> > + unsigned long flags; >> > + >> > +

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-03 Thread Liviu Dudau
On Wed, Dec 02, 2015 at 05:21:44PM +, Daniel Stone wrote: > Hi Liviu, Hi Daniel, > > On 2 December 2015 at 12:23, Liviu Dudau wrote: > > + if (irq_status & HDLCD_INTERRUPT_VSYNC) { > > + unsigned long flags; > > + > > + drm_handle_vblank(drm, 0); > > + > >

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Daniel Stone
Hi Liviu, On 2 December 2015 at 12:23, Liviu Dudau wrote: > + if (irq_status & HDLCD_INTERRUPT_VSYNC) { > + unsigned long flags; > + > + drm_handle_vblank(drm, 0); > + > + spin_lock_irqsave(&drm->event_lock, flags); > + if (hdlcd->even

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Liviu Dudau
On Wed, Dec 02, 2015 at 04:24:19PM +0100, Daniel Vetter wrote: > On Wed, Dec 02, 2015 at 12:23:00PM +, Liviu Dudau wrote: > > The HDLCD controller is a display controller that supports resolutions > > up to 4096x4096 pixels. It is present on various development boards > > produced by ARM Ltd an

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Liviu Dudau
On Wed, Dec 02, 2015 at 04:33:31PM +0100, Daniel Vetter wrote: > On Wed, Dec 02, 2015 at 04:24:19PM +0100, Daniel Vetter wrote: > > On Wed, Dec 02, 2015 at 12:23:00PM +, Liviu Dudau wrote: > > > The HDLCD controller is a display controller that supports resolutions > > > up to 4096x4096 pixels.

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Daniel Vetter
On Wed, Dec 02, 2015 at 04:24:19PM +0100, Daniel Vetter wrote: > On Wed, Dec 02, 2015 at 12:23:00PM +, Liviu Dudau wrote: > > The HDLCD controller is a display controller that supports resolutions > > up to 4096x4096 pixels. It is present on various development boards > > produced by ARM Ltd an

Re: [PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Daniel Vetter
On Wed, Dec 02, 2015 at 12:23:00PM +, Liviu Dudau wrote: > The HDLCD controller is a display controller that supports resolutions > up to 4096x4096 pixels. It is present on various development boards > produced by ARM Ltd and emulated by the latest Fast Models from the > company. > > Cc: David

[PATCH v3 2/4] drm: Add support for ARM's HDLCD controller.

2015-12-02 Thread Liviu Dudau
The HDLCD controller is a display controller that supports resolutions up to 4096x4096 pixels. It is present on various development boards produced by ARM Ltd and emulated by the latest Fast Models from the company. Cc: David Airlie Cc: Robin Murphy Signed-off-by: Liviu Dudau --- drivers/gpu/