Commit fa560afa authored by Dmitry Baryshkov's avatar Dmitry Baryshkov
Browse files

drm/msm: loop over encoders using drm_for_each_encoder() 



Rather than manually looping over encoders array, use standard
drm_for_each_encoder() macro.

Reviewed-by: default avatarStephen Boyd <swboyd@chromium.org>
Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/480942/
Link: https://lore.kernel.org/r/20220406101247.483649-3-dmitry.baryshkov@linaro.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent 745b5158

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

+3 −2
Original line number Diff line number Diff line
@@ -685,6 +685,7 @@ static int _dpu_kms_drm_obj_init(struct dpu_kms *dpu_kms) 
	struct drm_plane *primary_planes[MAX_PLANES], *plane;

	struct drm_plane *cursor_planes[MAX_PLANES] = { NULL };

	struct drm_crtc *crtc;

	struct drm_encoder *encoder;



	struct msm_drm_private *priv;

	struct dpu_mdss_cfg *catalog;
@@ -749,8 +750,8 @@ static int _dpu_kms_drm_obj_init(struct dpu_kms *dpu_kms) 
	}



	/* All CRTCs are compatible with all encoders */

	for (i = 0; i < priv->num_encoders; i++)

		priv->encoders[i]->possible_crtcs = (1 << priv->num_crtcs) - 1;

	drm_for_each_encoder(encoder, dev)

		encoder->possible_crtcs = (1 << priv->num_crtcs) - 1;



	return 0;

}

drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c

+2 −4
Original line number Diff line number Diff line
@@ -437,6 +437,7 @@ static int modeset_init(struct mdp5_kms *mdp5_kms) 
	int i, ret, pi = 0, ci = 0;

	struct drm_plane *primary[MAX_BASES] = { NULL };

	struct drm_plane *cursor[MAX_BASES] = { NULL };

	struct drm_encoder *encoder;



	/*

	 * Construct encoders and modeset initialize connector devices
@@ -502,11 +503,8 @@ static int modeset_init(struct mdp5_kms *mdp5_kms) 
	 * Now that we know the number of crtcs we've created, set the possible

	 * crtcs for the encoders

	 */

	for (i = 0; i < priv->num_encoders; i++) {

		struct drm_encoder *encoder = priv->encoders[i];



	drm_for_each_encoder(encoder, dev)

		encoder->possible_crtcs = (1 << priv->num_crtcs) - 1;

	}



	return 0;