Merged. With slight change to commit message to silence checkpatch warning. 
Thank you for the review.

--Radhakrishna(RK) Sripada

> -----Original Message-----
> From: Kahola, Mika <[email protected]>
> Sent: Friday, June 16, 2023 12:05 AM
> To: Sripada, Radhakrishna <[email protected]>; intel-
> [email protected]
> Cc: Taylor, Clinton A <[email protected]>; Almahallawy, Khaled
> <[email protected]>; Murthy, Arun R <[email protected]>
> Subject: RE: [PATCH] drm/i915/mtl: Fix SSC selection for MPLLA
> 
> > -----Original Message-----
> > From: Sripada, Radhakrishna <[email protected]>
> > Sent: Friday, June 16, 2023 7:40 AM
> > To: [email protected]
> > Cc: Sripada, Radhakrishna <[email protected]>; Kahola, Mika
> <[email protected]>; Taylor, Clinton A
> > <[email protected]>; Almahallawy, Khaled
> <[email protected]>; Murthy, Arun R
> > <[email protected]>
> > Subject: [PATCH] drm/i915/mtl: Fix SSC selection for MPLLA
> >
> > Driver does not clear the default SSC for MPLLA. This causes link training
> failure when trying to use 10G and 20G rates. Fix the
> > behaviour and enable ssc only when we really want.
> >
> > Fixes: 237e7be0bf57 ("drm/i915/mtl: For DP2.0 10G and 20G rates use
> MPLLA")
> > Cc: Mika Kahola <[email protected]>
> > Cc: Clint Taylor <[email protected]>
> > Cc: Khaled Almahallawy <[email protected]>
> > Cc: Arun R Murthy <[email protected]>
> 
> Reviewed-by: Mika Kahola <[email protected]>
> 
> > Signed-off-by: Radhakrishna Sripada <[email protected]>
> > ---
> >  drivers/gpu/drm/i915/display/intel_cx0_phy.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > index f235df5646ed..1b00ef2c6185 100644
> > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > @@ -2434,7 +2434,8 @@ static void intel_program_port_clock_ctl(struct
> intel_encoder *encoder,
> >
> >     intel_de_rmw(i915, XELPDP_PORT_CLOCK_CTL(encoder->port),
> >                  XELPDP_LANE1_PHY_CLOCK_SELECT |
> XELPDP_FORWARD_CLOCK_UNGATE |
> > -                XELPDP_DDI_CLOCK_SELECT_MASK |
> XELPDP_SSC_ENABLE_PLLB, val);
> > +                XELPDP_DDI_CLOCK_SELECT_MASK |
> XELPDP_SSC_ENABLE_PLLA |
> > +                XELPDP_SSC_ENABLE_PLLB, val);
> >  }
> >
> >  static u32 intel_cx0_get_powerdown_update(u8 lane_mask)
> > --
> > 2.34.1

Reply via email to