Re: [PATCH 1/2] drm/atomic: Make prepare_fb/cleanup_fb only take state, v3.

On 2 September 2015 at 09:42, Maarten Lankhorst maarten.lankhorst@linux.intel.com wrote:

This removes the need to separately track fb changes i915. That will be done as a separate commit, however.

Changes since v1:

• Add dri-devel to cc.
• Fix a check in intel's prepare and cleanup fb to take rotation into account.

Changes since v2:

• Split out i915 changes to a separate commit.

Cc: dri-devel@lists.freedesktop.org Signed-off-by: Maarten Lankhorst maarten.lankhorst@linux.intel.com

I'd probably prefer to see the change to call them unconditionally (regardless of fb != NULL) in a separate patch, but with or without: Reviewed-by: Daniel Stone daniels@collabora.com

Cheers, Daniel