[Intel-gfx] [PATCH 1/3] drm/i915: don't deactivate FBC at skylake_disable_primary_plane

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Tue Feb 2 09:27:21 UTC 2016


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 at linux.intel.com>
>     Date:   Thu Jan 7 11:54:11 2016 +0100
>         drm/i915: Use plane state for primary plane updates.
>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
>
Whole series looks good to me.

Reviewed-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>



More information about the Intel-gfx mailing list