[Intel-gfx] [PATCH v2 9/9] drm/tilcdc: Use drm_connector_has_possible_encoder()

Jyri Sarha jsarha at ti.com
Thu Jun 28 13:45:50 UTC 2018


On 28/06/18 16:13, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Use drm_connector_has_possible_encoder() for checking
> whether the encoder has an associated connector.
> 
> v2: Replace the drm_for_each_connector_encoder_ids() loop
>     with a simple drm_connector_has_possible_encoder() call
> 
> Cc: Jyri Sarha <jsarha at ti.com>
> Cc: Tomi Valkeinen <tomi.valkeinen at ti.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Acked-by: Jyri Sarha <jsarha at ti.com>

I guess you are going to merge these at one go trough drm-misc.

> ---
>  drivers/gpu/drm/tilcdc/tilcdc_external.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/tilcdc/tilcdc_external.c b/drivers/gpu/drm/tilcdc/tilcdc_external.c
> index d651bdd6597e..b4eaf9bc87f8 100644
> --- a/drivers/gpu/drm/tilcdc/tilcdc_external.c
> +++ b/drivers/gpu/drm/tilcdc/tilcdc_external.c
> @@ -103,12 +103,11 @@ struct drm_connector *tilcdc_encoder_find_connector(struct drm_device *ddev,
>  						    struct drm_encoder *encoder)
>  {
>  	struct drm_connector *connector;
> -	int i;
>  
> -	list_for_each_entry(connector, &ddev->mode_config.connector_list, head)
> -		for (i = 0; i < DRM_CONNECTOR_MAX_ENCODER; i++)
> -			if (connector->encoder_ids[i] == encoder->base.id)
> -				return connector;
> +	list_for_each_entry(connector, &ddev->mode_config.connector_list, head) {
> +		if (drm_connector_has_possible_encoder(connector, encoder))
> +			return connector;
> +	}
>  
>  	dev_err(ddev->dev, "No connector found for %s encoder (id %d)\n",
>  		encoder->name, encoder->base.id);
> 


-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki


More information about the Intel-gfx mailing list