[Intel-gfx] [PATCH] drm/i915: Don't hardcode the number of pipes in the error state dump

Chris Wilson chris at chris-wilson.co.uk
Wed Aug 15 20:42:15 CEST 2012


On Wed, 15 Aug 2012 19:23:25 +0100, Damien Lespiau <damien.lespiau at gmail.com> wrote:
> From: Damien Lespiau <damien.lespiau at intel.com>
> 
> New-ish devices have 3 pipes, so let's not just hardcode 2 but use the
> for_each_pipe() macro and make struct intel_display_error_state is big
> enough.
> 
> V2: Also add the number of pipes emitted (Chris Wilson)
> 
> Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

I guess next on your list will be adding some of the PCH register
states, along with a list of active encoders / connectors etc.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list