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

Op 02-09-15 om 17:24 schreef Daniel Vetter:

On Wed, Sep 02, 2015 at 03:36:33PM +0100, Daniel Stone wrote:

...

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

Applied to drm-misc, thanks. -Daniel

Daniel Stone, can you review 2/2 too?

~Maarten