[Intel-gfx] [PATCH v8 1/5] drm/i915: Factor out setup_private_pat()

[Wang, Zhi A]

Thanks! Learned. :) 

-----Original Message-----
From: Joonas Lahtinen [mailto:joonas.lahtinen at linux.intel.com] 
Sent: Friday, September 8, 2017 2:54 PM
To: Wang, Zhi A <zhi.a.wang at intel.com>; intel-gfx at lists.freedesktop.org; intel-gvt-dev at lists.freedesktop.org
Cc: chris at chris-wilson.co.uk; zhenyuw at linux.intel.com; Widawsky, Benjamin <benjamin.widawsky at intel.com>; Vivi, Rodrigo <rodrigo.vivi at intel.com>
Subject: Re: [PATCH v8 1/5] drm/i915: Factor out setup_private_pat()

On Fri, 2017-09-08 at 17:05 +0800, Zhi Wang wrote:
> Factor out setup_private_pat() for introducing the following patches.
> 
> Cc: Ben Widawsky <benjamin.widawsky at intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> v7
> Signed-off-by: Zhi Wang <zhi.a.wang at intel.com>

Keep this in chronological order when you add things, so R-b after S-o- b. Only add the version after R-b *if* the review might have been invalidated due to some changes, in this case there are no changes and no changelog, so you can keep the R-b.

If there was substantial enough changes, changelog would be added and
#v7 added to previous reviews to indicate they do not apply to latest changes.

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