Re: [PATCH 1/1] drm/xe: Set GT rp min frequency as 1.2GHz default for BMG/CRI

["Kumar G, Naresh" <naresh.kumar.g@intel.com>]

On 18-02-2026 14:34, Hellstrom, Thomas wrote:
> On Mon, 2026-02-16 at 19:21 +0530, Nareshkumar Gollakoti wrote:
>>   While previously applied only to the BMG G21(Battle image) platform
>>   via workaround Wa_14022085890,this 1.2 GHz minimum is now the
>> default
>>   for BMG and CRI platforms.
>>
>>   Setting this frequency floor(1.2GHz) default is critical in multi
>> GPU
>>   environment for supporting effective Peer-to-Peer(P2P) transactions.
>>
>> Signed-off-by: Nareshkumar Gollakoti <naresh.kumar.g@intel.com>
>> ---
>>   drivers/gpu/drm/xe/xe_guc_pc.c | 9 ++++++++-
>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c
>> b/drivers/gpu/drm/xe/xe_guc_pc.c
>> index 5e5495a39a3c..881054c9cf92 100644
>> --- a/drivers/gpu/drm/xe/xe_guc_pc.c
>> +++ b/drivers/gpu/drm/xe/xe_guc_pc.c
>> @@ -860,6 +860,7 @@ static int pc_adjust_freq_bounds(struct xe_guc_pc
>> *pc)
>>   {
>>   	struct xe_tile *tile = gt_to_tile(pc_to_gt(pc));
>>   	int ret;
>> +	struct xe_device *xe = pc_to_xe(pc);
>>   
>>   	lockdep_assert_held(&pc->freq_lock);
>>   
>> @@ -885,7 +886,13 @@ static int pc_adjust_freq_bounds(struct
>> xe_guc_pc *pc)
>>   	if (pc_get_min_freq(pc) > pc->rp0_freq)
>>   		ret = pc_set_min_freq(pc, pc->rp0_freq);
>>   
>> -	if (XE_DEVICE_WA(tile_to_xe(tile), 14022085890))
>> +	/*
>> +	 * Setting GT RP min frequency to 1.2GHz by default for
>> +	 * all BMG and CRI(For BMG G21 WA do the same).
>> +	 */
>> +	if (XE_DEVICE_WA(tile_to_xe(tile), 14022085890) ||
>> +	    (xe->info.platform == XE_BATTLEMAGE ||
>> +			 xe->info.platform == XE_CRESCENTISLAND))
>>   		ret = pc_set_min_freq(pc, max(BMG_MIN_FREQ,
>> pc_get_min_freq(pc)));
> 
> Copying the reply given to the cover letter for completeness:
> 
> Wasn't the idea to do this only for the Arc PRO BMGs (server SKUs)?
> Perhaps the compute team would want to chime in here for the scope of
> this workaround.
> 
> Thanks,
> Thomas
> 
> 
Hi Thomas,

As we have conclusive scope from compute team
about having the change for both BMG/CRI.

Let me know if any further review comments.

Thanks,
Naresh

> 
> 
> 
>>   
>>   out: