[Intel-gfx] [PATCH v2 1/2] drm/i915: Dynamically allocate s0ix struct for VLV

Chris Wilson chris at chris-wilson.co.uk
Tue Aug 20 11:14:03 UTC 2019


Quoting Daniele Ceraolo Spurio (2019-08-20 03:01:46)
> This is only required for a single platform so no need to reserve the
> memory on all of them.
> 
> This removes the last direct dependency of i915_drv.h on i915_reg.h
> (apart from the i915_reg_t definition).
> 
> v2: drop unneeded diff, keep the vlv prefix, call functions
>     unconditionally (Jani), fwd declaration of the struct (Chris)
> 
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Cc: Imre Deak <imre.deak at intel.com>
> Cc: Jani Nikula <jani.nikula at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>

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

Any chance we can move this to vlv_s0ix.c? Or perhaps
vlv_suspend.c, or vlv_pm.c?
-Chris


More information about the Intel-gfx mailing list