[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-25 Thread Shawn Guo
On Thu, Jul 24, 2014 at 11:56:32AM +0200, Marc Kleine-Budde wrote: > >> @@ -696,6 +696,44 @@ static int imx_drm_platform_remove(struct > >> platform_device *pdev) > >>return 0; > >> } > >> > >> +#if CONFIG_PM_SLEEP > > > > use #ifdef > > ...or remove #if/#ifdef and mark as __maybe_unused

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-25 Thread Shawn Guo
On Thu, Jul 24, 2014 at 11:38:52AM +0200, Philipp Zabel wrote: > Hi Shawn, > > Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: > > HDMI currently stops working after a system suspend/resume cycle. It > > turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get > >

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-25 Thread Shawn Guo
On Thu, Jul 24, 2014 at 11:47:55AM +0200, Lucas Stach wrote: > > +static int imx_drm_resume(struct device *dev) > > +{ > > + struct drm_device *drm_dev = dev_get_drvdata(dev); > > + struct drm_connector *connector; > > + > > + drm_modeset_lock_all(drm_dev); > > +

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-25 Thread Daniel Vetter
On Thu, Jul 24, 2014 at 10:59:36PM +0800, Shawn Guo wrote: > On Thu, Jul 24, 2014 at 12:38:59PM +0200, Daniel Vetter wrote: > > > > > +static int imx_drm_suspend(struct device *dev) > > > > > +{ > > > > > + struct drm_device *drm_dev = dev_get_drvdata(dev); > > > > > + struct drm_connector

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Shawn Guo
On Thu, Jul 24, 2014 at 12:38:59PM +0200, Daniel Vetter wrote: > > > > +static int imx_drm_suspend(struct device *dev) > > > > +{ > > > > + struct drm_device *drm_dev = dev_get_drvdata(dev); > > > > + struct drm_connector *connector; > > > > + > > > > +

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Shawn Guo
HDMI currently stops working after a system suspend/resume cycle. It turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get called from anywhere across suspend/resume cycle. The patch follows what exynos drm driver does to walk the list of connectors and call their .dpms

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Daniel Vetter
On Thu, Jul 24, 2014 at 10:54:30AM +0100, Russell King - ARM Linux wrote: > On Thu, Jul 24, 2014 at 11:47:55AM +0200, Lucas Stach wrote: > > Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: > > > HDMI currently stops working after a system suspend/resume cycle. It > > > turns out

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Marc Kleine-Budde
On 07/24/2014 11:47 AM, Lucas Stach wrote: > Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: >> HDMI currently stops working after a system suspend/resume cycle. It >> turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get >> called from anywhere across

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Lucas Stach
Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: > HDMI currently stops working after a system suspend/resume cycle. It > turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get > called from anywhere across suspend/resume cycle. > > The patch follows what exynos drm

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Philipp Zabel
Hi Shawn, Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: > HDMI currently stops working after a system suspend/resume cycle. It > turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get > called from anywhere across suspend/resume cycle. > > The patch follows what

[PATCH] imx-drm: imx-drm-core: add suspend/resume support

2014-07-24 Thread Russell King - ARM Linux
On Thu, Jul 24, 2014 at 11:47:55AM +0200, Lucas Stach wrote: > Am Donnerstag, den 24.07.2014, 17:17 +0800 schrieb Shawn Guo: > > HDMI currently stops working after a system suspend/resume cycle. It > > turns out that the cause is the imx-hdmi encoder .dpms hook doesn't get > > called from