[PATCH] drm/amd/powerplay: move the smc_if_version to asic file

[Huang, Ray]

> -----Original Message-----
> From: Wang, Kevin(Yang)
> Sent: Monday, March 11, 2019 11:06 AM
> To: amd-gfx at lists.freedesktop.org
> Cc: Huang, Ray <Ray.Huang at amd.com>; Wang, Kevin(Yang)
> <Kevin1.Wang at amd.com>
> Subject: [PATCH] drm/amd/powerplay: move the smc_if_version to asic file
> 
> each asic may be has different smc if version, so move its to asic file to
> implement.
> 
> Signed-off-by: Kevin Wang <kevin1.wang at amd.com>

Reviewed-by: Huang Rui <ray.huang at amd.com>

> ---
>  drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h | 2 ++
>  drivers/gpu/drm/amd/powerplay/smu_v11_0.c      | 4 ++--
>  drivers/gpu/drm/amd/powerplay/vega20_ppt.c     | 1 +
>  3 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> index 851234e..b0dc5a5 100644
> --- a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> +++ b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h
> @@ -401,6 +401,8 @@ struct smu_context
>  	uint32_t workload_setting[WORKLOAD_POLICY_MAX];
>  	uint32_t power_profile_mode;
>  	uint32_t default_power_profile_mode;
> +
> +	uint32_t smc_if_version;
>  };
> 
>  struct pptable_funcs {
> diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> index 4d1bdaa..e92da41 100644
> --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c
> @@ -224,8 +224,8 @@ static int smu_v11_0_check_fw_version(struct
> smu_context *smu)
>  	if (ret)
>  		goto err;
> 
> -	if (smu_version == SMU11_DRIVER_IF_VERSION)
> -		return 0;
> +	if (smu_version != smu->smc_if_version)
> +		ret = -EINVAL;
>  err:
>  	return ret;
>  }
> diff --git a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> index 1996f2c..214142c 100644
> --- a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c
> @@ -2704,4 +2704,5 @@ static const struct pptable_funcs vega20_ppt_funcs
> = {  void vega20_set_ppt_funcs(struct smu_context *smu)  {
>  	smu->ppt_funcs = &vega20_ppt_funcs;
> +	smu->smc_if_version = SMU11_DRIVER_IF_VERSION;
>  }
> --
> 2.7.4