> On 6/29/2022 8:55 AM, Murthy, Arun R wrote: > >> - if (intel_dp_is_edp(intel_dp) || is_low_voltage_sku(dev_priv, phy)) > >> + if (intel_dp_is_edp(intel_dp)) > >> return 540000; > > Shouldn't the low voltage check be for both eDP and is low voltage sku? > > For JSP/EHL there is SOC PHY restriction for Como phy port with eDP to > support only HBR2, 5.4GHz. > > But the DP can support HBR3 8.1 GHz. Bspec: 32247 > > So for eDP for this platform we do not check for low voltage IO port. > Its better to add this as a comment in the code. Upon adding this in comments Reviewed-by: Arun R Murthy <arun.r.murthy@xxxxxxxxx> Thanks and Regards, Arun R Murthy --------------------