[Intel-gfx] [PATCH] drm/i915/dp: Check for Low voltage IO only for eDP

Murthy, Arun R arun.r.murthy at intel.com
Wed Jun 29 04:56:37 UTC 2022


> 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 at intel.com>

Thanks and Regards,
Arun R Murthy
--------------------


More information about the Intel-gfx mailing list