Hey, Op 29-01-16 om 21:57 schreef Paulo Zanoni: > FBC is already deactivated at this point. > > Besides, nothing should be calling these lower-level function > pointers. A few months ago, the only caller of > dev_priv->fbc.deactivate was intel_pipe_set_base_atomic(), which was > the kgdboc function. But the following commit added it to the SKL > function: > > commit a8d201af68506b375b701d0d8dbe8487034256f2 > Author: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Date: Thu Jan 7 11:54:11 2016 +0100 > drm/i915: Use plane state for primary plane updates. > > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > Whole series looks good to me. Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx