Re: [PATCH 1/2] drm/amd/powerplay: export a function to read fan rpm

2016-10-29 Thread Edward O'Callaghan
Howdy, On 10/30/2016 07:28 AM, Grazvydas Ignotas wrote: > Powerplay hwmgr already has an implementation, all we need to do is to call > it. > > Signed-off-by: Grazvydas Ignotas > --- > drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 18 ++ >

Re: [PATCH] drm/amd/powerplay: don't succeed in getters if fan is missing

2016-10-29 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan On 10/30/2016 07:32 AM, Grazvydas Ignotas wrote: > Otherwise callers end up using uninitialized data. > > Signed-off-by: Grazvydas Ignotas > --- > drivers/gpu/drm/amd/powerplay/hwmgr/smu7_thermal.c | 6 +++--- > 1

[PATCH] drm/amd/powerplay: don't succeed in getters if fan is missing

2016-10-29 Thread Grazvydas Ignotas
Otherwise callers end up using uninitialized data. Signed-off-by: Grazvydas Ignotas --- drivers/gpu/drm/amd/powerplay/hwmgr/smu7_thermal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_thermal.c

[PATCH 2/2] drm/amd/amdgpu: expose fan rpm though hwmon

2016-10-29 Thread Grazvydas Ignotas
Only for cards that are supported by powerplay. Signed-off-by: Grazvydas Ignotas --- drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.h | 5 + drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 21 + 2 files changed, 26 insertions(+) diff --git

[PATCH 1/2] drm/amd/powerplay: export a function to read fan rpm

2016-10-29 Thread Grazvydas Ignotas
Powerplay hwmgr already has an implementation, all we need to do is to call it. Signed-off-by: Grazvydas Ignotas --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 18 ++ drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h | 1 + 2 files changed, 19