Reviewed-by: Evan Quan <evan.q...@amd.com>

However, if this is for vega20,  please be noticed that vega20 still takes old 
powerplay routines at default.

Regards,
Evan
> -----Original Message-----
> From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Liu,
> Shaoyun
> Sent: Saturday, April 06, 2019 12:02 AM
> To: amd-gfx@lists.freedesktop.org
> Cc: Liu, Shaoyun <shaoyun....@amd.com>
> Subject: [PATCH 1/2] drm/powerplay: Add smu set xgmi pstate interface
> 
> XGMI pstate is controlled by SMU, driver need this interface to communicate
> with SMU
> 
> Change-Id: I3a30797332557725d48d392bea0c9d59e2d0e427
> Signed-off-by: shaoyunl <shaoyun....@amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h | 4 ++++
>  drivers/gpu/drm/amd/powerplay/smu_v11_0.c      | 8 ++++++++
>  2 files changed, 12 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> index 2083139..0e52d1b 100644
> --- a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> +++ b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> @@ -546,6 +546,8 @@ struct smu_funcs
>       int (*get_fan_speed_percent)(struct smu_context *smu, uint32_t
> *speed);
>       int (*set_fan_speed_percent)(struct smu_context *smu, uint32_t
> speed);
>       int (*set_fan_speed_rpm)(struct smu_context *smu, uint32_t
> speed);
> +     int (*set_xgmi_pstate)(struct smu_context *smu, uint32_t pstate);
> +
>  };
> 
>  #define smu_init_microcode(smu) \
> @@ -729,6 +731,8 @@ struct smu_funcs
>       ((smu)->funcs->get_sclk ? (smu)->funcs->get_sclk((smu), (low)) : 0)
> #define smu_get_mclk(smu, low) \
>       ((smu)->funcs->get_mclk ? (smu)->funcs->get_mclk((smu), (low)) : 0)
> +#define smu_set_xgmi_pstate(smu, pstate) \
> +             ((smu)->funcs->set_xgmi_pstate ? (smu)->funcs-
> >set_xgmi_pstate((smu),
> +(pstate)) : 0)
> 
> 
>  extern int smu_get_atom_data_table(struct smu_context *smu, uint32_t
> table, diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index 9d75911..2e2ce71 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -1952,6 +1952,13 @@ static int smu_v11_0_set_fan_speed_rpm(struct
> smu_context *smu,
>       return ret;
>  }
> 
> +static int smu_v11_0_set_xgmi_pstate(struct smu_context *smu,
> +                                  uint32_t pstate)
> +{
> +     /* send msg to SMU to set pstate */
> +     return 0;
> +}
> +
>  static const struct smu_funcs smu_v11_0_funcs = {
>       .init_microcode = smu_v11_0_init_microcode,
>       .load_microcode = smu_v11_0_load_microcode, @@ -2008,6 +2015,7
> @@ static const struct smu_funcs smu_v11_0_funcs = {
>       .get_fan_speed_percent = smu_v11_0_get_fan_speed_percent,
>       .set_fan_speed_percent = smu_v11_0_set_fan_speed_percent,
>       .set_fan_speed_rpm = smu_v11_0_set_fan_speed_rpm,
> +     .set_xgmi_pstate = smu_v11_0_set_xgmi_pstate,
>  };
> 
>  void smu_v11_0_set_smu_funcs(struct smu_context *smu)
> --
> 2.7.4
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to