Intel-XE 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, imre.deak@intel.com,
	ville.syrjala@linux.intel.com, nagavenkata.srikanth.v@intel.com
Subject: [RFC v0 00/13] drm/i915/dp: 128b/132b uncompressed SST
Date: Fri, 13 Dec 2024 12:39:44 +0200	[thread overview]
Message-ID: <cover.1734085515.git.jani.nikula@intel.com> (raw)

This is a completely untested set towards enabling 128b/132b SST,
initially without support for compression. I erred on the side of
posting it early for comments instead of waiting until I've had a chance
to give it a spin on a display that supports 128b/132b SST.

I've chopped this up to unnecessarily small pieces just to make it
easier for me to juggle the patches. If desired, it's trivial to squash
them before posting the series for merging. But maybe this helps the
reviews too?

128b/132b SST+DSC still needs the compute config, and link training
fallbacks need to be reviewed with UHBR SST in mind, but this is the
first step.

Many thanks to Imre for helping me get the facts straight with MTP; all
errors are still mine.

BR,
Jani.


Jani Nikula (13):
  drm/mst: remove mgr parameter and debug logging from
    drm_dp_get_vc_payload_bw()
  drm/i915/mst: use intel_dp_compute_config_limits() for DP MST
  drm/i915/mst: drop connector parameter from intel_dp_mst_bw_overhead()
  drm/i915/mst: drop connector parameter from intel_dp_mst_compute_m_n()
  drm/i915/mst: change return value of
    mst_stream_find_vcpi_slots_for_bpp()
  drm/i915/mst: add helper independent of MST for figuring out the TU
  drm/i915/ddi: enable 128b/132b TRANS_DDI_FUNC_CTL mode for UHBR SST
  drm/i915/ddi: 128b/132b SST also needs DP_TP_CTL_MODE_MST
  drm/i915/ddi: write payload for 128b/132b SST
  drm/i915/ddi: initialize 128b/132b SST DP2 VFREQ registers
  drm/i915/ddi: enable ACT handling for 128b/132b SST
  drm/i915/ddi: start distinguishing 128b/132b SST and MST at state
    readout
  drm/i915/dp: compute config for 128b/132b SST w/o DSC

 drivers/gpu/drm/display/drm_dp_mst_topology.c |  10 +-
 drivers/gpu/drm/i915/display/intel_ddi.c      |  76 ++++++++-
 drivers/gpu/drm/i915/display/intel_dp.c       |  67 +++++---
 drivers/gpu/drm/i915/display/intel_dp.h       |  10 +-
 drivers/gpu/drm/i915/display/intel_dp_mst.c   | 154 ++++++++----------
 drivers/gpu/drm/i915/display/intel_dp_mst.h   |   7 +
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |   3 +-
 .../gpu/drm/tests/drm_dp_mst_helper_test.c    |   4 +-
 include/drm/display/drm_dp_mst_helper.h       |   3 +-
 9 files changed, 193 insertions(+), 141 deletions(-)

-- 
2.39.5


             reply	other threads:[~2024-12-13 10:40 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-13 10:39 Jani Nikula [this message]
2024-12-13 10:39 ` [RFC v0 01/13] drm/mst: remove mgr parameter and debug logging from drm_dp_get_vc_payload_bw() Jani Nikula
2024-12-13 10:39 ` [RFC v0 02/13] drm/i915/mst: use intel_dp_compute_config_limits() for DP MST Jani Nikula
2024-12-13 10:39 ` [RFC v0 03/13] drm/i915/mst: drop connector parameter from intel_dp_mst_bw_overhead() Jani Nikula
2024-12-13 10:39 ` [RFC v0 04/13] drm/i915/mst: drop connector parameter from intel_dp_mst_compute_m_n() Jani Nikula
2024-12-13 10:39 ` [RFC v0 05/13] drm/i915/mst: change return value of mst_stream_find_vcpi_slots_for_bpp() Jani Nikula
2024-12-13 10:39 ` [RFC v0 06/13] drm/i915/mst: add helper independent of MST for figuring out the TU Jani Nikula
2024-12-13 18:44   ` Imre Deak
2024-12-13 10:39 ` [RFC v0 07/13] drm/i915/ddi: enable 128b/132b TRANS_DDI_FUNC_CTL mode for UHBR SST Jani Nikula
2024-12-13 10:39 ` [RFC v0 08/13] drm/i915/ddi: 128b/132b SST also needs DP_TP_CTL_MODE_MST Jani Nikula
2024-12-13 10:39 ` [RFC v0 09/13] drm/i915/ddi: write payload for 128b/132b SST Jani Nikula
2024-12-13 10:39 ` [RFC v0 10/13] drm/i915/ddi: initialize 128b/132b SST DP2 VFREQ registers Jani Nikula
2024-12-13 10:39 ` [RFC v0 11/13] drm/i915/ddi: enable ACT handling for 128b/132b SST Jani Nikula
2024-12-13 10:39 ` [RFC v0 12/13] drm/i915/ddi: start distinguishing 128b/132b SST and MST at state readout Jani Nikula
2024-12-13 10:39 ` [RFC v0 13/13] drm/i915/dp: compute config for 128b/132b SST w/o DSC Jani Nikula
2024-12-13 12:53 ` ✓ CI.Patch_applied: success for drm/i915/dp: 128b/132b uncompressed SST Patchwork
2024-12-13 12:54 ` ✓ CI.checkpatch: " Patchwork
2024-12-13 12:55 ` ✓ CI.KUnit: " Patchwork
2024-12-13 13:13 ` ✓ CI.Build: " Patchwork
2024-12-13 13:15 ` ✓ CI.Hooks: " Patchwork
2024-12-13 13:17 ` ✗ CI.checksparse: warning " Patchwork
2024-12-13 13:52 ` ✓ Xe.CI.BAT: success " Patchwork
2024-12-13 23:15 ` ✗ Xe.CI.Full: failure " 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.1734085515.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=imre.deak@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=nagavenkata.srikanth.v@intel.com \
    --cc=ville.syrjala@linux.intel.com \
    /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