Re: [PATCH v2 4/9] drm/i915: Add reverse mapping between port and intel_encoder

Takashi Iwai
2015-12-04T15:09:58Z

On Fri, 04 Dec 2015 16:02:29 +0100,
Daniel Vetter wrote:
>
> On Fri, Dec 04, 2015 at 03:40:03PM +0100, Takashi Iwai wrote:
> > On Tue, 01 Dec 2015 17:09:53 +0100,
> > Takashi Iwai wrote:
> > >
> > > This patch adds a reverse mapping from a digital port number to
> > > intel_encoder object containing the corresponding intel_digital_port.
> > > It simplifies the query of the encoder a lot.
> >
> > While this is good for a code reduction, I guess it's better to leave
> > away for now, as there will be more changes there for MST support.
> > It may put yet another loop, and the mapping implemented here might
> > not be the best way. So I'm going to drop this from the next
> > patchset.
> >
> > If anyone still thinks this is worth to include, let me know. I'll
> > re-add this.
>
> Yeah I think this looks good. Fixing up conflicts with MST shouldn't be
> that much trouble really.

Alright, resurrected from ash.

Takashi