[Intel-gfx] [PATCH 00/10] GEM object create and driver init dev_priv cleanups

Chris Wilson chris at chris-wilson.co.uk
Wed Nov 30 12:05:59 UTC 2016


On Wed, Nov 30, 2016 at 11:35:30AM +0000, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> 
> Autumn of churn continues. :)
> 
> This series tidies GEM object construction to take dev_priv instead of dev
> in all cases and also does a bit of random tidy in the driver load/init code.
> 
> Basically functions which only need dev_priv are changed to take dev_priv
> instead of dev. There is often a cascade effect and the end result is a
> saving in lines of code. And even in binary occasionally but not worth much
> mention.
> 
> Tvrtko Ursulin (10):
>   drm/i915: Make GEM object alloc/free and stolen created take dev_priv
>   drm/i915: Make GEM object create and create from data take dev_priv
>   drm/i915: Make various init functions take dev_priv
>   drm/i915: More GEM init dev_priv cleanup
>   drm/i915: dev_priv cleanup in bridge/bar/mmio init code
>   drm/i915: Unexport VGA switcheroo functions
>   drm/i915: Make gmbus setup take dev_priv
>   drm/i915: Make i915_destroy_error_state take dev_priv
>   drm/i915: Make i915_save/restore_state and intel_i2c_reset take
>     dev_priv
>   drm/i915: Make intel_pm_setup take dev_priv

They all look ok and many are a sight for sore eyes.
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

>  28 files changed, 214 insertions(+), 261 deletions(-)

Just from superlus locals and declarations, nice!
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list