[Intel-gfx] [REBASED PATCH RESEND 5/5 v3] drm/i915: debugfs spring cleaning

Chris Wilson chris at chris-wilson.co.uk
Mon Aug 22 11:04:20 UTC 2016


On Mon, Aug 22, 2016 at 01:59:31PM +0300, David Weinehall wrote:
> Just like with sysfs, we do some major overhaul.
> 
> Pass dev_priv instead of dev to all feature macros (IS_, HAS_,
> INTEL_, etc.). This has the side effect that a bunch of functions
> now get dev_priv passed instead of dev.
> 
> All calls to INTEL_INFO()->gen have been replaced with
> INTEL_GEN().
> 
> We want access to to_i915(node->minor->dev) in a lot of places,
> so add the node_to_i915() helper to accomodate for this.
> 
> Finally, we have quite a few cases where we get a void * pointer,
> and need to cast it to drm_device *, only to run to_i915() on it.
> Add cast_to_i915() to do this.
> 
> v2: Don't introduce extra dev (Chris)
> 
> v3: Make pipe_crc_info have a pointer to drm_i915_private instead of
>     drm_device. This saves a bit of space, since we never use
>     drm_device anywhere in these functions.
> 
>     Also some minor fixup that I missed in the previous version.
> 
> v4: Changed the code a bit so that dev_priv is passed directly
>     to various functions, thus removing the need for the
>     cast_to_i915() helper. Also did some additional cleanup.
> 
> v5: Additional cleanup of newly introduced changes.
> 
> v6: Rebase again because of conflict.

I improvised your missing s-o-b.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list