Re: [PATCH 12/13] drm/i915: Consolidate legacy semaphore initialization

Chris Wilson
2016-06-29T15:34:54Z

On Wed, Jun 29, 2016 at 04:09:31PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> Replace per-engine initialization with a common half-programatic,
> half-data driven code for ease of maintenance and compactness.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

This is the biggest pill to swallow (since our 5x5 table is only
sparsely populated), but it looks correct, and more importantly easier to
read.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

--
Chris Wilson, Intel Open Source Technology Centre