[Intel-gfx] [REBASED PATCH RESEND 0/5 v2] Various cleanup

David Weinehall david.weinehall at linux.intel.com
Mon Aug 22 10:32:40 UTC 2016


This patch series aims to do some cleanup of our driver.
Patch 1, 2, and 4 should be fairly non-controversial.
Patch 3 and 5 does major cleanups of i915_sysfs and i915_debugfs,
respectively. Due to the nature of these patches they are rather
big, but that's mainly because they change the parameter passed
to macros that are used a lot.

While the secondary benefits of this patch series (the primary
of course being slightly cleaner code) might be a bit opaque,
a move to always passing dev_priv to our feature macros
(and thus allowing us to make those macros non-polymorphic)
will save more than 30kB.

That's still a bit down the road though, but I've got a full
patch series to accomplish this in a fairly straightforward
manner.

Resend: mailing list issues

David Weinehall (5):
  drm/i915: cosmetic fixes to i915_drv.h
  drm/i915: consistent struct device naming
  drm/i915: i915_sysfs.c cleanup
  drm/i915: pdev cleanup
  drm/i915: debugfs spring cleaning

 drivers/gpu/drm/i915/i915_debugfs.c     | 801 ++++++++++++++------------------
 drivers/gpu/drm/i915/i915_drv.c         | 157 ++++---
 drivers/gpu/drm/i915/i915_drv.h         |  16 +-
 drivers/gpu/drm/i915/i915_gem_gtt.c     |  16 +-
 drivers/gpu/drm/i915/i915_gem_stolen.c  |  19 +-
 drivers/gpu/drm/i915/i915_gpu_error.c   |   9 +-
 drivers/gpu/drm/i915/i915_suspend.c     |   6 +-
 drivers/gpu/drm/i915/i915_sysfs.c       | 163 +++----
 drivers/gpu/drm/i915/intel_audio.c      |  46 +-
 drivers/gpu/drm/i915/intel_display.c    |  25 +-
 drivers/gpu/drm/i915/intel_fbdev.c      |   3 +-
 drivers/gpu/drm/i915/intel_guc_loader.c |   3 +-
 drivers/gpu/drm/i915/intel_i2c.c        |   3 +-
 drivers/gpu/drm/i915/intel_runtime_pm.c |  57 +--
 drivers/gpu/drm/i915/intel_sdvo.c       |   4 +-
 15 files changed, 629 insertions(+), 699 deletions(-)

-- 
2.9.3



More information about the Intel-gfx mailing list