On Wed, Nov 30, 2016 at 11:35:30AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > 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@xxxxxxxxxxxxxxxxxx> > 28 files changed, 214 insertions(+), 261 deletions(-) Just from superlus locals and declarations, nice! -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
