On Tue, Jan 2, 2018 at 10:15 PM, Evan Quan <evan.quan at amd.com> wrote:
> Change-Id: Ib4156edff8e3daaead720956142d8540df3254d8
> Signed-off-by: Evan Quan <evan.quan at amd.com>
Please include a commit message. Something like:
This will be used by powerplay to update the dpm temp range structure used to
interface with hwmon.
With that added:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 13 +++++++++++++
> drivers/gpu/drm/amd/include/cgs_common.h | 7 +++++++
> 2 files changed, 20 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> index 4466f35..bccb0f7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> @@ -1187,6 +1187,18 @@ static int amdgpu_cgs_call_acpi_method(struct cgs_device *cgs_device,
> return amdgpu_cgs_acpi_eval_object(cgs_device, &info);
> }
>
> +static int amdgpu_cgs_set_temperature_range(struct cgs_device *cgs_device,
> + int min_temperature,
> + int max_temperature)
> +{
> + CGS_FUNC_ADEV;
> +
> + adev->pm.dpm.thermal.min_temp = min_temperature;
> + adev->pm.dpm.thermal.max_temp = max_temperature;
> +
> + return 0;
> +}
> +
> static const struct cgs_ops amdgpu_cgs_ops = {
> .alloc_gpu_mem = amdgpu_cgs_alloc_gpu_mem,
> .free_gpu_mem = amdgpu_cgs_free_gpu_mem,
> @@ -1214,6 +1226,7 @@ static const struct cgs_ops amdgpu_cgs_ops = {
> .enter_safe_mode = amdgpu_cgs_enter_safe_mode,
> .lock_grbm_idx = amdgpu_cgs_lock_grbm_idx,
> .register_pp_handle = amdgpu_cgs_register_pp_handle,
> + .set_temperature_range = amdgpu_cgs_set_temperature_range,
> };
>
> static const struct cgs_os_ops amdgpu_cgs_os_ops = {
> diff --git a/drivers/gpu/drm/amd/include/cgs_common.h b/drivers/gpu/drm/amd/include/cgs_common.h
> index 675988d..f5c7397 100644
> --- a/drivers/gpu/drm/amd/include/cgs_common.h
> +++ b/drivers/gpu/drm/amd/include/cgs_common.h
> @@ -427,6 +427,9 @@ struct amd_pp_init;
> typedef void* (*cgs_register_pp_handle)(struct cgs_device *cgs_device,
> int (*call_back_func)(struct amd_pp_init *, void **));
>
> +typedef int (*cgs_set_temperature_range)(struct cgs_device *cgs_device,
> + int min_temperature,
> + int max_temperature);
> struct cgs_ops {
> /* memory management calls (similar to KFD interface) */
> cgs_alloc_gpu_mem_t alloc_gpu_mem;
> @@ -464,6 +467,7 @@ struct cgs_ops {
> cgs_enter_safe_mode enter_safe_mode;
> cgs_lock_grbm_idx lock_grbm_idx;
> cgs_register_pp_handle register_pp_handle;
> + cgs_set_temperature_range set_temperature_range;
> };
>
> struct cgs_os_ops; /* To be define in OS-specific CGS header */
> @@ -545,4 +549,7 @@ struct cgs_device
> #define cgs_register_pp_handle(cgs_device, call_back_func) \
> CGS_CALL(register_pp_handle, cgs_device, call_back_func)
>
> +#define cgs_set_temperature_range(dev, min_temp, max_temp) \
> + CGS_CALL(set_temperature_range, dev, min_temp, max_temp)
> +
> #endif /* _CGS_COMMON_H */
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
