On Wed, Nov 16, 2016 at 08:55:30AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Yet more churn. :) But good news are they are mostly really small patches, with > the exception to intel_display.c and some heavier function prototype changes > in i915_gem_gtt.c. Just to avoid having a lot of functions which take dev and > actually need dev_priv. > > Small shrink of the driver binary and small source code lines shrink. > > text data bss dec hex filename > 1076575 23749 576 1100900 10cc64 i915.ko.0 > 1076079 23749 576 1100404 10ca74 i915.ko.2 > > Tvrtko Ursulin (15): > drm/i915: dev_priv and a small cascade of cleanups in i915_gem.c > drm/i915: Use dev_priv in INTEL_INFO in i915_gem_execbuffer.c > drm/i915: Use dev_priv in INTEL_INFO in i915_gem_fence_reg.c > drm/i915: dev_priv cleanup in i915_gem_gtt.c > drm/i915: dev_priv cleanup in i915_gem_stolen.c > drm/i915: dev_priv cleanup in i915_gem_tiling.c > drm/i915: dev_priv cleanup in i915_gpu_error.c > drm/i915: dev_priv cleanup in i915_irq.c > drm/i915: dev_priv cleanup in i915_suspend.c > drm/i915: Assorted INTEL_INFO(dev) cleanups > drm/i915: dev_priv cleanup in intel_dp.c > drm/i915: dev_priv cleanup in intel_pm.c > drm/i915: dev_priv cleanup in intel_display.c > drm/i915: Fix for_each_pipe argument in vlv_display_power_well_init > drm/i915: Remove __I915__ magic macro I read them on trybot and couldn't see anything I didn't object to, so Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> I did feel like there were some more opportunities remaining to remove additional extraneous dev_priv dancing. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
