Commit c6023d73 authored by Roman Li's avatar Roman Li Committed by Alex Deucher
Browse files

drm/amd/display: Align dcn314_smu logging with other DCNs 



[Why]
Assert on non-OK response from SMU is unnecessary.
It was replaced with respective log message on other asics
in the past with commit:
"drm/amd/display: Removing assert statements for Linux"

[How]
Remove assert and add dbg logging as on other DCNs.

Signed-off-by: default avatarRoman Li <roman.li@amd.com>
Reviewed-by: default avatarSaaem Rizvi <SyedSaaem.Rizvi@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 5ca9b33e

drivers/gpu/drm/amd/display/dc/clk_mgr/dcn314/dcn314_smu.c

+9 −2
Original line number Diff line number Diff line
@@ -123,9 +123,10 @@ static int dcn314_smu_send_msg_with_param(struct clk_mgr_internal *clk_mgr, 
	uint32_t result;



	result = dcn314_smu_wait_for_response(clk_mgr, 10, 200000);

	ASSERT(result == VBIOSSMC_Result_OK);



	smu_print("SMU response after wait: %d\n", result);

	if (result != VBIOSSMC_Result_OK)

		smu_print("SMU Response was not OK. SMU response after wait received is: %d\n",

				result);



	if (result == VBIOSSMC_Status_BUSY)

		return -1;
@@ -216,6 +217,12 @@ int dcn314_smu_set_hard_min_dcfclk(struct clk_mgr_internal *clk_mgr, int request 
			VBIOSSMC_MSG_SetHardMinDcfclkByFreq,

			khz_to_mhz_ceil(requested_dcfclk_khz));



#ifdef DBG

	smu_print("actual_dcfclk_set_mhz %d is set to : %d\n",

			actual_dcfclk_set_mhz,

			actual_dcfclk_set_mhz * 1000);

#endif



	return actual_dcfclk_set_mhz * 1000;

}