Commit fa063950 authored by Sean Paul's avatar Sean Paul Committed by Rob Clark
Browse files

drm/msm/dpu: Remove useless checks in dpu_encoder 



A couple more useless checks to remove in dpu_encoder.

Reviewed-by: default avatarStephen Boyd <swboyd@chromium.org>
Reviewed-by: default avatarAbhinav Kumar <abhinavk@codeaurora.org>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarSean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210913175747.47456-10-sean@poorly.run #v1
Link: https://patchwork.freedesktop.org/patch/msgid/20210915203834.1439-10-sean@poorly.run #v2
Link: https://patchwork.freedesktop.org/patch/msgid/20211001151145.55916-10-sean@poorly.run #v3

Changes in v2:
-None
Changes in v3:
-None
Changes in v4:
-None
Link: https://lore.kernel.org/r/20211105030434.2828845-10-sean@poorly.run



Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent b4e7ba4a

drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

+0 −12
Original line number Diff line number Diff line
@@ -1148,10 +1148,6 @@ static void dpu_encoder_virt_enable(struct drm_encoder *drm_enc) 
	struct msm_drm_private *priv;

	struct drm_display_mode *cur_mode = NULL;



	if (!drm_enc) {

		DPU_ERROR("invalid encoder\n");

		return;

	}

	dpu_enc = to_dpu_encoder_virt(drm_enc);



	mutex_lock(&dpu_enc->enc_lock);
@@ -1197,14 +1193,6 @@ static void dpu_encoder_virt_disable(struct drm_encoder *drm_enc) 
	struct msm_drm_private *priv;

	int i = 0;



	if (!drm_enc) {

		DPU_ERROR("invalid encoder\n");

		return;

	} else if (!drm_enc->dev) {

		DPU_ERROR("invalid dev\n");

		return;

	}



	dpu_enc = to_dpu_encoder_virt(drm_enc);

	DPU_DEBUG_ENC(dpu_enc, "\n");