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

[Nareshkumar Gollakoti <naresh.kumar.g@intel.com>]

 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.

v2:
- Fix Indentation(Thomas)
- Add comment about power impact(Stuart)

Signed-off-by: Nareshkumar Gollakoti <naresh.kumar.g@intel.com>
---
 drivers/gpu/drm/xe/xe_guc_pc.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c b/drivers/gpu/drm/xe/xe_guc_pc.c
index 21fe73ab4583..5b3d6604c4c9 100644
--- a/drivers/gpu/drm/xe/xe_guc_pc.c
+++ b/drivers/gpu/drm/xe/xe_guc_pc.c
@@ -886,6 +886,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);
 
@@ -911,7 +912,17 @@ 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).
+	 *
+	 * This is an active frequency, so if the device is idle
+	 * we aren't expecting high power output across board
+	 *
+	 */
+	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)));
 
 out:
-- 
2.43.0