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

[Ville Syrjälä]

On Thu, Jun 28, 2018 at 04:45:50PM +0300, Jyri Sarha wrote:
> 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.

That's the plan. Thanks for the ack.

> 
> > ---
> >  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

-- 
Ville Syrjälä
Intel