[PATCH v2] drm/connector: Allow max possible encoders to attach to a connector

Ville Syrjälä ville.syrjala at linux.intel.com
Fri Sep 6 11:27:51 UTC 2019


On Thu, Sep 05, 2019 at 02:09:27PM -0700, José Roberto de Souza wrote:
> From: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> 
> Currently we restrict the number of encoders that can be linked to
> a connector to 3, increase it to match the maximum number of encoders
> that can be initialized(32).
> 
> To more effiently do that lets switch from an array of encoder ids to
> bitmask.
> 
> Also removing the best_encoder hook from the drivers that only have
> one encoder per connector(this ones have one encoder in the whole
> driver), pick_single_encoder_for_connector() will do the same job
> with no functional change.

I don't think non-atomic drivers have that fallback in place.
They probable should...

Apart from that lgtm
Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

> 
> v2: Fixing missed return on amdgpu_dm_connector_to_encoder()
> 
> Suggested-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: dri-devel at lists.freedesktop.org
> Cc: intel-gfx at lists.freedesktop.org
> Cc: nouveau at lists.freedesktop.org
> Cc: amd-gfx at lists.freedesktop.org
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
> ---
>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 23 +++++---------
>  drivers/gpu/drm/amd/amdgpu/dce_virtual.c      |  5 ++-
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  6 +++-
>  drivers/gpu/drm/ast/ast_mode.c                | 12 -------
>  drivers/gpu/drm/drm_atomic_helper.c           |  9 ++++--
>  drivers/gpu/drm/drm_client_modeset.c          |  3 +-
>  drivers/gpu/drm/drm_connector.c               | 31 +++++--------------
>  drivers/gpu/drm/drm_probe_helper.c            |  3 +-
>  drivers/gpu/drm/mgag200/mgag200_mode.c        | 11 -------
>  drivers/gpu/drm/nouveau/dispnv04/disp.c       |  2 +-
>  drivers/gpu/drm/nouveau/dispnv50/disp.c       |  2 +-
>  drivers/gpu/drm/nouveau/nouveau_connector.c   |  7 ++---
>  drivers/gpu/drm/radeon/radeon_connectors.c    | 27 ++++++----------
>  drivers/gpu/drm/udl/udl_connector.c           |  8 -----
>  include/drm/drm_connector.h                   | 18 +++++------
>  15 files changed, 53 insertions(+), 114 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> index ece55c8fa673..d8729285f731 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> @@ -217,11 +217,10 @@ amdgpu_connector_update_scratch_regs(struct drm_connector *connector,
>  	struct drm_encoder *encoder;
>  	const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
>  	bool connected;
> -	int i;
>  
>  	best_encoder = connector_funcs->best_encoder(connector);
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if ((encoder == best_encoder) && (status == connector_status_connected))
>  			connected = true;
>  		else
> @@ -236,9 +235,8 @@ amdgpu_connector_find_encoder(struct drm_connector *connector,
>  			       int encoder_type)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (encoder->encoder_type == encoder_type)
>  			return encoder;
>  	}
> @@ -347,10 +345,9 @@ static struct drm_encoder *
>  amdgpu_connector_best_single_encoder(struct drm_connector *connector)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
>  	/* pick the first one */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> +	drm_connector_for_each_possible_encoder(connector, encoder)
>  		return encoder;
>  
>  	return NULL;
> @@ -1065,9 +1062,8 @@ amdgpu_connector_dvi_detect(struct drm_connector *connector, bool force)
>  	/* find analog encoder */
>  	if (amdgpu_connector->dac_load_detect) {
>  		struct drm_encoder *encoder;
> -		int i;
>  
> -		drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +		drm_connector_for_each_possible_encoder(connector, encoder) {
>  			if (encoder->encoder_type != DRM_MODE_ENCODER_DAC &&
>  			    encoder->encoder_type != DRM_MODE_ENCODER_TVDAC)
>  				continue;
> @@ -1117,9 +1113,8 @@ amdgpu_connector_dvi_encoder(struct drm_connector *connector)
>  {
>  	struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector);
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (amdgpu_connector->use_digital == true) {
>  			if (encoder->encoder_type == DRM_MODE_ENCODER_TMDS)
>  				return encoder;
> @@ -1134,7 +1129,7 @@ amdgpu_connector_dvi_encoder(struct drm_connector *connector)
>  
>  	/* then check use digitial */
>  	/* pick the first one */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> +	drm_connector_for_each_possible_encoder(connector, encoder)
>  		return encoder;
>  
>  	return NULL;
> @@ -1271,9 +1266,8 @@ u16 amdgpu_connector_encoder_get_dp_bridge_encoder_id(struct drm_connector *conn
>  {
>  	struct drm_encoder *encoder;
>  	struct amdgpu_encoder *amdgpu_encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		amdgpu_encoder = to_amdgpu_encoder(encoder);
>  
>  		switch (amdgpu_encoder->encoder_id) {
> @@ -1292,10 +1286,9 @@ static bool amdgpu_connector_encoder_is_hbr2(struct drm_connector *connector)
>  {
>  	struct drm_encoder *encoder;
>  	struct amdgpu_encoder *amdgpu_encoder;
> -	int i;
>  	bool found = false;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		amdgpu_encoder = to_amdgpu_encoder(encoder);
>  		if (amdgpu_encoder->caps & ATOM_ENCODER_CAP_RECORD_HBR2)
>  			found = true;
> diff --git a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> index f1fadb5ba191..99d12e88de92 100644
> --- a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> +++ b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> @@ -260,15 +260,14 @@ static struct drm_encoder *
>  dce_virtual_encoder(struct drm_connector *connector)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (encoder->encoder_type == DRM_MODE_ENCODER_VIRTUAL)
>  			return encoder;
>  	}
>  
>  	/* pick the first one */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> +	drm_connector_for_each_possible_encoder(connector, encoder)
>  		return encoder;
>  
>  	return NULL;
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 0a71ed1e7762..d6e3925ca633 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -4822,7 +4822,11 @@ static int to_drm_connector_type(enum signal_type st)
>  
>  static struct drm_encoder *amdgpu_dm_connector_to_encoder(struct drm_connector *connector)
>  {
> -	return drm_encoder_find(connector->dev, NULL, connector->encoder_ids[0]);
> +	struct drm_encoder *encoder;
> +
> +	/* There is only one encoder per connector */
> +	drm_connector_for_each_possible_encoder(connector, encoder)
> +		return encoder;
>  }
>  
>  static void amdgpu_dm_get_native_mode(struct drm_connector *connector)
> diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
> index d349c721501c..eef95e1af06b 100644
> --- a/drivers/gpu/drm/ast/ast_mode.c
> +++ b/drivers/gpu/drm/ast/ast_mode.c
> @@ -687,17 +687,6 @@ static void ast_encoder_destroy(struct drm_encoder *encoder)
>  	kfree(encoder);
>  }
>  
> -
> -static struct drm_encoder *ast_best_single_encoder(struct drm_connector *connector)
> -{
> -	int enc_id = connector->encoder_ids[0];
> -	/* pick the encoder ids */
> -	if (enc_id)
> -		return drm_encoder_find(connector->dev, NULL, enc_id);
> -	return NULL;
> -}
> -
> -
>  static const struct drm_encoder_funcs ast_enc_funcs = {
>  	.destroy = ast_encoder_destroy,
>  };
> @@ -847,7 +836,6 @@ static void ast_connector_destroy(struct drm_connector *connector)
>  static const struct drm_connector_helper_funcs ast_connector_helper_funcs = {
>  	.mode_valid = ast_mode_valid,
>  	.get_modes = ast_get_modes,
> -	.best_encoder = ast_best_single_encoder,
>  };
>  
>  static const struct drm_connector_funcs ast_connector_funcs = {
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 4706439fb490..c440027cb537 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -104,8 +104,13 @@ drm_atomic_helper_plane_changed(struct drm_atomic_state *state,
>  static struct drm_encoder *
>  pick_single_encoder_for_connector(struct drm_connector *connector)
>  {
> -	WARN_ON(connector->encoder_ids[1]);
> -	return drm_encoder_find(connector->dev, NULL, connector->encoder_ids[0]);
> +	struct drm_encoder *encoder;
> +
> +	WARN_ON(hweight32(connector->possible_encoders) > 1);
> +	drm_connector_for_each_possible_encoder(connector, encoder)
> +		return encoder;
> +
> +	return NULL;
>  }
>  
>  static int handle_conflicting_encoders(struct drm_atomic_state *state,
> diff --git a/drivers/gpu/drm/drm_client_modeset.c b/drivers/gpu/drm/drm_client_modeset.c
> index c8922b7cac09..895b73f23079 100644
> --- a/drivers/gpu/drm/drm_client_modeset.c
> +++ b/drivers/gpu/drm/drm_client_modeset.c
> @@ -415,9 +415,8 @@ static bool connector_has_possible_crtc(struct drm_connector *connector,
>  					struct drm_crtc *crtc)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (encoder->possible_crtcs & drm_crtc_mask(crtc))
>  			return true;
>  	}
> diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
> index 4c766624b20d..43896c711b50 100644
> --- a/drivers/gpu/drm/drm_connector.c
> +++ b/drivers/gpu/drm/drm_connector.c
> @@ -365,8 +365,6 @@ EXPORT_SYMBOL(drm_connector_attach_edid_property);
>  int drm_connector_attach_encoder(struct drm_connector *connector,
>  				 struct drm_encoder *encoder)
>  {
> -	int i;
> -
>  	/*
>  	 * In the past, drivers have attempted to model the static association
>  	 * of connector to encoder in simple connector/encoder devices using a
> @@ -381,18 +379,15 @@ int drm_connector_attach_encoder(struct drm_connector *connector,
>  	if (WARN_ON(connector->encoder))
>  		return -EINVAL;
>  
> -	for (i = 0; i < ARRAY_SIZE(connector->encoder_ids); i++) {
> -		if (connector->encoder_ids[i] == 0) {
> -			connector->encoder_ids[i] = encoder->base.id;
> -			return 0;
> -		}
> -	}
> -	return -ENOMEM;
> +	connector->possible_encoders |= drm_encoder_mask(encoder);
> +
> +	return 0;
>  }
>  EXPORT_SYMBOL(drm_connector_attach_encoder);
>  
>  /**
> - * drm_connector_has_possible_encoder - check if the connector and encoder are assosicated with each other
> + * drm_connector_has_possible_encoder - check if the connector and encoder are
> + * associated with each other
>   * @connector: the connector
>   * @encoder: the encoder
>   *
> @@ -402,15 +397,7 @@ EXPORT_SYMBOL(drm_connector_attach_encoder);
>  bool drm_connector_has_possible_encoder(struct drm_connector *connector,
>  					struct drm_encoder *encoder)
>  {
> -	struct drm_encoder *enc;
> -	int i;
> -
> -	drm_connector_for_each_possible_encoder(connector, enc, i) {
> -		if (enc == encoder)
> -			return true;
> -	}
> -
> -	return false;
> +	return connector->possible_encoders & drm_encoder_mask(encoder);
>  }
>  EXPORT_SYMBOL(drm_connector_has_possible_encoder);
>  
> @@ -2121,7 +2108,6 @@ int drm_mode_getconnector(struct drm_device *dev, void *data,
>  	int encoders_count = 0;
>  	int ret = 0;
>  	int copied = 0;
> -	int i;
>  	struct drm_mode_modeinfo u_mode;
>  	struct drm_mode_modeinfo __user *mode_ptr;
>  	uint32_t __user *encoder_ptr;
> @@ -2136,14 +2122,13 @@ int drm_mode_getconnector(struct drm_device *dev, void *data,
>  	if (!connector)
>  		return -ENOENT;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> -		encoders_count++;
> +	encoders_count = hweight32(connector->possible_encoders);
>  
>  	if ((out_resp->count_encoders >= encoders_count) && encoders_count) {
>  		copied = 0;
>  		encoder_ptr = (uint32_t __user *)(unsigned long)(out_resp->encoders_ptr);
>  
> -		drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +		drm_connector_for_each_possible_encoder(connector, encoder) {
>  			if (put_user(encoder->base.id, encoder_ptr + copied)) {
>  				ret = -EFAULT;
>  				goto out;
> diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c
> index 351cbc40f0f8..a7c87abe88d0 100644
> --- a/drivers/gpu/drm/drm_probe_helper.c
> +++ b/drivers/gpu/drm/drm_probe_helper.c
> @@ -93,7 +93,6 @@ drm_mode_validate_pipeline(struct drm_display_mode *mode,
>  	struct drm_device *dev = connector->dev;
>  	enum drm_mode_status ret = MODE_OK;
>  	struct drm_encoder *encoder;
> -	int i;
>  
>  	/* Step 1: Validate against connector */
>  	ret = drm_connector_mode_valid(connector, mode);
> @@ -101,7 +100,7 @@ drm_mode_validate_pipeline(struct drm_display_mode *mode,
>  		return ret;
>  
>  	/* Step 2: Validate against encoders and crtcs */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		struct drm_crtc *crtc;
>  
>  		ret = drm_encoder_mode_valid(encoder, mode);
> diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c
> index 5e778b5f1a10..68226556044b 100644
> --- a/drivers/gpu/drm/mgag200/mgag200_mode.c
> +++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
> @@ -1638,16 +1638,6 @@ static enum drm_mode_status mga_vga_mode_valid(struct drm_connector *connector,
>  	return MODE_OK;
>  }
>  
> -static struct drm_encoder *mga_connector_best_encoder(struct drm_connector
> -						  *connector)
> -{
> -	int enc_id = connector->encoder_ids[0];
> -	/* pick the encoder ids */
> -	if (enc_id)
> -		return drm_encoder_find(connector->dev, NULL, enc_id);
> -	return NULL;
> -}
> -
>  static void mga_connector_destroy(struct drm_connector *connector)
>  {
>  	struct mga_connector *mga_connector = to_mga_connector(connector);
> @@ -1659,7 +1649,6 @@ static void mga_connector_destroy(struct drm_connector *connector)
>  static const struct drm_connector_helper_funcs mga_vga_connector_helper_funcs = {
>  	.get_modes = mga_vga_get_modes,
>  	.mode_valid = mga_vga_mode_valid,
> -	.best_encoder = mga_connector_best_encoder,
>  };
>  
>  static const struct drm_connector_funcs mga_vga_connector_funcs = {
> diff --git a/drivers/gpu/drm/nouveau/dispnv04/disp.c b/drivers/gpu/drm/nouveau/dispnv04/disp.c
> index dc64863b5fd8..44ee82d0c9b6 100644
> --- a/drivers/gpu/drm/nouveau/dispnv04/disp.c
> +++ b/drivers/gpu/drm/nouveau/dispnv04/disp.c
> @@ -256,7 +256,7 @@ nv04_display_create(struct drm_device *dev)
>  
>  	list_for_each_entry_safe(connector, ct,
>  				 &dev->mode_config.connector_list, head) {
> -		if (!connector->encoder_ids[0]) {
> +		if (!connector->possible_encoders) {
>  			NV_WARN(drm, "%s has no encoders, removing\n",
>  				connector->name);
>  			connector->funcs->destroy(connector);
> diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> index 307584107d77..98c87195d711 100644
> --- a/drivers/gpu/drm/nouveau/dispnv50/disp.c
> +++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> @@ -2391,7 +2391,7 @@ nv50_display_create(struct drm_device *dev)
>  
>  	/* cull any connectors we created that don't have an encoder */
>  	list_for_each_entry_safe(connector, tmp, &dev->mode_config.connector_list, head) {
> -		if (connector->encoder_ids[0])
> +		if (connector->possible_encoders)
>  			continue;
>  
>  		NV_WARN(drm, "%s has no encoders, removing\n",
> diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c b/drivers/gpu/drm/nouveau/nouveau_connector.c
> index 56871d34e3fb..3a5db17bc5c7 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_connector.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
> @@ -365,9 +365,8 @@ find_encoder(struct drm_connector *connector, int type)
>  {
>  	struct nouveau_encoder *nv_encoder;
>  	struct drm_encoder *enc;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, enc, i) {
> +	drm_connector_for_each_possible_encoder(connector, enc) {
>  		nv_encoder = nouveau_encoder(enc);
>  
>  		if (type == DCB_OUTPUT_ANY ||
> @@ -414,10 +413,10 @@ nouveau_connector_ddc_detect(struct drm_connector *connector)
>  	struct drm_device *dev = connector->dev;
>  	struct nouveau_encoder *nv_encoder = NULL, *found = NULL;
>  	struct drm_encoder *encoder;
> -	int i, ret;
> +	int ret;
>  	bool switcheroo_ddc = false;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		nv_encoder = nouveau_encoder(encoder);
>  
>  		switch (nv_encoder->dcb->type) {
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index b684cd719612..c07427d3c199 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -249,11 +249,10 @@ radeon_connector_update_scratch_regs(struct drm_connector *connector, enum drm_c
>  	struct drm_encoder *encoder;
>  	const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
>  	bool connected;
> -	int i;
>  
>  	best_encoder = connector_funcs->best_encoder(connector);
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if ((encoder == best_encoder) && (status == connector_status_connected))
>  			connected = true;
>  		else
> @@ -269,9 +268,8 @@ radeon_connector_update_scratch_regs(struct drm_connector *connector, enum drm_c
>  static struct drm_encoder *radeon_find_encoder(struct drm_connector *connector, int encoder_type)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (encoder->encoder_type == encoder_type)
>  			return encoder;
>  	}
> @@ -380,10 +378,9 @@ static int radeon_ddc_get_modes(struct drm_connector *connector)
>  static struct drm_encoder *radeon_best_single_encoder(struct drm_connector *connector)
>  {
>  	struct drm_encoder *encoder;
> -	int i;
>  
>  	/* pick the first one */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> +	drm_connector_for_each_possible_encoder(connector, encoder)
>  		return encoder;
>  
>  	return NULL;
> @@ -428,14 +425,13 @@ radeon_connector_analog_encoder_conflict_solve(struct drm_connector *connector,
>  
>  	list_for_each_entry(conflict, &dev->mode_config.connector_list, head) {
>  		struct drm_encoder *enc;
> -		int i;
>  
>  		if (conflict == connector)
>  			continue;
>  
>  		radeon_conflict = to_radeon_connector(conflict);
>  
> -		drm_connector_for_each_possible_encoder(conflict, enc, i) {
> +		drm_connector_for_each_possible_encoder(conflict, enc) {
>  			/* if the IDs match */
>  			if (enc == encoder) {
>  				if (conflict->status != connector_status_connected)
> @@ -1363,9 +1359,7 @@ radeon_dvi_detect(struct drm_connector *connector, bool force)
>  
>  	/* find analog encoder */
>  	if (radeon_connector->dac_load_detect) {
> -		int i;
> -
> -		drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +		drm_connector_for_each_possible_encoder(connector, encoder) {
>  			if (encoder->encoder_type != DRM_MODE_ENCODER_DAC &&
>  			    encoder->encoder_type != DRM_MODE_ENCODER_TVDAC)
>  				continue;
> @@ -1443,9 +1437,8 @@ static struct drm_encoder *radeon_dvi_encoder(struct drm_connector *connector)
>  {
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
>  	struct drm_encoder *encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		if (radeon_connector->use_digital == true) {
>  			if (encoder->encoder_type == DRM_MODE_ENCODER_TMDS)
>  				return encoder;
> @@ -1460,7 +1453,7 @@ static struct drm_encoder *radeon_dvi_encoder(struct drm_connector *connector)
>  
>  	/* then check use digitial */
>  	/* pick the first one */
> -	drm_connector_for_each_possible_encoder(connector, encoder, i)
> +	drm_connector_for_each_possible_encoder(connector, encoder)
>  		return encoder;
>  
>  	return NULL;
> @@ -1603,9 +1596,8 @@ u16 radeon_connector_encoder_get_dp_bridge_encoder_id(struct drm_connector *conn
>  {
>  	struct drm_encoder *encoder;
>  	struct radeon_encoder *radeon_encoder;
> -	int i;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		radeon_encoder = to_radeon_encoder(encoder);
>  
>  		switch (radeon_encoder->encoder_id) {
> @@ -1624,10 +1616,9 @@ static bool radeon_connector_encoder_is_hbr2(struct drm_connector *connector)
>  {
>  	struct drm_encoder *encoder;
>  	struct radeon_encoder *radeon_encoder;
> -	int i;
>  	bool found = false;
>  
> -	drm_connector_for_each_possible_encoder(connector, encoder, i) {
> +	drm_connector_for_each_possible_encoder(connector, encoder) {
>  		radeon_encoder = to_radeon_encoder(encoder);
>  		if (radeon_encoder->caps & ATOM_ENCODER_CAP_RECORD_HBR2)
>  			found = true;
> diff --git a/drivers/gpu/drm/udl/udl_connector.c b/drivers/gpu/drm/udl/udl_connector.c
> index ddb61a60c610..b4ae3e89a7b4 100644
> --- a/drivers/gpu/drm/udl/udl_connector.c
> +++ b/drivers/gpu/drm/udl/udl_connector.c
> @@ -90,13 +90,6 @@ udl_detect(struct drm_connector *connector, bool force)
>  	return connector_status_connected;
>  }
>  
> -static struct drm_encoder*
> -udl_best_single_encoder(struct drm_connector *connector)
> -{
> -	int enc_id = connector->encoder_ids[0];
> -	return drm_encoder_find(connector->dev, NULL, enc_id);
> -}
> -
>  static int udl_connector_set_property(struct drm_connector *connector,
>  				      struct drm_property *property,
>  				      uint64_t val)
> @@ -120,7 +113,6 @@ static void udl_connector_destroy(struct drm_connector *connector)
>  static const struct drm_connector_helper_funcs udl_connector_helper_funcs = {
>  	.get_modes = udl_get_modes,
>  	.mode_valid = udl_mode_valid,
> -	.best_encoder = udl_best_single_encoder,
>  };
>  
>  static const struct drm_connector_funcs udl_connector_funcs = {
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index 681cb590f952..c6e993e78dbd 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -1288,12 +1288,12 @@ struct drm_connector {
>  	/** @override_edid: has the EDID been overwritten through debugfs for testing? */
>  	bool override_edid;
>  
> -#define DRM_CONNECTOR_MAX_ENCODER 3
>  	/**
> -	 * @encoder_ids: Valid encoders for this connector. Please only use
> -	 * drm_connector_for_each_possible_encoder() to enumerate these.
> +	 * @possible_encoders: Bit mask of encoders that can drive this
> +	 * connector, drm_encoder_index() determines the index into the bitfield
> +	 * and the bits are set with drm_connector_attach_encoder().
>  	 */
> -	uint32_t encoder_ids[DRM_CONNECTOR_MAX_ENCODER];
> +	u32 possible_encoders;
>  
>  	/**
>  	 * @encoder: Currently bound encoder driving this connector, if any.
> @@ -1608,13 +1608,9 @@ bool drm_connector_has_possible_encoder(struct drm_connector *connector,
>   * drm_connector_for_each_possible_encoder - iterate connector's possible encoders
>   * @connector: &struct drm_connector pointer
>   * @encoder: &struct drm_encoder pointer used as cursor
> - * @__i: int iteration cursor, for macro-internal use
>   */
> -#define drm_connector_for_each_possible_encoder(connector, encoder, __i) \
> -	for ((__i) = 0; (__i) < ARRAY_SIZE((connector)->encoder_ids) && \
> -		     (connector)->encoder_ids[(__i)] != 0; (__i)++) \
> -		for_each_if((encoder) = \
> -			    drm_encoder_find((connector)->dev, NULL, \
> -					     (connector)->encoder_ids[(__i)])) \
> +#define drm_connector_for_each_possible_encoder(connector, encoder) \
> +	drm_for_each_encoder_mask(encoder, (connector)->dev, \
> +				  (connector)->possible_encoders)
>  
>  #endif
> -- 
> 2.23.0

-- 
Ville Syrjälä
Intel


More information about the amd-gfx mailing list