JSL has update in vswing table for eDP.

BSpec: 21257

Changes since V3 :
        - Changed IS_EHL_JSL to IS_JSL_EH
        - Reverted removal of IS_ELKHARTLAKE and also
              added IS_JASPERLAKE
        - Corrected mistake of using IS_ELKHARTLAKE twice and
          missing IS_JASPERLAKE

Changes since V2 :
        - Added IS_EHL_JSL to replace IS_ELKHARTLAKE
        - EHL/JSL PCI ids split added
        - Changes rebased as per new drm top commit

Changes since V1 :
            - IS_ELKHARTLAKE and IS_JASPERLAKE is replaced with
              HAS_PCH_MCC(EHL) and HAS_PCH_JSP(JSL) respectively
            - Reverted EHL/JSL PCI ids split change

Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadh...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_ddi.c | 93 ++++++++++++++++++++++--
 1 file changed, 88 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c 
b/drivers/gpu/drm/i915/display/intel_ddi.c
index b4c520348b3b..7589508ff7e2 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -582,6 +582,34 @@ static const struct cnl_ddi_buf_trans 
ehl_combo_phy_ddi_translations_dp[] = {
        { 0x6, 0x7F, 0x3F, 0x00, 0x00 },        /* 900   900      0.0   */
 };
 
+static const struct cnl_ddi_buf_trans jsl_combo_phy_ddi_translations_edp_hbr[] 
= {
+                                               /* NT mV Trans mV db    */
+       { 0x8, 0x7F, 0x3F, 0x00, 0x00 },        /* 200   200      0.0   */
+       { 0x8, 0x7F, 0x38, 0x00, 0x07 },        /* 200   250      1.9   */
+       { 0x1, 0x7F, 0x33, 0x00, 0x0C },        /* 200   300      3.5   */
+       { 0xA, 0x35, 0x36, 0x00, 0x09 },        /* 200   350      4.9   */
+       { 0x8, 0x7F, 0x3F, 0x00, 0x00 },        /* 250   250      0.0   */
+       { 0x1, 0x7F, 0x38, 0x00, 0x07 },        /* 250   300      1.6   */
+       { 0xA, 0x35, 0x35, 0x00, 0x0A },        /* 250   350      2.9   */
+       { 0x1, 0x7F, 0x3F, 0x00, 0x00 },        /* 300   300      0.0   */
+       { 0xA, 0x35, 0x38, 0x00, 0x07 },        /* 300   350      1.3   */
+       { 0xA, 0x35, 0x3F, 0x00, 0x00 },        /* 350   350      0.0   */
+};
+
+static const struct cnl_ddi_buf_trans 
jsl_combo_phy_ddi_translations_edp_hbr2[] = {
+                                               /* NT mV Trans mV db    */
+       { 0x8, 0x7F, 0x3F, 0x00, 0x00 },        /* 200   200      0.0   */
+       { 0x8, 0x7F, 0x3F, 0x00, 0x00 },        /* 200   250      1.9   */
+       { 0x1, 0x7F, 0x3D, 0x00, 0x02 },        /* 200   300      3.5   */
+       { 0xA, 0x35, 0x38, 0x00, 0x07 },        /* 200   350      4.9   */
+       { 0x8, 0x7F, 0x3F, 0x00, 0x00 },        /* 250   250      0.0   */
+       { 0x1, 0x7F, 0x3F, 0x00, 0x00 },        /* 250   300      1.6   */
+       { 0xA, 0x35, 0x3A, 0x00, 0x05 },        /* 250   350      2.9   */
+       { 0x1, 0x7F, 0x3F, 0x00, 0x00 },        /* 300   300      0.0   */
+       { 0xA, 0x35, 0x38, 0x00, 0x07 },        /* 300   350      1.3   */
+       { 0xA, 0x35, 0x3F, 0x00, 0x00 },        /* 350   350      0.0   */
+};
+
 struct icl_mg_phy_ddi_buf_trans {
        u32 cri_txdeemph_override_11_6;
        u32 cri_txdeemph_override_5_0;
@@ -1167,6 +1195,57 @@ ehl_get_combo_buf_trans(struct intel_encoder *encoder,
                return ehl_get_combo_buf_trans_dp(encoder, crtc_state, 
n_entries);
 }
 
+static const struct cnl_ddi_buf_trans *
+jsl_get_combo_buf_trans_hdmi(struct intel_encoder *encoder,
+                            const struct intel_crtc_state *crtc_state,
+                            int *n_entries)
+{
+       *n_entries = ARRAY_SIZE(icl_combo_phy_ddi_translations_hdmi);
+       return icl_combo_phy_ddi_translations_hdmi;
+}
+
+static const struct cnl_ddi_buf_trans *
+jsl_get_combo_buf_trans_dp(struct intel_encoder *encoder,
+                          const struct intel_crtc_state *crtc_state,
+                          int *n_entries)
+{
+       *n_entries = ARRAY_SIZE(icl_combo_phy_ddi_translations_dp_hbr2);
+       return icl_combo_phy_ddi_translations_dp_hbr2;
+}
+
+static const struct cnl_ddi_buf_trans *
+jsl_get_combo_buf_trans_edp(struct intel_encoder *encoder,
+                           const struct intel_crtc_state *crtc_state,
+                           int *n_entries)
+{
+       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
+
+       if (dev_priv->vbt.edp.low_vswing) {
+               if (crtc_state->port_clock > 270000) {
+                       *n_entries = 
ARRAY_SIZE(jsl_combo_phy_ddi_translations_edp_hbr2);
+                       return jsl_combo_phy_ddi_translations_edp_hbr2;
+               } else {
+                       *n_entries = 
ARRAY_SIZE(jsl_combo_phy_ddi_translations_edp_hbr);
+                       return jsl_combo_phy_ddi_translations_edp_hbr;
+               }
+       }
+
+       return jsl_get_combo_buf_trans_dp(encoder, crtc_state, n_entries);
+}
+
+static const struct cnl_ddi_buf_trans *
+jsl_get_combo_buf_trans(struct intel_encoder *encoder,
+                       const struct intel_crtc_state *crtc_state,
+                       int *n_entries)
+{
+       if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
+               return jsl_get_combo_buf_trans_hdmi(encoder, crtc_state, 
n_entries);
+       else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP))
+               return jsl_get_combo_buf_trans_edp(encoder, crtc_state, 
n_entries);
+       else
+               return jsl_get_combo_buf_trans_dp(encoder, crtc_state, 
n_entries);
+}
+
 static const struct cnl_ddi_buf_trans *
 tgl_get_combo_buf_trans_hdmi(struct intel_encoder *encoder,
                             const struct intel_crtc_state *crtc_state,
@@ -2368,7 +2447,9 @@ static u8 intel_ddi_dp_voltage_max(struct intel_dp 
*intel_dp,
                else
                        tgl_get_dkl_buf_trans(encoder, crtc_state, &n_entries);
        } else if (INTEL_GEN(dev_priv) == 11) {
-               if (IS_ELKHARTLAKE(dev_priv))
+               if (IS_JASPERLAKE(dev_priv))
+                       jsl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
+               else if (IS_ELKHARTLAKE(dev_priv))
                        ehl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
                else if (intel_phy_is_combo(dev_priv, phy))
                        icl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
@@ -2549,6 +2630,8 @@ static void icl_ddi_combo_vswing_program(struct 
intel_encoder *encoder,
 
        if (INTEL_GEN(dev_priv) >= 12)
                ddi_translations = tgl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
+       else if (IS_JASPERLAKE(dev_priv))
+               ddi_translations = jsl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
        else if (IS_ELKHARTLAKE(dev_priv))
                ddi_translations = ehl_get_combo_buf_trans(encoder, crtc_state, 
&n_entries);
        else
@@ -3140,7 +3223,7 @@ static void intel_ddi_clk_select(struct intel_encoder 
*encoder,
                if (!intel_phy_is_combo(dev_priv, phy))
                        intel_de_write(dev_priv, DDI_CLK_SEL(port),
                                       icl_pll_to_ddi_clk_sel(encoder, 
crtc_state));
-               else if (IS_ELKHARTLAKE(dev_priv) && port >= PORT_C)
+               else if (IS_JSL_EHL(dev_priv) && port >= PORT_C)
                        /*
                         * MG does not exist but the programming is required
                         * to ungate DDIC and DDID
@@ -3189,7 +3272,7 @@ static void intel_ddi_clk_disable(struct intel_encoder 
*encoder)
 
        if (INTEL_GEN(dev_priv) >= 11) {
                if (!intel_phy_is_combo(dev_priv, phy) ||
-                   (IS_ELKHARTLAKE(dev_priv) && port >= PORT_C))
+                   (IS_JSL_EHL(dev_priv) && port >= PORT_C))
                        intel_de_write(dev_priv, DDI_CLK_SEL(port),
                                       DDI_CLK_SEL_NONE);
        } else if (IS_CANNONLAKE(dev_priv)) {
@@ -4334,7 +4417,7 @@ void intel_ddi_compute_min_voltage_level(struct 
drm_i915_private *dev_priv,
 {
        if (INTEL_GEN(dev_priv) >= 12 && crtc_state->port_clock > 594000)
                crtc_state->min_voltage_level = 2;
-       else if (IS_ELKHARTLAKE(dev_priv) && crtc_state->port_clock > 594000)
+       else if (IS_JSL_EHL(dev_priv) && crtc_state->port_clock > 594000)
                crtc_state->min_voltage_level = 3;
        else if (INTEL_GEN(dev_priv) >= 11 && crtc_state->port_clock > 594000)
                crtc_state->min_voltage_level = 1;
@@ -5186,7 +5269,7 @@ void intel_ddi_init(struct drm_i915_private *dev_priv, 
enum port port)
                encoder->hpd_pin = rkl_hpd_pin(dev_priv, port);
        else if (INTEL_GEN(dev_priv) >= 12)
                encoder->hpd_pin = tgl_hpd_pin(dev_priv, port);
-       else if (IS_ELKHARTLAKE(dev_priv))
+       else if (IS_JSL_EHL(dev_priv))
                encoder->hpd_pin = ehl_hpd_pin(dev_priv, port);
        else if (IS_GEN(dev_priv, 11))
                encoder->hpd_pin = icl_hpd_pin(dev_priv, port);
-- 
2.28.0

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to