[Intel-gfx] [PATCH 06/13] drm/i915: Retrieve the current mode upon KMS takeover

Jesse Barnes jbarnes at virtuousgeek.org
Wed Mar 27 00:24:48 CET 2013


On Wed, 20 Mar 2013 14:36:25 +0200
Imre Deak <imre.deak at intel.com> wrote:

> > +		pipe = -1;
> >  
> >  		if (encoder->get_hw_state(encoder, &pipe)) {
> > -			encoder->base.crtc =
> > -				dev_priv->pipe_to_crtc_mapping[pipe];
> > -		} else {
> > +			crtc = to_intel_crtc(dev_priv->pipe_to_crtc_mapping[pipe]);  
> 
> get_hw_state() can return true and still leave pipe at -1, causing an
> invalid access here.

Fixed, thanks.

-- 
Jesse Barnes, Intel Open Source Technology Center



More information about the Intel-gfx mailing list