All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/3] Make GEN macros more similar
@ 2018-12-06  6:11 Lucas De Marchi
  2018-12-06  6:11 ` [PATCH v3 1/3] drm/i915: Rename IS_GEN to IS_GEN_RANGE Lucas De Marchi
                   ` (6 more replies)
  0 siblings, 7 replies; 19+ messages in thread
From: Lucas De Marchi @ 2018-12-06  6:11 UTC (permalink / raw)
  To: intel-gfx; +Cc: Rodrigo Vivi

Third version of the series trying to make GEN checks more similar. It's
basically a dial down from what was done in v2 to get the points we have
rough consensus that should be done. Hopefully.

Changes from v2:
  - Don't use GT, but continue with IS_ prefix. If we ever add a check
    for display version (which we are uncertain is actually necessary),
    we can document the convention of using IS_GEN for GT and
    IS_DISPLAY_GEN for display

  - Don't port comparisons using >, >=, <, <=, == over to the macro:
    minimize code churn and in most cases it makes the code more
    difficult to read

  - Don't bring GEN_FOREVER back since we are not covering unbound
    checks anymore with the macros

  - Don't remove INTEL_GEN

Lucas De Marchi (3):
  drm/i915: Rename IS_GEN to IS_GEN_RANGE
  drm/i915: replace IS_GEN<N> with IS_GEN(..., N)
  drm/i915: merge gen checks to use range

 drivers/gpu/drm/i915/gvt/vgpu.c            |  4 +-
 drivers/gpu/drm/i915/i915_cmd_parser.c     |  2 +-
 drivers/gpu/drm/i915/i915_debugfs.c        | 16 ++---
 drivers/gpu/drm/i915/i915_drv.c            | 18 +++---
 drivers/gpu/drm/i915/i915_drv.h            | 31 +++------
 drivers/gpu/drm/i915/i915_gem.c            | 14 ++--
 drivers/gpu/drm/i915/i915_gem_context.c    |  2 +-
 drivers/gpu/drm/i915/i915_gem_execbuffer.c |  4 +-
 drivers/gpu/drm/i915/i915_gem_fence_reg.c  | 10 +--
 drivers/gpu/drm/i915/i915_gem_gtt.c        |  6 +-
 drivers/gpu/drm/i915/i915_gem_stolen.c     |  7 +-
 drivers/gpu/drm/i915/i915_gem_tiling.c     |  4 +-
 drivers/gpu/drm/i915/i915_gpu_error.c      | 18 +++---
 drivers/gpu/drm/i915/i915_irq.c            | 24 +++----
 drivers/gpu/drm/i915/i915_perf.c           |  8 +--
 drivers/gpu/drm/i915/i915_suspend.c        | 12 ++--
 drivers/gpu/drm/i915/intel_atomic.c        |  2 +-
 drivers/gpu/drm/i915/intel_audio.c         |  2 +-
 drivers/gpu/drm/i915/intel_bios.c          |  2 +-
 drivers/gpu/drm/i915/intel_cdclk.c         | 10 +--
 drivers/gpu/drm/i915/intel_crt.c           |  6 +-
 drivers/gpu/drm/i915/intel_device_info.c   | 16 ++---
 drivers/gpu/drm/i915/intel_display.c       | 74 +++++++++++-----------
 drivers/gpu/drm/i915/intel_dp.c            | 24 +++----
 drivers/gpu/drm/i915/intel_engine_cs.c     |  6 +-
 drivers/gpu/drm/i915/intel_fbc.c           | 24 +++----
 drivers/gpu/drm/i915/intel_fifo_underrun.c |  6 +-
 drivers/gpu/drm/i915/intel_guc_fw.c        |  2 +-
 drivers/gpu/drm/i915/intel_hangcheck.c     |  4 +-
 drivers/gpu/drm/i915/intel_lrc.c           |  4 +-
 drivers/gpu/drm/i915/intel_lvds.c          |  4 +-
 drivers/gpu/drm/i915/intel_mocs.c          |  2 +-
 drivers/gpu/drm/i915/intel_overlay.c       | 10 +--
 drivers/gpu/drm/i915/intel_panel.c         |  8 +--
 drivers/gpu/drm/i915/intel_pipe_crc.c      |  8 +--
 drivers/gpu/drm/i915/intel_pm.c            | 60 +++++++++---------
 drivers/gpu/drm/i915/intel_psr.c           |  4 +-
 drivers/gpu/drm/i915/intel_ringbuffer.c    | 36 +++++------
 drivers/gpu/drm/i915/intel_ringbuffer.h    |  4 +-
 drivers/gpu/drm/i915/intel_runtime_pm.c    |  4 +-
 drivers/gpu/drm/i915/intel_sprite.c        |  6 +-
 drivers/gpu/drm/i915/intel_uc.c            |  2 +-
 drivers/gpu/drm/i915/intel_uncore.c        | 30 ++++-----
 drivers/gpu/drm/i915/intel_wopcm.c         |  4 +-
 drivers/gpu/drm/i915/intel_workarounds.c   |  4 +-
 45 files changed, 268 insertions(+), 280 deletions(-)

-- 
2.19.1.1.g56c4683e68

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2018-12-13  1:01 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-06  6:11 [PATCH v3 0/3] Make GEN macros more similar Lucas De Marchi
2018-12-06  6:11 ` [PATCH v3 1/3] drm/i915: Rename IS_GEN to IS_GEN_RANGE Lucas De Marchi
2018-12-06 12:28   ` Tvrtko Ursulin
2018-12-11 14:24   ` Jani Nikula
2018-12-06  6:11 ` [PATCH v3 2/3] drm/i915: replace IS_GEN<N> with IS_GEN(..., N) Lucas De Marchi
2018-12-06 12:37   ` Tvrtko Ursulin
2018-12-07  1:17     ` Lucas De Marchi
2018-12-07 11:30       ` Tvrtko Ursulin
2018-12-07 11:42         ` Tvrtko Ursulin
2018-12-07 20:57         ` Lucas De Marchi
2018-12-10  8:27           ` Tvrtko Ursulin
2018-12-11 14:34   ` Jani Nikula
2018-12-06  6:11 ` [PATCH v3 3/3] drm/i915: merge gen checks to use range Lucas De Marchi
2018-12-11 14:35   ` Jani Nikula
2018-12-13  1:01     ` Rodrigo Vivi
2018-12-06  6:32 ` ✗ Fi.CI.CHECKPATCH: warning for Make GEN macros more similar (rev3) Patchwork
2018-12-06  6:34 ` ✗ Fi.CI.SPARSE: " Patchwork
2018-12-06  6:49 ` ✓ Fi.CI.BAT: success " Patchwork
2018-12-06 20:20 ` ✓ Fi.CI.IGT: " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.