Re: [PATCH v1.1 2/2] drm: rcar-du: Repair vblank for DRM page flips using the VSP1

2017-07-02 Thread Laurent Pinchart
Hi Morimoto-san, On Friday 30 Jun 2017 08:32:04 Kuninori Morimoto wrote: > Hi Kieran > > > -static void rcar_du_vsp_complete(void *private) > > +static void rcar_du_vsp_complete(void *private, bool completed) > > { > > struct rcar_du_crtc *crtc = private; > > > > -

Re: [PATCH v1.1 2/2] drm: rcar-du: Repair vblank for DRM page flips using the VSP1

2017-06-30 Thread Kuninori Morimoto
Hi Kieran > -static void rcar_du_vsp_complete(void *private) > +static void rcar_du_vsp_complete(void *private, bool completed) > { > struct rcar_du_crtc *crtc = private; > > - rcar_du_crtc_finish_page_flip(crtc); > + if (crtc->vblank_enable) > +

[PATCH v1.1 2/2] drm: rcar-du: Repair vblank for DRM page flips using the VSP1

2017-06-29 Thread Kieran Bingham
A recent change to the frame completion handling has changed the order in which the vblank timestamps are updated. To fix this requires handling the vblank events on the frame end event which comes from the VSP1 driver on Gen3 instead. Prevent the CRTC IRQ from being enabled on Gen3 hardware and