On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@xxxxxxxxx> wrote: >> -----Original Message----- >> From: Nikula, Jani <jani.nikula@xxxxxxxxx> >> Sent: Wednesday, May 8, 2024 5:18 PM >> To: Murthy, Arun R <arun.r.murthy@xxxxxxxxx>; Ville Syrjälä >> <ville.syrjala@xxxxxxxxxxxxxxx> >> Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx >> Subject: RE: [PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl >> >> On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@xxxxxxxxx> wrote: >> >> -----Original Message----- >> >> From: Nikula, Jani <jani.nikula@xxxxxxxxx> >> >> Sent: Wednesday, May 8, 2024 3:29 PM >> >> To: Murthy, Arun R <arun.r.murthy@xxxxxxxxx>; Ville Syrjälä >> >> <ville.syrjala@xxxxxxxxxxxxxxx> >> >> Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx >> >> Subject: RE: [PATCH] drm/i915/display/dp: Remove UHBR13.5 for icl >> >> >> >> On Wed, 08 May 2024, "Murthy, Arun R" <arun.r.murthy@xxxxxxxxx> 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