For the complete audio compute config calculations, we would
need to have the transcoder M/N values as well. So move
intel_dp_audio_compute_config after the transcoder M/N values
are calculated in intel_link_compute_m_n().This becomes more
relevant after the follow-up patches where we will need to
assess if an audio frequency can be supported with a pipe config.

Signed-off-by: Vinod Govindapillai <vinod.govindapil...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c 
b/drivers/gpu/drm/i915/display/intel_dp.c
index 0fd9c1c51a43..dfe42f4dc2d1 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -3131,8 +3131,6 @@ intel_dp_compute_config(struct intel_encoder *encoder,
                adjusted_mode->crtc_clock /= n;
        }
 
-       intel_dp_audio_compute_config(encoder, pipe_config, conn_state);
-
        intel_link_compute_m_n(link_bpp_x16,
                               pipe_config->lane_count,
                               adjusted_mode->crtc_clock,
@@ -3140,6 +3138,8 @@ intel_dp_compute_config(struct intel_encoder *encoder,
                               
intel_dp_bw_fec_overhead(pipe_config->fec_enable),
                               &pipe_config->dp_m_n);
 
+       intel_dp_audio_compute_config(encoder, pipe_config, conn_state);
+
        /* FIXME: abstract this better */
        if (pipe_config->splitter.enable)
                pipe_config->dp_m_n.data_m *= pipe_config->splitter.link_count;
-- 
2.34.1

Reply via email to