From: Imre Deak <imre.deak@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: Jani Nikula <jani.nikula@intel.com>
Subject: [PATCH v2 3/5] drm/i915/dp_mst: Add a way to disable the DP2 config
Date: Wed, 30 Oct 2024 21:23:11 +0200 [thread overview]
Message-ID: <20241030192313.4030617-4-imre.deak@intel.com> (raw)
In-Reply-To: <20241030192313.4030617-1-imre.deak@intel.com>
Add a way to disable the DP2 configuration, required by the next patch
during transcoder disabling.
While at it drop the redundant encoder parameter.
v2:
- Keep intel_ddi_config_transcoder_dp2() static. (Jani)
- Remove the encoder parameter.
Cc: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
---
drivers/gpu/drm/i915/display/intel_ddi.c | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 069cca4b38b2e..dcd43087fa7ed 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -455,14 +455,18 @@ static u32 bdw_trans_port_sync_master_select(enum transcoder master_transcoder)
}
static void
-intel_ddi_config_transcoder_dp2(struct intel_encoder *encoder,
- const struct intel_crtc_state *crtc_state)
+intel_ddi_config_transcoder_dp2(const struct intel_crtc_state *crtc_state,
+ bool enable)
{
- struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+ struct intel_display *display = to_intel_display(crtc_state);
+ struct drm_i915_private *i915 = to_i915(display->drm);
enum transcoder cpu_transcoder = crtc_state->cpu_transcoder;
u32 val = 0;
- if (intel_dp_is_uhbr(crtc_state))
+ if (!HAS_DP20(i915))
+ return;
+
+ if (enable && intel_dp_is_uhbr(crtc_state))
val = TRANS_DP2_128B132B_CHANNEL_CODING;
intel_de_write(i915, TRANS_DP2_CTL(cpu_transcoder), val);
@@ -2549,7 +2553,7 @@ static void mtl_ddi_pre_enable_dp(struct intel_atomic_state *state,
/*
* 6.b If DP v2.0/128b mode - Configure TRANS_DP2_CTL register settings.
*/
- intel_ddi_config_transcoder_dp2(encoder, crtc_state);
+ intel_ddi_config_transcoder_dp2(crtc_state, true);
/*
* 6.c Configure TRANS_DDI_FUNC_CTL DDI Select, DDI Mode Select & MST
@@ -2686,8 +2690,7 @@ static void tgl_ddi_pre_enable_dp(struct intel_atomic_state *state,
*/
intel_ddi_enable_transcoder_clock(encoder, crtc_state);
- if (HAS_DP20(dev_priv))
- intel_ddi_config_transcoder_dp2(encoder, crtc_state);
+ intel_ddi_config_transcoder_dp2(crtc_state, true);
/*
* 7.b Configure TRANS_DDI_FUNC_CTL DDI Select, DDI Mode Select & MST
--
2.44.2
next prev parent reply other threads:[~2024-10-30 19:22 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-30 19:23 [PATCH v2 0/5] drm/dp_mst: Fix DDI function/DP2 config programming Imre Deak
2024-10-30 19:23 ` [PATCH v2 1/5] drm/i915/ptl/dp_mst: Fix slave transcoder enabling wrt. DDI function Imre Deak
2024-10-31 10:44 ` Luca Coelho
2024-10-31 10:57 ` Imre Deak
2024-10-31 10:59 ` Luca Coelho
2024-10-30 19:23 ` [PATCH v2 2/5] drm/i915/adlp+/dp_mst: Align slave transcoder enabling with spec " Imre Deak
2024-10-31 10:49 ` Luca Coelho
2024-10-31 11:40 ` Imre Deak
2024-11-06 12:38 ` Luca Coelho
2024-10-30 19:23 ` Imre Deak [this message]
2024-10-31 10:52 ` [PATCH v2 3/5] drm/i915/dp_mst: Add a way to disable the DP2 config Luca Coelho
2024-10-31 12:36 ` Jani Nikula
2024-10-31 12:42 ` Imre Deak
2024-10-30 19:23 ` [PATCH v2 4/5] drm/i915/adlp+/dp_mst: Align slave transcoder sequences with spec wrt. " Imre Deak
2024-10-31 10:54 ` Luca Coelho
2024-10-30 19:23 ` [PATCH v2 5/5] drm/i915/adlp+/dp_mst: Align master transcoder disabling " Imre Deak
2024-10-31 10:56 ` Luca Coelho
2024-10-31 11:03 ` Imre Deak
2024-10-30 19:50 ` ✗ Fi.CI.SPARSE: warning for drm/dp_mst: Fix DDI function/DP2 config programming Patchwork
2024-10-30 21:24 ` ✗ Fi.CI.BAT: failure " Patchwork
2024-10-30 21:47 ` Imre Deak
2024-11-04 10:50 ` Illipilli, TejasreeX
2024-11-04 10:41 ` ✓ Fi.CI.BAT: success " Patchwork
2024-11-04 16:24 ` ✗ Fi.CI.IGT: failure " Patchwork
2024-11-06 16:29 ` Imre Deak
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=20241030192313.4030617-4-imre.deak@intel.com \
--to=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@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