Re: [PATCH v3 1/2] drm/i915/display/dp: Remove UHBR13.5 for icl

["Murthy, Arun R" <arun.r.murthy@intel.com>]

On 02-04-2025 15:58, Jani Nikula wrote:
> On Wed, 02 Apr 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
>> Per Bspec display 14+ has uhbr13.5 Due to the retimer constraint none of
>> the Intel platforms support uhbr13.5.
> Those two sentences together make no sense.
>
> The subject makes no sense, because icl never supported UHBR.
Will change.
>> This has been removed for mtl
>> by the commit caf3d748f646 ("drm/i915/dp: Remove support for UHBR13.5")
>> This patch will remove the support for DG2.
> "This patch" and "will" are unnecessary fillers. And once a patch is
> committed, "this patch" is meaningless.
>
> Just "Remove ..." is fine.
Ok
> dg2_max_source_rate() *still* refers to 1350000 as max rate, and that's
> arguably more important than the rates array. And you can just remove
> the entire function because it becomes a constant.

UHBR10 is still supported. Only support for UHBR 13.5 will be removed.

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

> BR,
> Jani.
>
>> v2: Reframed the commit msg (Jani)
>>
>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>> ---
>>   drivers/gpu/drm/i915/display/intel_dp.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
>> index f21f9b441fc2a4e644c69410e6ec6b3d37907478..d7955351dd43ff61ebfd57e5b6a4ba99c1c7a2b2 100644
>> --- a/drivers/gpu/drm/i915/display/intel_dp.c
>> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
>> @@ -550,7 +550,7 @@ intel_dp_set_source_rates(struct intel_dp *intel_dp)
>>   	};
>>   	static const int icl_rates[] = {
>>   		162000, 216000, 270000, 324000, 432000, 540000, 648000, 810000,
>> -		1000000, 1350000,
>> +		1000000,
>>   	};
>>   	static const int bxt_rates[] = {
>>   		162000, 216000, 243000, 270000, 324000, 432000, 540000