Re: [PATCH 2/2] drm: hdlcd: Suspend/resume only active crtcs

2016-05-05 Thread Robin Murphy
Hi Daniel, On 05/05/16 18:06, Daniel Vetter wrote: On Thu, May 05, 2016 at 05:13:38PM +0100, Robin Murphy wrote: The current PM ops simply unconditionally enable/disable the HDLCD, which proves problematic when there is no display plugged in - since without a crtc the hardware itself is still i

Re: [PATCH 2/2] drm: hdlcd: Suspend/resume only active crtcs

2016-05-05 Thread liviu . dudau
On Thu, May 05, 2016 at 07:06:02PM +0200, Daniel Vetter wrote: > On Thu, May 05, 2016 at 05:13:38PM +0100, Robin Murphy wrote: > > The current PM ops simply unconditionally enable/disable the HDLCD, > > which proves problematic when there is no display plugged in - since > > without a crtc the hard

Re: [PATCH 2/2] drm: hdlcd: Suspend/resume only active crtcs

2016-05-05 Thread Daniel Vetter
On Thu, May 05, 2016 at 05:13:38PM +0100, Robin Murphy wrote: > The current PM ops simply unconditionally enable/disable the HDLCD, > which proves problematic when there is no display plugged in - since > without a crtc the hardware itself is still in an uninitialised state, > coming out of suspend

Re: [PATCH 2/2] drm: hdlcd: Suspend/resume only active crtcs

2016-05-05 Thread liviu . dudau
On Thu, May 05, 2016 at 05:13:38PM +0100, Robin Murphy wrote: > The current PM ops simply unconditionally enable/disable the HDLCD, > which proves problematic when there is no display plugged in - since > without a crtc the hardware itself is still in an uninitialised state, > coming out of suspend

[PATCH 2/2] drm: hdlcd: Suspend/resume only active crtcs

2016-05-05 Thread Robin Murphy
The current PM ops simply unconditionally enable/disable the HDLCD, which proves problematic when there is no display plugged in - since without a crtc the hardware itself is still in an uninitialised state, coming out of suspend results in it being enabled without a valid framebuffer address, whic