On Fri, Jul 05, 2013 at 11:57:41AM +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > Passing the appropriate crtc to intel_update_watermarks() should help > in avoiding needless work in the future. > > Also make the call intel_update_watermarks() just once during the > modeset. Ideally it should happen independently when each plane gets > enabled/disabled, but for now it seems better to keep it in one place. > We can improve things when we get all the planes sorted out in a better > way. Two patches in one here. I'd like to have the change in calling semantics from set_base as a separate patch. -Chris -- Chris Wilson, Intel Open Source Technology Centre