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....@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++) {

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to