From: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
To: intel-xe@lists.freedesktop.org, intel-gfx@lists.freedesktop.org
Cc: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Subject: [v5 1/2] drm/i915/hdmi: Move audio compute config after format selection
Date: Tue, 1 Sep 2026 16:06:38 +0530 [thread overview]
Message-ID: <20260901103639.617757-2-chaitanya.kumar.borah@intel.com> (raw)
In-Reply-To: <20260901103639.617757-1-chaitanya.kumar.borah@intel.com>
As we plan to add audio BW calculation in i915,
intel_audio_compute_config() would need the final output_format
and pipe_bpp to correctly determine audio bandwidth constraints.
Move the intel_audio_compute_config() call to after
intel_hdmi_compute_formats() returns, ensuring the audio
configuration is computed with the final parameters.
Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
---
drivers/gpu/drm/i915/display/intel_hdmi.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c b/drivers/gpu/drm/i915/display/intel_hdmi.c
index 9b637e38a1a5..b6fa1a1558b8 100644
--- a/drivers/gpu/drm/i915/display/intel_hdmi.c
+++ b/drivers/gpu/drm/i915/display/intel_hdmi.c
@@ -2399,10 +2399,6 @@ int intel_hdmi_compute_config(struct intel_encoder *encoder,
if (!intel_link_bw_compute_pipe_bpp(pipe_config))
return -EINVAL;
- pipe_config->has_audio =
- intel_hdmi_has_audio(encoder, pipe_config, conn_state) &&
- intel_audio_compute_config(encoder, pipe_config, conn_state);
-
/*
* Try to respect downstream TMDS clock limits first, if
* that fails assume the user might know something we don't.
@@ -2417,6 +2413,10 @@ int intel_hdmi_compute_config(struct intel_encoder *encoder,
return ret;
}
+ pipe_config->has_audio =
+ intel_hdmi_has_audio(encoder, pipe_config, conn_state) &&
+ intel_audio_compute_config(encoder, pipe_config, conn_state);
+
ret = intel_pfit_compute_config(pipe_config, conn_state);
if (ret)
return ret;
--
2.50.1
next prev parent reply other threads:[~2026-09-01 11:00 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-01 10:36 [v5 0/2] drm/i915/audio: Add HDMI TMDS audio bandwidth check Chaitanya Kumar Borah
2026-09-01 10:36 ` Chaitanya Kumar Borah [this message]
2026-09-02 7:34 ` [v5 1/2] drm/i915/hdmi: Move audio compute config after format selection Nautiyal, Ankit K
2026-09-01 10:36 ` [v5 2/2] drm/i915/audio: Prune ELD SADs based on HDMI audio bandwidth Chaitanya Kumar Borah
2026-09-02 8:00 ` Nautiyal, Ankit K
2026-09-01 11:08 ` ✓ CI.KUnit: success for drm/i915/audio: Add HDMI TMDS audio bandwidth check (rev5) Patchwork
2026-09-01 11:46 ` ✓ Xe.CI.BAT: " Patchwork
2026-09-01 13:07 ` ✓ Xe.CI.FULL: " Patchwork
2026-09-01 15:07 ` ✓ i915.CI.BAT: " Patchwork
2026-09-01 21:18 ` ✗ i915.CI.Full: failure " Patchwork
2026-09-02 11:26 ` ✓ i915.CI.Full: success " 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=20260901103639.617757-2-chaitanya.kumar.borah@intel.com \
--to=chaitanya.kumar.borah@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 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.