[Intel-gfx] [PATCH v2 09/12] drm/i915: Remove connectors_active from intel_dp.c.

Ander Conselvan De Oliveira conselvan2 at gmail.com
Thu Jul 30 02:16:55 PDT 2015


On Thu, 2015-07-30 at 08:54 +0200, Maarten Lankhorst wrote:
> Op 29-07-15 om 15:26 schreef Ander Conselvan De Oliveira:
> > On Mon, 2015-07-27 at 14:35 +0200, Maarten Lankhorst wrote:
> > > Now that everything's atomic, checking encoder->base.crtc is enough.
> > Don't you need to check encoder->base.crtc->state->active too?
> Not sure, I think stealing a encoder being bound to any crtc is probably good enough reason to 
> warn.
> We probably don't hold the right lock to dereference crtc->state.

Fair enough. Add that to the commit message and you can add

Reviewed-by: Ander Conselvan de Oliveira <conselvan2 at gmail.com>



More information about the Intel-gfx mailing list