Re: [RESEND 0/5] drm/mxsfb: Fix runtime PM for unpowering lcdif block
From: Sean Paul
Date: Wed Sep 26 2018 - 04:13:33 EST
On Mon, Sep 17, 2018 at 04:42:10PM +0300, Leonard Crestez wrote:
> Adding lcdif nodes to a power domain currently doesn't work, it results
> in black/corrupted screens or hangs. While the driver does enable
> runtime pm it does not deal correctly with the block being unpowered.
>
> ---
>
> All patches in this series have review tags from a while ago and I
> tested them again on top of next-20180913. No changes since last
> version: https://lkml.org/lkml/2018/8/27/299
>
> This series stalled so I reached out to Marek on IRC and he was
> surprised to be listed as maintainer and asked me to resend and add
> Daniel Vetter.
>
> Perhaps it would help to clarify that the pengutronix people should feel
> free to push patches in this area?
>
> Right now drm/imx is mostly for IPUv3 but there are other display output
> paths on imx, such as the LCDIF supported by this driver. This LCDIF
> block is included on imx8 so still quite relevant.
>
> Leonard Crestez (5):
> drm/mxsfb: Move axi clk enable/disable to crtc enable/disable
> drm/mxsfb: Fix initial corrupt frame when activating display
> drm/mxsfb: Add pm_runtime calls to pipe_enable/disable
> drm/mxsfb: Add PM_SLEEP support
> drm/mxsfb: Switch to drm_atomic_helper_commit_tail_rpm
The whole set is
Reviewed-by: Sean Paul <seanpaul@xxxxxxxxxxxx>
>
> drivers/gpu/drm/mxsfb/mxsfb_crtc.c | 53 +++++++++++++++++++-----------
> drivers/gpu/drm/mxsfb/mxsfb_drv.c | 40 ++++++++++++++++++++++
> 2 files changed, 74 insertions(+), 19 deletions(-)
>
> --
> 2.17.1
>
--
Sean Paul, Software Engineer, Google / Chromium OS