[PATCH 2/3] drm/amd/powerplay: issue pre-display settings for display change event

[Quan, Evan]

Ping ...

> -----Original Message-----
> From: Evan Quan <evan.quan at amd.com>
> Sent: 2018年11月28日 16:54
> To: amd-gfx at lists.freedesktop.org
> Cc: Quan, Evan <Evan.Quan at amd.com>
> Subject: [PATCH 2/3] drm/amd/powerplay: issue pre-display settings for
> display change event
> 
> For display config change event only, pre-display config settings are needed.
> 
> Change-Id: Ifeee7cf35afc00e2fc0269c2a189c560b2091c49
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c  | 3 +++
> drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c | 2 --
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
> index 47ac92369739..0173d0480024 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
> @@ -352,6 +352,9 @@ int hwmgr_handle_task(struct pp_hwmgr *hwmgr,
> enum amd_pp_task task_id,
> 
>  	switch (task_id) {
>  	case AMD_PP_TASK_DISPLAY_CONFIG_CHANGE:
> +		ret = phm_pre_display_configuration_changed(hwmgr);
> +		if (ret)
> +			return ret;
>  		ret = phm_set_cpu_power_state(hwmgr);
>  		if (ret)
>  			return ret;
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c
> index 91ffb7bc4ee7..56437866d120 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c
> @@ -265,8 +265,6 @@ int psm_adjust_power_state_dynamic(struct
> pp_hwmgr *hwmgr, bool skip,
>  	if (skip)
>  		return 0;
> 
> -	phm_pre_display_configuration_changed(hwmgr);
> -
>  	phm_display_configuration_changed(hwmgr);
> 
>  	if (hwmgr->ps)
> --
> 2.19.2