[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