Re: [PATCH] drm/i915/dp: Check for Low voltage IO only for eDP

["Nautiyal, Ankit K" <ankit.k.nautiyal@xxxxxxxxx>]

Hi Arun,

Thanks for the comments. Please find my response inline:

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.


Regards,

Ankit


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