[PATCH 2/4] drm/amd/powerplay: valid Vega10 DPMTABLE_OD_UPDATE_VDDC settings

[Quan, Evan]

Just sent out a V2 version and drop this one.

> -----Original Message-----
> From: Evan Quan <evan.quan at amd.com>
> Sent: 2019年5月7日 14:09
> To: amd-gfx at lists.freedesktop.org
> Cc: yanik at yiannakis.de; Deucher, Alexander
> <Alexander.Deucher at amd.com>; Quan, Evan <Evan.Quan at amd.com>
> Subject: [PATCH 2/4] drm/amd/powerplay: valid Vega10
> DPMTABLE_OD_UPDATE_VDDC settings
> 
> With user specified voltage(DPMTABLE_OD_UPDATE_VDDC), the AVFS will
> be disabled. However, the buggy code makes this actually not working as
> expected.
> 
> Change-Id: Ifa83a6255bb3f6fa4bdb4de616521cb7bab6830a
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> index 138f9f9ea765..103f7e3f0783 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> @@ -2466,11 +2466,6 @@ static void
> vega10_check_dpm_table_updated(struct pp_hwmgr *hwmgr)
>  			return;
>  		}
>  	}
> -
> -	if (data->need_update_dpm_table &
> DPMTABLE_OD_UPDATE_VDDC) {
> -		data->need_update_dpm_table &=
> ~DPMTABLE_OD_UPDATE_VDDC;
> -		data->need_update_dpm_table |=
> DPMTABLE_OD_UPDATE_SCLK | DPMTABLE_OD_UPDATE_MCLK;
> -	}
>  }
> 
>  /**
> @@ -3683,7 +3678,7 @@ static int vega10_set_power_state_tasks(struct
> pp_hwmgr *hwmgr,
> 
>  	vega10_update_avfs(hwmgr);
> 
> -	data->need_update_dpm_table &= DPMTABLE_OD_UPDATE_VDDC;
> +	data->need_update_dpm_table = 0;
> 
>  	return 0;
>  }
> --
> 2.21.0