[Intel-gfx] [PATCH 2/8] drm/i915: skip redundant operations whilst enabling pipes and planes

2011-03-18 Thread Chris Wilson
If the pipe or plane is already enabled, then we do not need to enable it again and can skip the delay. References: https://bugs.freedesktop.org/show_bug.cgi?id=34601 Cc: Jesse Barnes jbar...@virtuousgeek.org Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk ---

Re: [Intel-gfx] [PATCH 2/8] drm/i915: skip redundant operations whilst enabling pipes and planes

2011-03-18 Thread Keith Packard
On Fri, 18 Mar 2011 08:02:04 +, Chris Wilson ch...@chris-wilson.co.uk wrote: If the pipe or plane is already enabled, then we do not need to enable it again and can skip the delay. This should mention that you're also skipping the operation on disable. Otherwise, Reviewed-by: Keith

Re: [Intel-gfx] [PATCH 2/8] drm/i915: skip redundant operations whilst enabling pipes and planes

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 08:02:04 + Chris Wilson ch...@chris-wilson.co.uk wrote: If the pipe or plane is already enabled, then we do not need to enable it again and can skip the delay. References: https://bugs.freedesktop.org/show_bug.cgi?id=34601 Cc: Jesse Barnes jbar...@virtuousgeek.org

Re: [Intel-gfx] [PATCH 2/8] drm/i915: skip redundant operations whilst enabling pipes and planes

2011-03-18 Thread Chris Wilson
On Fri, 18 Mar 2011 09:25:22 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: On Fri, 18 Mar 2011 08:02:04 + Chris Wilson ch...@chris-wilson.co.uk wrote: If the pipe or plane is already enabled, then we do not need to enable it again and can skip the delay. References:

Re: [Intel-gfx] [PATCH 2/8] drm/i915: skip redundant operations whilst enabling pipes and planes

2011-03-18 Thread Jesse Barnes
On Fri, 18 Mar 2011 17:37:06 + Chris Wilson ch...@chris-wilson.co.uk wrote: On Fri, 18 Mar 2011 09:25:22 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: On Fri, 18 Mar 2011 08:02:04 + Chris Wilson ch...@chris-wilson.co.uk wrote: If the pipe or plane is already enabled,