Re: [PATCH 01/18] drm/i915: Unify intel_logical_ring_emit and intel_ring_emit

Joonas Lahtinen
2016-07-21T11:29:16Z

On ke, 2016-07-20 at 14:11 +0100, Chris Wilson wrote:
> Both perform the same actions with more or less indirection, so just
> unify the code.
>

Don't really like removing the engine = req->engine aliases, but seems
like req->engine is used plenty already. And assuming this was a
mechanical change with no hidden functional changes. Then,

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>

Regards, Joonas
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporation