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@xxxxxxxxxxxxxxx>
> >
> > 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@xxxxxx>
> > Cc: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
> > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Acked-by: Jyri Sarha <jsarha@xxxxxx>
>
> 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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
