[Intel-gfx] [PATCH 2/2] drm/i915: switch vlv_suspend to use intel uncore register accessors

Jani Nikula jani.nikula at intel.com
Mon Feb 17 09:36:59 UTC 2020


On Thu, 13 Feb 2020, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> Quoting Jani Nikula (2020-02-12 14:40:58)
>> Prefer intel_uncore_* over I915_READ, I915_WRITE, and POSTING_READ.
>> 
>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>
> A couple of older checkpatch errors that could be cleaned up (pure
> whitespacing).
>
> Both
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

Thanks, pushed with one space added to the 2nd patch to silence the
checkpatch.

> Half this code should be removed as we explicitly reset the registers on
> resume. (And if we need to add a few special runtime-resume hooks, would
> not be a bad thing). And the other half, probably should be removed with
> a bit of extra work.

It's been rainy lately, but I guess I'll be waiting for a rainier day
yet to embark on that. ;)

BR,
Jani.

-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the Intel-gfx mailing list