Re: [PATCH v2 4/7] drm/msm: dpu: Remove dpu_plane_sspp_enabled()

[Jeykumar Sankaran <jsanka@xxxxxxxxxxxxxx>]

On 2018-09-12 06:54, Sean Paul wrote:
> From: Sean Paul <seanpaul@xxxxxxxxxxxx>
>
> It's doing the same thing dpu_plane_enabled() is.
>
> Changes in v2:
> - None
>
> Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx>
> ---

Reviewed-by: Jeykumar Sankaran <jsanka@xxxxxxxxxxxxxx>

>  drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
> index 2b1f61114e1c..0c6798df1e6b 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c
> @@ -138,11 +138,6 @@ static bool dpu_plane_enabled(struct 
> drm_plane_state
> *state)
>  	return state && state->fb && state->crtc;
>  }
>
> -static bool dpu_plane_sspp_enabled(struct drm_plane_state *state)
> -{
> -	return state && state->crtc;
> -}
> -
>  /**
>   * _dpu_plane_calc_fill_level - calculate fill level of the given 
> source
> format
>   * @plane:		Pointer to drm plane
> @@ -1398,7 +1393,7 @@ static void dpu_plane_atomic_update(struct 
> drm_plane
> *plane,
>
>  	DPU_DEBUG_PLANE(pdpu, "\n");
>
> -	if (!dpu_plane_sspp_enabled(state)) {
> +	if (!dpu_plane_enabled(state)) {
>  		_dpu_plane_atomic_disable(plane, old_state);
>  	} else {
>  		int ret;

--
Jeykumar S