drm/i915: Rename ICL_PORT_TX_DW6 bits
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 8 Mar 2024 07:24:00 +0000 (09:24 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 20 Mar 2024 13:26:42 +0000 (15:26 +0200)
Our definitions for bit 7 and bit 0 of ICL_PORT_TX_DW6 are
swapped. Functionally it doesn't matter as we always set both
bits, but let's rename the bits to match bspec 100%.

And while at it, add the definition for bits 1-6 as well, just
to have it all fully documented.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240308072400.28918-1-ville.syrjala@linux.intel.com
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/display/intel_combo_phy_regs.h
drivers/gpu/drm/i915/display/intel_display_power_well.c

index 63601129b7361b5da8b3a44cd9c8d4221cfc2bc0..0964e392d02c7921f5ba78290a2c0ec1cba5d56f 100644 (file)
 #define ICL_PORT_TX_DW6_AUX(phy)               _MMIO(_ICL_PORT_TX_DW_AUX(6, phy))
 #define ICL_PORT_TX_DW6_GRP(phy)               _MMIO(_ICL_PORT_TX_DW_GRP(6, phy))
 #define ICL_PORT_TX_DW6_LN(ln, phy)            _MMIO(_ICL_PORT_TX_DW_LN(6, ln, phy))
-#define   ICL_AUX_ANAOVRD1_LDO_BYPASS          REG_BIT(7)
-#define   ICL_AUX_ANAOVRD1_ENABLE              REG_BIT(0)
+#define   O_FUNC_OVRD_EN                       REG_BIT(7)
+#define   O_LDO_REF_SEL_CRI                    REG_GENMASK(6, 1)
+#define   O_LDO_BYPASS_CRI                     REG_BIT(0)
 
 #define ICL_PORT_TX_DW7_AUX(phy)               _MMIO(_ICL_PORT_TX_DW_AUX(7, phy))
 #define ICL_PORT_TX_DW7_GRP(phy)               _MMIO(_ICL_PORT_TX_DW_GRP(7, phy))
index 217f82f1da841d0144e609e37c431191c8a19b84..78005d12638c6d186f39f63ffbd9fe3aea83ad01 100644 (file)
@@ -425,7 +425,7 @@ icl_combo_phy_aux_power_well_enable(struct drm_i915_private *dev_priv,
        if (pw_idx >= ICL_PW_CTL_IDX_AUX_A && pw_idx <= ICL_PW_CTL_IDX_AUX_B &&
            !intel_aux_ch_is_edp(dev_priv, ICL_AUX_PW_TO_CH(pw_idx)))
                intel_de_rmw(dev_priv, ICL_PORT_TX_DW6_AUX(ICL_AUX_PW_TO_PHY(pw_idx)),
-                            0, ICL_AUX_ANAOVRD1_ENABLE | ICL_AUX_ANAOVRD1_LDO_BYPASS);
+                            0, O_FUNC_OVRD_EN | O_LDO_BYPASS_CRI);
 }
 
 static void