Re: [PATCH 2/2] drm/i915: give gen8_setup_page_directory_pointer or more accurate name

Chris Wilson
2016-12-02T10:48:22Z

On Fri, Dec 02, 2016 at 10:30:21AM +0000, Matthew Auld wrote:
> The function name gen8_setup_page_directory_pointer is misleading, and only
> serves to confuse the reader, it's not setting up a pdp, but
> rather encoding a specific pml4e with a given pdp.
>
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>

Both need s/or/a/ in subject
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

--
Chris Wilson, Intel Open Source Technology Centre