Commit c04b8aae authored by Vinay Belgaumkar's avatar Vinay Belgaumkar Committed by Daniele Ceraolo Spurio
Browse files

drm/xe/guc: Add some failure checks 



Return failures from pc_adjust_freq_bounds.

Signed-off-by: default avatarVinay Belgaumkar <vinay.belgaumkar@intel.com>
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: default avatarDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240321191219.243583-1-vinay.belgaumkar@intel.com
parent 8f6444e1

drivers/gpu/drm/xe/xe_guc_pc.c

+9 −5
Original line number Diff line number Diff line
@@ -714,24 +714,28 @@ static int pc_adjust_freq_bounds(struct xe_guc_pc *pc) 


	ret = pc_action_query_task_state(pc);

	if (ret)

		return ret;

		goto out;



	/*

	 * GuC defaults to some RPmax that is not actually achievable without

	 * overclocking. Let's adjust it to the Hardware RP0, which is the

	 * regular maximum

	 */

	if (pc_get_max_freq(pc) > pc->rp0_freq)

		pc_set_max_freq(pc, pc->rp0_freq);

	if (pc_get_max_freq(pc) > pc->rp0_freq) {

		ret = pc_set_max_freq(pc, pc->rp0_freq);

		if (ret)

			goto out;

	}



	/*

	 * Same thing happens for Server platforms where min is listed as

	 * RPMax

	 */

	if (pc_get_min_freq(pc) > pc->rp0_freq)

		pc_set_min_freq(pc, pc->rp0_freq);

		ret = pc_set_min_freq(pc, pc->rp0_freq);



	return 0;

out:

	return ret;

}



static int pc_adjust_requested_freq(struct xe_guc_pc *pc)