Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [PATCH 0/9] drm/i915/dp: clean up DP testing
Date: Fri, 20 Sep 2024 14:56:42 +0300	[thread overview]
Message-ID: <cover.1726833193.git.jani.nikula@intel.com> (raw)

Whenever I look at doing anything in intel_dp.c I think it's grown too
big. It's over 7k lines.

The DP test functionality is fairly isolated, and mostly irrelevant for
normal operation. Move it all to its own file. This reduces intel_dp.c
by about 500 lines, and intel_display_debugfs.c by about 200 lines. And
intel_dp->compliance is now fully handled within intel_dp_test.c.

BR,
Jani.

Jani Nikula (9):
  drm/i915/dp: split out intel_dp_test.[ch] to a dedicated file
  drm/i915/dp: fix style issues in intel_dp_test.c
  drm/i915/dp: convert intel_dp_test.c struct intel_display
  drm/i915/dp: clean up intel_dp_test.[ch] interface
  drm/i915/dp: move DP test debugfs files next to the functionality
  drm/i915/dp: fix style issues in DP test debugfs
  drm/i915/display: remove the loop in fifo underrun debugfs file
    creation
  drm/i915/dp: convert DP test debugfs to struct intel_display
  drm/i915/dp: add intel_dp_test_reset() and intel_dp_test_short_pulse()

 drivers/gpu/drm/i915/Makefile                 |   1 +
 drivers/gpu/drm/i915/display/g4x_dp.c         |   7 +-
 drivers/gpu/drm/i915/display/intel_ddi.c      |   7 +-
 .../drm/i915/display/intel_display_debugfs.c  | 214 +----
 drivers/gpu/drm/i915/display/intel_dp.c       | 520 +-----------
 drivers/gpu/drm/i915/display/intel_dp.h       |   9 +-
 drivers/gpu/drm/i915/display/intel_dp_mst.c   |   5 +-
 drivers/gpu/drm/i915/display/intel_dp_test.c  | 765 ++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_dp_test.h  |  23 +
 drivers/gpu/drm/xe/Makefile                   |   1 +
 10 files changed, 816 insertions(+), 736 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_dp_test.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_dp_test.h

-- 
2.39.2


             reply	other threads:[~2024-09-20 11:57 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-20 11:56 Jani Nikula [this message]
2024-09-20 11:56 ` [PATCH 1/9] drm/i915/dp: split out intel_dp_test.[ch] to a dedicated file Jani Nikula
2024-09-20 11:56 ` [PATCH 2/9] drm/i915/dp: fix style issues in intel_dp_test.c Jani Nikula
2024-09-20 11:56 ` [PATCH 3/9] drm/i915/dp: convert intel_dp_test.c struct intel_display Jani Nikula
2024-09-20 11:56 ` [PATCH 4/9] drm/i915/dp: clean up intel_dp_test.[ch] interface Jani Nikula
2024-09-20 11:56 ` [PATCH 5/9] drm/i915/dp: move DP test debugfs files next to the functionality Jani Nikula
2024-09-20 11:56 ` [PATCH 6/9] drm/i915/dp: fix style issues in DP test debugfs Jani Nikula
2024-09-20 11:56 ` [PATCH 7/9] drm/i915/display: remove the loop in fifo underrun debugfs file creation Jani Nikula
2024-09-20 11:56 ` [PATCH 8/9] drm/i915/dp: convert DP test debugfs to struct intel_display Jani Nikula
2024-09-20 11:56 ` [PATCH 9/9] drm/i915/dp: add intel_dp_test_reset() and intel_dp_test_short_pulse() Jani Nikula
2024-09-20 12:26 ` ✓ Fi.CI.BAT: success for drm/i915/dp: clean up DP testing Patchwork
2024-09-20 12:27 ` ✗ Fi.CI.CHECKPATCH: warning " Patchwork
2024-09-20 12:27 ` ✗ Fi.CI.SPARSE: " Patchwork
2024-09-20 13:34 ` [PATCH 0/9] " Ville Syrjälä
2024-09-23  7:33   ` Jani Nikula
2024-09-21 12:26 ` ✗ Fi.CI.IGT: failure for " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1726833193.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox