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