Re: [PATCH v1 2/3] drm: rcar-du: Add suspend resume helpers

2017-09-15 Thread Laurent Pinchart
Hi Kieran, On Friday, 15 September 2017 20:49:15 EEST Kieran Bingham wrote: > On 15/09/17 18:02, Laurent Pinchart wrote: > > On Friday, 15 September 2017 19:42:06 EEST Kieran Bingham wrote: > >> The pipeline needs to ensure that the hardware is idle for suspend and > >> resume operations. > > > >

Re: [PATCH v1 2/3] drm: rcar-du: Add suspend resume helpers

2017-09-15 Thread Kieran Bingham
Hi Laurent, Thanks for your speedy review! On 15/09/17 18:02, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Friday, 15 September 2017 19:42:06 EEST Kieran Bingham wrote: >> The pipeline needs to ensure that the hardware is idle for suspend and >> resume operations. >

Re: [PATCH v1 2/3] drm: rcar-du: Add suspend resume helpers

2017-09-15 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Friday, 15 September 2017 19:42:06 EEST Kieran Bingham wrote: > The pipeline needs to ensure that the hardware is idle for suspend and > resume operations. I'm not sure to really understand this sentence. > Implement suspend and resume functions using the

[PATCH v1 2/3] drm: rcar-du: Add suspend resume helpers

2017-09-15 Thread Kieran Bingham
The pipeline needs to ensure that the hardware is idle for suspend and resume operations. Implement suspend and resume functions using the DRM atomic helper functions. CC: dri-de...@lists.freedesktop.org Signed-off-by: Kieran Bingham --- drivers/gpu/drm/rcar-du/rcar_du_drv.c | 18 +