On Fri, 20 Dec 2019 at 10:30, Zhihui Chen <chenzhihui4@huawei.com> wrote:

> both crtc_state->adjusted_mode.hdisplay and
> crtc_state->adjusted_mode.vdisplay are 0 when switch dpms off,
> return -EINVAL cause switch dpms off fail.
>
> Signed-off-by: Zhihui Chen <chenzhihui4@huawei.com>
>

Thanks for the patch.
Acked-by: Xinliang Liu <xinliang.liu@linaro.org>
Applied to drm-misc-next.


> ---
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> index 6527a97f68a3..722e369f30d4 100644
> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> @@ -80,6 +80,9 @@ static int hibmc_plane_atomic_check(struct drm_plane
> *plane,
>                 return -EINVAL;
>         }
>
> +       if (!crtc_state->enable)
> +               return 0;
> +
>         if (state->crtc_x + state->crtc_w >
>             crtc_state->adjusted_mode.hdisplay ||
>             state->crtc_y + state->crtc_h >
> --
> 2.20.1
>
>
>