[PATCH 2/2] drm/amdgpu: add sclk/uclk sensor support for vega20

[Deucher, Alexander]

Ah, I missed that they were already handled in smu_v11_0.c.  In that case, I think I can drop these patches.

Alex
________________________________
From: Quan, Evan
Sent: Wednesday, July 17, 2019 8:58 PM
To: Alex Deucher; amd-gfx at lists.freedesktop.org
Cc: Deucher, Alexander
Subject: RE: [PATCH 2/2] drm/amdgpu: add sclk/uclk sensor support for vega20

I think the AMDGPU_PP_SENSOR_GFX_SCLK and AMDGPU_PP_SENSOR_GFX_MCLK requests are handled in smu_v11_0_read_sensor.
It means it cannot reach navi10_ppt.c and vega20_ppt.c.
Maybe this should be fixed in smu_v11_0_read_sensor.

Regards,
Evan
> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of Alex
> Deucher
> Sent: Wednesday, July 17, 2019 10:36 PM
> To: amd-gfx at lists.freedesktop.org
> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>
> Subject: [PATCH 2/2] drm/amdgpu: add sclk/uclk sensor support for vega20
>
> Query the metrics table to get the average sclk and uclk.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/vega20_ppt.c | 35
> ++++++++++++++++++++++
>  1 file changed, 35 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> index 9204e4e50d09..763d73af6cd1 100644
> --- a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> @@ -3117,6 +3117,36 @@ static int vega20_thermal_get_temperature(struct
> smu_context *smu,
>
>        return 0;
>  }
> +
> +static int vega20_get_avg_clocks(struct smu_context *smu,
> +                              enum amd_pp_sensors sensor,
> +                              uint32_t *value)
> +{
> +     SmuMetrics_t metrics;
> +     int ret = 0;
> +
> +     if (!value)
> +             return -EINVAL;
> +
> +     ret = smu_update_table(smu, SMU_TABLE_SMU_METRICS, (void
> *)&metrics, false);
> +     if (ret)
> +             return ret;
> +
> +     switch (sensor) {
> +     case AMDGPU_PP_SENSOR_GFX_SCLK:
> +             *value = metrics.AverageGfxclkFrequency * 100;
> +             break;
> +     case AMDGPU_PP_SENSOR_GFX_MCLK:
> +             *value = metrics.AverageUclkFrequency * 100;
> +             break;
> +     default:
> +             pr_err("Invalid sensor for retrieving avg clock\n");
> +             return -EINVAL;
> +     }
> +
> +     return 0;
> +}
> +
>  static int vega20_read_sensor(struct smu_context *smu,
>                                 enum amd_pp_sensors sensor,
>                                 void *data, uint32_t *size)
> @@ -3147,6 +3177,11 @@ static int vega20_read_sensor(struct smu_context
> *smu,
>                ret = vega20_thermal_get_temperature(smu, sensor,
> (uint32_t *)data);
>                *size = 4;
>                break;
> +     case AMDGPU_PP_SENSOR_GFX_SCLK:
> +     case AMDGPU_PP_SENSOR_GFX_MCLK:
> +             ret = vega20_get_avg_clocks(smu, sensor, (uint32_t *)data);
> +             *size = 4;
> +             break;
>        default:
>                return -EINVAL;
>        }
> --
> 2.20.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20190718/3d01b9d7/attachment.html>