This is better than keep those values in the code that you always
need to check the DP spec to know what level of HBR it is.

Signed-off-by: José Roberto de Souza <jose.so...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_ddi.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c 
b/drivers/gpu/drm/i915/display/intel_ddi.c
index a976606d21c7..914f0cc4d237 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -49,6 +49,10 @@
 #include "intel_tc.h"
 #include "intel_vdsc.h"
 
+#define HBR_RATE 270000
+#define HBR2_RATE 540000
+#define HBR3_RATE 810000
+
 struct ddi_buf_trans {
        u32 trans1;     /* balance leg enable, de-emph level */
        u32 trans2;     /* vref sel, vswing */
@@ -888,7 +892,7 @@ icl_get_combo_buf_trans(struct drm_i915_private *dev_priv, 
int type, int rate,
        if (type == INTEL_OUTPUT_HDMI) {
                *n_entries = ARRAY_SIZE(icl_combo_phy_ddi_translations_hdmi);
                return icl_combo_phy_ddi_translations_hdmi;
-       } else if (rate > 540000 && type == INTEL_OUTPUT_EDP) {
+       } else if (rate > HBR2_RATE && type == INTEL_OUTPUT_EDP) {
                *n_entries = 
ARRAY_SIZE(icl_combo_phy_ddi_translations_edp_hbr3);
                return icl_combo_phy_ddi_translations_edp_hbr3;
        } else if (type == INTEL_OUTPUT_EDP && dev_priv->vbt.edp.low_vswing) {
-- 
2.24.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to