[PATCH] drm/i915/display: Implement wa_16011342517
Kandpal, Suraj
suraj.kandpal at intel.com
Fri May 30 04:16:42 UTC 2025
> > Subject: RE: [PATCH] drm/i915/display: Implement wa_16011342517
> >
> >
> >
> > > -----Original Message-----
> > > From: Intel-gfx <intel-gfx-bounces at lists.freedesktop.org> On Behalf
> > > Of Nemesa Garg
> > > Sent: Friday, May 16, 2025 8:01 PM
> > > To: intel-gfx at lists.freedesktop.org; intel-xe at lists.freedesktop.org
> > > Cc: Garg, Nemesa <nemesa.garg at intel.com>
> > > Subject: [PATCH] drm/i915/display: Implement wa_16011342517
> > >
> > > Workaround to prevent skew violation on type-c phy for DP 1.62 and
> HDMI.
The commit message should mention what exactly you are doing here
> > >
> Bspec: 54956
This should actually be 55359
Also I cant find details in the WA number as to what needs to be done
> > > Signed-off-by: Nemesa Garg <nemesa.garg at intel.com>
> > > ---
> > > drivers/gpu/drm/i915/display/intel_ddi.c | 16 ++++++++++++++++
> > > .../gpu/drm/i915/display/intel_dkl_phy_regs.h | 4 ++++
> > > 2 files changed, 20 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c
> > > b/drivers/gpu/drm/i915/display/intel_ddi.c
> > > index 74132c1d6385..34b372b18aab 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> > > @@ -77,6 +77,7 @@
> > > #include "intel_psr.h"
> > > #include "intel_quirks.h"
> > > #include "intel_snps_phy.h"
> > > +#include "intel_step.h"
> > > #include "intel_tc.h"
> > > #include "intel_vdsc.h"
> > > #include "intel_vdsc_regs.h"
> > > @@ -1439,6 +1440,21 @@ static void
> > > tgl_dkl_phy_set_signal_levels(struct
> > > intel_encoder *encoder,
> > >
> > > DKL_TX_DPCNTL2_CFG_LOADGENSELECT_TX2_MASK,
> > > val);
> > > }
> > > +
> > > + /* Wa_16011342517:adl-p */
> > > + if (display->platform.alderlake_p &&
> > > + IS_DISPLAY_STEP(display, STEP_A0, STEP_D0)) {
> > > + if ((intel_crtc_has_type(crtc_state,
> > > INTEL_OUTPUT_HDMI) &&
> > > + crtc_state->port_clock == 594000) ||
> > > + (intel_crtc_has_type(crtc_state,
> > > INTEL_OUTPUT_DP) &&
> > > + crtc_state->port_clock == 162000)) {
> > > + intel_dkl_phy_rmw(display,
> > > DKLP_TX_DPCNTL2(tc_port),
> > > +
> > > LOADGEN_SHARING_PMD_DISABLE, 1);
> > > + } else {
> > > + intel_dkl_phy_rmw(display,
> > > DKLP_TX_DPCNTL2(tc_port),
> > > +
> > > LOADGEN_SHARING_PMD_DISABLE, 0);
> > > + }
> > > + }
> > > }
> > > }
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_dkl_phy_regs.h
> > > b/drivers/gpu/drm/i915/display/intel_dkl_phy_regs.h
> > > index 56085b32956d..fa3bad5efca9 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_dkl_phy_regs.h
> > > +++ b/drivers/gpu/drm/i915/display/intel_dkl_phy_regs.h
> > > @@ -188,6 +188,10 @@ struct intel_dkl_phy_reg {
> > >
> > > _DKL_CMN_UC_DW27)
> > > #define DKL_CMN_UC_DW27_UC_HEALTH (0x1 << 15)
> > >
> > > +#define _DKLP_PCS_GLUE_TX_DPCNTL2 0xB68
> > > +#define DKLP_TX_DPCNTL2(tc_port)
> _DKL_REG(tc_port, \
This should be DKLP_PCS_GLUE_TX_DP_CNTL2
Can't find DKLP_TX_DP_CNTL2 in bspec
Regards,
Suraj Kandpal
> > > +
> >
> > Bspec link from where you are getting this
> >
> > Regards,
> > Suraj Kandpal
> > > _DKLP_PCS_GLUE_TX_DPCNTL2)
> > > +#define LOADGEN_SHARING_PMD_DISABLE REG_BIT(12)
> > > /*
> > > * Each Dekel PHY is addressed through a 4KB aperture. Each PHY has
> > > more than
> > > * 4KB of register space, so a separate index is programmed in
> > > HIP_INDEX_REG0
> > > --
> > > 2.25.1
More information about the Intel-xe
mailing list