Re: PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl

[Jani Nikula <jani.nikula@intel.com>]

On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
>> -----Original Message-----
>> From: Nikula, Jani <jani.nikula@intel.com>
>> Sent: Wednesday, May 8, 2024 5:18 PM
>> To: Murthy, Arun R <arun.r.murthy@intel.com>; Ville Syrjälä
>> <ville.syrjala@linux.intel.com>
>> Cc: intel-gfx@lists.freedesktop.org
>> Subject: RE: [PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl
>> 
>> On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
>> >> -----Original Message-----
>> >> From: Nikula, Jani <jani.nikula@intel.com>
>> >> Sent: Wednesday, May 8, 2024 3:29 PM
>> >> To: Murthy, Arun R <arun.r.murthy@intel.com>; Ville Syrjälä
>> >> <ville.syrjala@linux.intel.com>
>> >> Cc: intel-gfx@lists.freedesktop.org
>> >> Subject: RE: [PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl
>> >>
>> >> On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
>> >> > This is the limitation from the retimer hardware.
>> >>
>> >> The commit message needs to reflect this.
>> >>
>> >> Do all units have a retimer with the limitation? Or could you have a
>> >> DG2 without the limitation? Do we need to check for that instead of
>> >> blanket removal of UHBR 13.5 for DG2?
>> >>
>> > Changed the commit msg as below
>> >
>> > Display 12+ hardware supports UHBR13.5
>> 
>> Not true.
>> 
>> > but due to the retimer constraints platform doesn't support UHBR13.5.
>> 
>> Which platform?
>> 
>> > This is the same for mtl and UHBR13.5 is removed as part of the commit
>> > caf3d748f646 ("drm/i915/dp: Remove support for UHBR13.5") This patch
>> removes UHBR13.5 support for icl/dg2.
>> 
>> ICL does not support UHBR 13.5.
>> 
>
> Per bspec display12+ has uhbr13.5 Due to the retimer constraint none of the Intel platforms supports UHBR13.5
> Uhbr15.3 is already removed for mtl platform as part of commit caf3d748f646 ("drm/i915/dp: Remove support for UHBR13.5") this patch will remove for dg2.

Where does display 12+ come from?

We support UHBR starting from DG2 and display version 14+, right?

BR,
Jani.

>
> Thanks and Regards,
> Arun R Murthy
> --------------------
>> BR,
>> Jani.
>> 
>> >
>> > Thanks and Regards,
>> > Arun R Murthy
>> > --------------------
>> >> BR,
>> >> Jani.
>> >>
>> >>
>> >> --
>> >> Jani Nikula, Intel
>> 
>> --
>> Jani Nikula, Intel

-- 
Jani Nikula, Intel