RE: [PATCH 1/2] drm/powerplay: Add smu set xgmi pstate interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewed-by: Evan Quan <evan.quan@xxxxxxx>

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-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Liu,
> Shaoyun
> Sent: Saturday, April 06, 2019 12:02 AM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Liu, Shaoyun <Shaoyun.Liu@xxxxxxx>
> 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.liu@xxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux