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: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: Rob Clark <robdclark@chromium.org>

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c
index 665eb1d4cb8a4b56f517d26cb478edb381490a18..b2be39b9144e44904904f95a5d493c5c4e4eb1fd 100644 (file)
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c
@@ -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(