[PATCH 3/3] drm/atomic-helper: Don't call atomic_update_plane when it stays off

2015-04-16 Thread Daniel Vetter
On Thu, Apr 16, 2015 at 12:46:53AM +0300, Laurent Pinchart wrote: > Hi Daniel, > > Thank you for the patch. > > On Friday 10 April 2015 16:22:39 Daniel Vetter wrote: > > It's a silly thing to do and surprises driver writers. Most likely > > this did already blow up for exynos. > > > > It's also

[PATCH 3/3] drm/atomic-helper: Don't call atomic_update_plane when it stays off

2015-04-16 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Friday 10 April 2015 16:22:39 Daniel Vetter wrote: > It's a silly thing to do and surprises driver writers. Most likely > this did already blow up for exynos. > > It's also a silly thing to change plane state when it's off, but fbdev > is silly (it does an

[PATCH 3/3] drm/atomic-helper: Don't call atomic_update_plane when it stays off

2015-04-10 Thread Daniel Vetter
It's a silly thing to do and surprises driver writers. Most likely this did already blow up for exynos. It's also a silly thing to change plane state when it's off, but fbdev is silly (it does an unconditional modeset over all planes). And userspace can be evil. So I think we need this. With