On 2023-02-21 10:42:54, Jessica Zhang wrote:
> Add a NULL check before calling prepare_commit() in
> msm_atomic_commit_tail()
> 
> Signed-off-by: Jessica Zhang <quic_jessz...@quicinc.com>
> Reviewed-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suij...@somainline.org>

> ---
>  drivers/gpu/drm/msm/msm_atomic.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/msm/msm_atomic.c 
> b/drivers/gpu/drm/msm/msm_atomic.c
> index 1686fbb611fd..c8a0a5cc5ca5 100644
> --- a/drivers/gpu/drm/msm/msm_atomic.c
> +++ b/drivers/gpu/drm/msm/msm_atomic.c
> @@ -206,7 +206,8 @@ void msm_atomic_commit_tail(struct drm_atomic_state 
> *state)
>        * Now that there is no in-progress flush, prepare the
>        * current update:
>        */
> -     kms->funcs->prepare_commit(kms, state);
> +     if (kms->funcs->prepare_commit)
> +             kms->funcs->prepare_commit(kms, state);
>  
>       /*
>        * Push atomic updates down to hardware:
> -- 
> 2.39.2
> 

Reply via email to