[PATCH] drm/amd/powerplay: fix copy error in powerplay.

William Lewis minutemaidpark at hotmail.com
Thu Jun 8 12:47:58 UTC 2017



On 06/08/2017 02:50 AM, Rex Zhu wrote:
> should disable led dpm feature when stop dpm.
>
> Change-Id: I9d78459ff2467d2eba0b26d9696138f7eba484a0
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> ---
>   drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> index 9338aca..b6df00c 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
> @@ -2702,9 +2702,9 @@ static int vega10_stop_dpm(struct pp_hwmgr *hwmgr, uint32_t bitmap)
>   
>   	if(data->smu_features[GNLD_LED_DISPLAY].supported == true){
>   		PP_ASSERT_WITH_CODE(!vega10_enable_smc_features(hwmgr->smumgr,
> -				true, data->smu_features[GNLD_LED_DISPLAY].smu_feature_bitmap),
> -		"Attempt to Enable LED DPM feature Failed!", return -EINVAL);
> -		data->smu_features[GNLD_LED_DISPLAY].enabled = true;
> +				false, data->smu_features[GNLD_LED_DISPLAY].smu_feature_bitmap),
> +		"Attempt to diable LED DPM feature Failed!", return -EINVAL);

s/diable/disable/

Also, why not make "Failed" lower-case at the same time?

> +		data->smu_features[GNLD_LED_DISPLAY].enabled = false;
>   	}
>   
>   	for (i = 0; i < GNLD_DPM_MAX; i++) {



More information about the amd-gfx mailing list