[PATCH] drm/amdgpu/powerplay/smu10: refactor AMDGPU_PP_SENSOR_GPU_LOAD

Alex Deucher alexdeucher at gmail.com
Fri Mar 19 18:27:33 UTC 2021


On Fri, Mar 19, 2021 at 3:04 AM Shirish S <shirish.s at amd.com> wrote:
>
> refactor AMDGPU_PP_SENSOR_GPU_LOAD to ensure code consistency with other
> commands
>
> Signed-off-by: Shirish S <shirish.s at amd.com>

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

> ---
>  .../gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c    | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c
> index f5d59fa3a030..f5fe540cd536 100644
> --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c
> +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c
> @@ -1297,19 +1297,18 @@ static int smu10_read_sensor(struct pp_hwmgr *hwmgr, int idx,
>                 *size = 4;
>                 break;
>         case AMDGPU_PP_SENSOR_GPU_LOAD:
> -               if (has_gfx_busy) {
> +               if (!has_gfx_busy)
> +                       ret = -EOPNOTSUPP;
> +               else {
>                         ret = smum_send_msg_to_smc(hwmgr,
>                                                    PPSMC_MSG_GetGfxBusy,
>                                                    &activity_percent);
>                         if (!ret)
> -                               activity_percent = activity_percent > 100 ? 100 : activity_percent;
> +                               *((uint32_t *)value) = min(activity_percent, (u32)100);
>                         else
> -                               return -EIO;
> -                       *((uint32_t *)value) = activity_percent;
> -                       return 0;
> -               } else {
> -                       return -EOPNOTSUPP;
> +                               ret = -EIO;
>                 }
> +               break;
>         default:
>                 ret = -EOPNOTSUPP;
>                 break;
> --
> 2.17.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list