Hi Imre and Jani,
Could you please review this series in order to add DP2.1 reg defn.
Thank You
Khaled
On Thu, 2023-06-08 at 22:49 -0700, Almahallawy, Khaled wrote:
> Starting from DP2.0 specs, DPCD 248h is renamed
> LINK_QUAL_PATTERN_SELECT and it has the same values of registers
> DPCD 10Bh-10Eh.
> Use the PHY pattern names defined for DPCD 10Bh-10Eh in order to add
> CP2520 Pattern 3 (TPS4) phy pattern support in the next
> patch of this series and DP2.1 PHY patterns for future series.
>
> CC: Jani Nikula <jani.nikula@xxxxxxxxx>
> Cc: Imre Deak <imre.deak@xxxxxxxxx>
> Cc: Lee Shawn C <shawn.c.lee@xxxxxxxxx>
> Signed-off-by: Khaled Almahallawy <khaled.almahallawy@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 0cc57681dc4d..08b607288a6a 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -3882,27 +3882,27 @@ static void
> intel_dp_phy_pattern_update(struct intel_dp *intel_dp,
> u32 pattern_val;
>
> switch (data->phy_pattern) {
> - case DP_PHY_TEST_PATTERN_NONE:
> + case DP_LINK_QUAL_PATTERN_DISABLE:
> drm_dbg_kms(&dev_priv->drm, "Disable Phy Test
> Pattern\n");
> intel_de_write(dev_priv, DDI_DP_COMP_CTL(pipe), 0x0);
> break;
> - case DP_PHY_TEST_PATTERN_D10_2:
> + case DP_LINK_QUAL_PATTERN_D10_2:
> drm_dbg_kms(&dev_priv->drm, "Set D10.2 Phy Test
> Pattern\n");
> intel_de_write(dev_priv, DDI_DP_COMP_CTL(pipe),
> DDI_DP_COMP_CTL_ENABLE |
> DDI_DP_COMP_CTL_D10_2);
> break;
> - case DP_PHY_TEST_PATTERN_ERROR_COUNT:
> + case DP_LINK_QUAL_PATTERN_ERROR_RATE:
> drm_dbg_kms(&dev_priv->drm, "Set Error Count Phy Test
> Pattern\n");
> intel_de_write(dev_priv, DDI_DP_COMP_CTL(pipe),
> DDI_DP_COMP_CTL_ENABLE |
> DDI_DP_COMP_CTL_SCRAMBLED_0);
> break;
> - case DP_PHY_TEST_PATTERN_PRBS7:
> + case DP_LINK_QUAL_PATTERN_PRBS7:
> drm_dbg_kms(&dev_priv->drm, "Set PRBS7 Phy Test
> Pattern\n");
> intel_de_write(dev_priv, DDI_DP_COMP_CTL(pipe),
> DDI_DP_COMP_CTL_ENABLE |
> DDI_DP_COMP_CTL_PRBS7);
> break;
> - case DP_PHY_TEST_PATTERN_80BIT_CUSTOM:
> + case DP_LINK_QUAL_PATTERN_80BIT_CUSTOM:
> /*
> * FIXME: Ideally pattern should come from DPCD 0x250.
> As
> * current firmware of DPR-100 could not set it, so
> hardcoding
> @@ -3920,7 +3920,7 @@ static void intel_dp_phy_pattern_update(struct
> intel_dp *intel_dp,
> DDI_DP_COMP_CTL_ENABLE |
> DDI_DP_COMP_CTL_CUSTOM80);
> break;
> - case DP_PHY_TEST_PATTERN_CP2520:
> + case DP_LINK_QUAL_PATTERN_CP2520_PAT_1:
> /*
> * FIXME: Ideally pattern should come from DPCD 0x24A.
> As
> * current firmware of DPR-100 could not set it, so
> hardcoding
