[PATCH] drm/amd/powerplay: fix PSI not enabled by kmd

[Deucher, Alexander]

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Rex Zhu
> Sent: Wednesday, August 02, 2017 7:29 AM
> To: amd-gfx at lists.freedesktop.org
> Cc: Zhu, Rex
> Subject: [PATCH] drm/amd/powerplay: fix PSI not enabled by kmd
> 
> cherry-pick from windows driver.
> 
> Change-Id: Ie7ff2e26bdf65a98eab21779a2bf94ee416c52be
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c | 2 +-
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> index cd33eb1..c062844 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppatomfwctrl.c
> @@ -142,7 +142,7 @@ int pp_atomfwctrl_get_voltage_table_v4(struct
> pp_hwmgr *hwmgr,
>  		}
>  	} else if (voltage_mode == VOLTAGE_OBJ_SVID2) {
>  		voltage_table->psi1_enable =
> -			voltage_object->svid2_voltage_obj.loadline_psi1 &
> 0x1;
> +			(voltage_object->svid2_voltage_obj.loadline_psi1 &
> 0x20) >> 5;
>  		voltage_table->psi0_enable =
>  			voltage_object->svid2_voltage_obj.psi0_enable &
> 0x1;
>  		voltage_table->max_vid_step =
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> index 2aab33d..679c768 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> @@ -2521,6 +2521,9 @@ static int vega10_init_smc_table(struct pp_hwmgr
> *hwmgr)
>  	pp_table->DisplayDpmVoltageMode =
>  			(uint8_t)(table_info->uc_dcef_dpm_voltage_mode);
> 
> +	data->vddc_voltage_table.psi0_enable = voltage_table.psi0_enable;
> +	data->vddc_voltage_table.psi1_enable = voltage_table.psi1_enable;
> +
>  	if (data->registry_data.ulv_support &&
>  			table_info->us_ulv_voltage_offset) {
>  		result = vega10_populate_ulv_state(hwmgr);
> --
> 1.9.1
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx