[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-16 Thread Oded Gabbay
On Wed, Jun 15, 2016 at 1:08 PM, Daniel Vetter wrote: > It's not obvious at first sight that this is a fastpath, make that > clearer with a goto. Fallout from a discussion with Liviu on irc. > > Cc: Liviu.Dudau at arm.com > Acked-by: Liviu.Dudau at arm.com > Signed-off-by: Daniel Vetter > --- >

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-16 Thread Daniel Vetter
On Thu, Jun 16, 2016 at 5:30 PM, Oded Gabbay wrote: > On Wed, Jun 15, 2016 at 1:08 PM, Daniel Vetter > wrote: >> It's not obvious at first sight that this is a fastpath, make that >> clearer with a goto. Fallout from a discussion with Liviu on irc. >> >> Cc: Liviu.Dudau at arm.com >> Acked-by: L

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-15 Thread Daniel Vetter
On Wed, Jun 15, 2016 at 03:29:16PM +0100, Liviu.Dudau at arm.com wrote: > On Wed, Jun 15, 2016 at 12:24:26PM +0200, Daniel Vetter wrote: > > It's not obvious at first sight that this is a fastpath, make that > > clearer with a goto. Fallout from a discussion with Liviu on irc. > > > > v2: Drop bog

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-15 Thread liviu.du...@arm.com
On Wed, Jun 15, 2016 at 12:24:26PM +0200, Daniel Vetter wrote: > It's not obvious at first sight that this is a fastpath, make that > clearer with a goto. Fallout from a discussion with Liviu on irc. > > v2: Drop bogus hunks that crept in. > > Cc: Liviu.Dudau at arm.com > Acked-by: Liviu.Dudau at

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-15 Thread Daniel Vetter
It's not obvious at first sight that this is a fastpath, make that clearer with a goto. Fallout from a discussion with Liviu on irc. v2: Drop bogus hunks that crept in. Cc: Liviu.Dudau at arm.com Acked-by: Liviu.Dudau at arm.com Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_atomic_helper

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-15 Thread Daniel Vetter
It's not obvious at first sight that this is a fastpath, make that clearer with a goto. Fallout from a discussion with Liviu on irc. Cc: Liviu.Dudau at arm.com Acked-by: Liviu.Dudau at arm.com Signed-off-by: Daniel Vetter --- .../gpu/drm/amd/amdkfd/kfd_device_queue_manager.h | 4 ++-- drivers/

[PATCH] drm/atomic-helpers: Clear up cleanup_done a bit

2016-06-15 Thread liviu.du...@arm.com
On Wed, Jun 15, 2016 at 12:08:29PM +0200, Daniel Vetter wrote: > It's not obvious at first sight that this is a fastpath, make that > clearer with a goto. Fallout from a discussion with Liviu on irc. > > Cc: Liviu.Dudau at arm.com > Acked-by: Liviu.Dudau at arm.com Nope, I did not agree to *all*