On Wed, 21 Jun 2023, "Sripada, Radhakrishna" <radhakrishna.sripada@xxxxxxxxx> wrote:
>> -----Original Message-----
>> From: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>
>> Sent: Wednesday, June 21, 2023 12:32 AM
>> To: Sripada, Radhakrishna <radhakrishna.sripada@xxxxxxxxx>; Taylor, Clinton A
>> <clinton.a.taylor@xxxxxxxxx>; intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>> Subject: Re: [PATCH] drm/i915/mtl: Skip using vbt hdmi_level_shifter
>> selection on MTL
>>
>> On Tue, 20 Jun 2023, "Sripada, Radhakrishna"
>> <radhakrishna.sripada@xxxxxxxxx> wrote:
>> > Merged. Thank you for the review.
>>
>> Where are the CI results?
>>
>> Please never merge patches without CI results.
> Apologies Jani. Overlooked the green as CI results as the patch was sent 3 days ago. Let me trigger a fresh CI run.
The fresh run will fail because it can no longer apply the patch, as
it's in the baseline already.
> Will keep this in mind for future.
Thanks,
Jani.
>
> --Radhakrishna(RK) Sripada
>
>>
>> BR,
>> Jani.
>>
>> >
>> > --Radhakrishna(RK) Sripada
>> >
>> >> -----Original Message-----
>> >> From: Taylor, Clinton A <clinton.a.taylor@xxxxxxxxx>
>> >> Sent: Tuesday, June 20, 2023 2:10 PM
>> >> To: Sripada, Radhakrishna <radhakrishna.sripada@xxxxxxxxx>; intel-
>> >> gfx@xxxxxxxxxxxxxxxxxxxxx
>> >> Cc: Lee, Shawn C <shawn.c.lee@xxxxxxxxx>; Almahallawy, Khaled
>> >> <khaled.almahallawy@xxxxxxxxx>
>> >> 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.almahallawy@xxxxxxxxx>
>> >> > Cc: Lee Shawn C <shawn.c.lee@xxxxxxxxx>
>> >> > Cc: Clint Taylor <Clinton.A.Taylor@xxxxxxxxx>
>> >> > Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@xxxxxxxxx>
>> >> > ---
>> >> > 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.Taylor@xxxxxxxxx>
>> >>
>> >> -Clint
>> >>
>> >> > return devdata->child.hdmi_level_shifter_value;
>>
>> --
>> Jani Nikula, Intel Open Source Graphics Center
--
Jani Nikula, Intel Open Source Graphics Center
