Merged. Thank you for the review.

--Radhakrishna(RK) Sripada

> -----Original Message-----
> From: Taylor, Clinton A <clinton.a.tay...@intel.com>
> Sent: Tuesday, June 20, 2023 2:10 PM
> To: Sripada, Radhakrishna <radhakrishna.srip...@intel.com>; intel-
> g...@lists.freedesktop.org
> Cc: Lee, Shawn C <shawn.c....@intel.com>; Almahallawy, Khaled
> <khaled.almahall...@intel.com>
> Subject: Re: [PATCH] drm/i915/mtl: Skip using vbt hdmi_level_shifter selection
> on MTL
> 
> On Fri, 2023-06-16 at 14:00 -0700, Radhakrishna Sripada wrote:
> > The hdmi_level_shifter part of General Bytes definition in VBT, which was
> > used for choosing different levels on earlier platforms is now a hidden
> > optin and shows the default value of 0. The level shifter is now to be
> > deduced from hdmi_default_entry in  intel_ddi_buf_trans for each phy.
> >
> > Skip providing the default hw provided value to force driver to choose hdmi
> > default entry.
> >
> > Bspec: 20124
> > Cc: Khaled Almahallawy <khaled.almahall...@intel.com>
> > Cc: Lee Shawn C <shawn.c....@intel.com>
> > Cc: Clint Taylor <clinton.a.tay...@intel.com>
> > Signed-off-by: Radhakrishna Sripada <radhakrishna.srip...@intel.com>
> > ---
> >  drivers/gpu/drm/i915/display/intel_bios.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_bios.c
> > b/drivers/gpu/drm/i915/display/intel_bios.c
> > index 34a397adbd6b..4b9bf76e137d 100644
> > --- a/drivers/gpu/drm/i915/display/intel_bios.c
> > +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> > @@ -2615,7 +2615,8 @@ intel_bios_encoder_is_lspcon(const struct
> intel_bios_encoder_data
> > *devdata)
> >  /* This is an index in the HDMI/DVI DDI buffer translation table, or -1 */
> >  int intel_bios_hdmi_level_shift(const struct intel_bios_encoder_data
> *devdata)
> >  {
> > -   if (!devdata || devdata->i915->display.vbt.version < 158)
> > +   if (!devdata || devdata->i915->display.vbt.version < 158 ||
> > +       DISPLAY_VER(devdata->i915) >= 14)
> >             return -1;
> >
> Reviewed-by: Clint Taylor <clinton.a.tay...@intel.com>
> 
> -Clint
> 
> >     return devdata->child.hdmi_level_shifter_value;

Reply via email to