mirror of
https://github.com/torvalds/linux.git
synced 2026-05-31 10:33:41 +02:00
drm/amd/display: DAL ACR, dc part, fix missing dcn30
[Why] - missing in dcn30 function - Fix a divide by 0 when ACR trigger [How] - Add IS_SMU_TIMEOUT() to dcn30_smu_send_msg_with_param - Add zero check in dcn20_update_clocks_update_dentist Acked-by: Alan Liu <HaoPing.Liu@amd.com> Signed-off-by: Ian Chen <ian.chen@amd.com> Tested-by: Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
f69e98a91a
commit
a34136a3b3
|
|
@ -126,16 +126,24 @@ void dcn20_update_clocks_update_dpp_dto(struct clk_mgr_internal *clk_mgr,
|
|||
|
||||
void dcn20_update_clocks_update_dentist(struct clk_mgr_internal *clk_mgr, struct dc_state *context)
|
||||
{
|
||||
int dpp_divider = DENTIST_DIVIDER_RANGE_SCALE_FACTOR
|
||||
* clk_mgr->base.dentist_vco_freq_khz / clk_mgr->base.clks.dppclk_khz;
|
||||
int disp_divider = DENTIST_DIVIDER_RANGE_SCALE_FACTOR
|
||||
* clk_mgr->base.dentist_vco_freq_khz / clk_mgr->base.clks.dispclk_khz;
|
||||
|
||||
uint32_t dppclk_wdivider = dentist_get_did_from_divider(dpp_divider);
|
||||
uint32_t dispclk_wdivider = dentist_get_did_from_divider(disp_divider);
|
||||
int dpp_divider = 0;
|
||||
int disp_divider = 0;
|
||||
uint32_t dppclk_wdivider = 0;
|
||||
uint32_t dispclk_wdivider = 0;
|
||||
uint32_t current_dispclk_wdivider;
|
||||
uint32_t i;
|
||||
|
||||
if (clk_mgr->base.clks.dppclk_khz == 0 || clk_mgr->base.clks.dispclk_khz == 0)
|
||||
return;
|
||||
|
||||
dpp_divider = DENTIST_DIVIDER_RANGE_SCALE_FACTOR
|
||||
* clk_mgr->base.dentist_vco_freq_khz / clk_mgr->base.clks.dppclk_khz;
|
||||
disp_divider = DENTIST_DIVIDER_RANGE_SCALE_FACTOR
|
||||
* clk_mgr->base.dentist_vco_freq_khz / clk_mgr->base.clks.dispclk_khz;
|
||||
|
||||
dppclk_wdivider = dentist_get_did_from_divider(dpp_divider);
|
||||
dispclk_wdivider = dentist_get_did_from_divider(disp_divider);
|
||||
|
||||
REG_GET(DENTIST_DISPCLK_CNTL,
|
||||
DENTIST_DISPCLK_WDIVIDER, ¤t_dispclk_wdivider);
|
||||
|
||||
|
|
|
|||
|
|
@ -28,6 +28,8 @@
|
|||
|
||||
#include "clk_mgr_internal.h"
|
||||
#include "reg_helper.h"
|
||||
#include "dm_helpers.h"
|
||||
|
||||
#include "dalsmc.h"
|
||||
#include "dcn30_smu11_driver_if.h"
|
||||
|
||||
|
|
@ -74,6 +76,7 @@ static uint32_t dcn30_smu_wait_for_response(struct clk_mgr_internal *clk_mgr, un
|
|||
|
||||
static bool dcn30_smu_send_msg_with_param(struct clk_mgr_internal *clk_mgr, uint32_t msg_id, uint32_t param_in, uint32_t *param_out)
|
||||
{
|
||||
uint32_t result;
|
||||
/* Wait for response register to be ready */
|
||||
dcn30_smu_wait_for_response(clk_mgr, 10, 200000);
|
||||
|
||||
|
|
@ -86,8 +89,14 @@ static bool dcn30_smu_send_msg_with_param(struct clk_mgr_internal *clk_mgr, uint
|
|||
/* Trigger the message transaction by writing the message ID */
|
||||
REG_WRITE(DAL_MSG_REG, msg_id);
|
||||
|
||||
result = dcn30_smu_wait_for_response(clk_mgr, 10, 200000);
|
||||
|
||||
if (IS_SMU_TIMEOUT(result)) {
|
||||
dm_helpers_smu_timeout(CTX, msg_id, param_in, 10 * 200000);
|
||||
}
|
||||
|
||||
/* Wait for response */
|
||||
if (dcn30_smu_wait_for_response(clk_mgr, 10, 200000) == DALSMC_Result_OK) {
|
||||
if (result == DALSMC_Result_OK) {
|
||||
if (param_out)
|
||||
*param_out = REG_READ(DAL_ARG_REG);
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user