On Tue, Apr 30, 2013 at 02:01:45PM +0200, Daniel Vetter wrote: > --- a/drivers/gpu/drm/i915/intel_drv.h > +++ b/drivers/gpu/drm/i915/intel_drv.h > @@ -223,6 +222,10 @@ struct intel_crtc_config { > /* Controls for the clock computation, to override various stages. */ > bool clock_set; > > + /* SDVO TV has a bunch of special case. To make multifunction encoders > + * work correctly, we need to track this at runtime.*/ > + bool sdvo_tv_clock; > + > /* > * crtc bandwidth limit, don't increase pipe bpp or clock if not really > * required. This is set in the 2nd loop of calling encoder's What is becoming less clear over time is what is derived state internal to the modesetting sequence, and what is the state used to select a mode e.g. to decide if two configs are compatible. -Chris -- Chris Wilson, Intel Open Source Technology Centre