Commit fe286893 authored by AngeloGioacchino Del Regno's avatar AngeloGioacchino Del Regno Committed by Rob Clark
Browse files

drm/msm/dpu: Remove unused call in wait_for_commit_done 



The call to dpu_encoder_phys_cmd_prepare_for_kickoff is useless as
it's unused because the serialize_wait4pp variable is never set to
true by .. anything, literally: remove the call.
While at it, also reduce indentation by inverting the check for
dpu_encoder_phys_cmd_is_master.

Signed-off-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent da9e7b76

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

+3 −8
Original line number Diff line number Diff line
@@ -685,20 +685,15 @@ static int dpu_encoder_phys_cmd_wait_for_tx_complete( 
static int dpu_encoder_phys_cmd_wait_for_commit_done(

		struct dpu_encoder_phys *phys_enc)

{

	int rc = 0;

	struct dpu_encoder_phys_cmd *cmd_enc;



	cmd_enc = to_dpu_encoder_phys_cmd(phys_enc);



	/* only required for master controller */

	if (dpu_encoder_phys_cmd_is_master(phys_enc))

		rc = _dpu_encoder_phys_cmd_wait_for_ctl_start(phys_enc);



	/* required for both controllers */

	if (!rc && cmd_enc->serialize_wait4pp)

		dpu_encoder_phys_cmd_prepare_for_kickoff(phys_enc);

	if (!dpu_encoder_phys_cmd_is_master(phys_enc))

		return 0;



	return rc;

	return _dpu_encoder_phys_cmd_wait_for_ctl_start(phys_enc);

}



static int dpu_encoder_phys_cmd_wait_for_vblank(