Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-15 Thread Samuel Pitoiset
c: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre- powerplay chips On 15/02/17 01:17 AM, Deucher, Alexander wrote: From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Samuel Pitoiset Otherwise, it seems to work though the GPU_TEMP reads as &qu

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-15 Thread Tom St Denis
pu: expose amdgpu_sensors on pre- powerplay chips On 15/02/17 01:17 AM, Deucher, Alexander wrote: From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Samuel Pitoiset Otherwise, it seems to work though the GPU_TEMP reads as "5000" on my Kaveri (meaning temp of

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: Michel Dänzer [mailto:mic...@daenzer.net] > Sent: Tuesday, February 14, 2017 9:52 PM > To: Deucher, Alexander; 'Samuel Pitoiset'; StDenis, Tom > Cc: amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_senso

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Michel Dänzer
On 15/02/17 01:17 AM, Deucher, Alexander wrote: >> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf >> Of Samuel Pitoiset >> >>> Otherwise, it seems to work though the GPU_TEMP reads as "5000" on my >>> Kaveri (meaning temp of 5C which isn't true). >>> >>> I suspect the dpm co

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: Samuel Pitoiset [mailto:samuel.pitoi...@gmail.com] > Sent: Tuesday, February 14, 2017 12:04 PM > To: Deucher, Alexander; StDenis, Tom; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre- > powerplay c

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: StDenis, Tom > Sent: Tuesday, February 14, 2017 11:58 AM > To: Deucher, Alexander; 'Samuel Pitoiset'; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre- > powerplay chips > > On 14/

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Samuel Pitoiset
On 02/14/2017 06:01 PM, Deucher, Alexander wrote: -Original Message- From: StDenis, Tom Sent: Tuesday, February 14, 2017 11:58 AM To: Deucher, Alexander; 'Samuel Pitoiset'; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre- powerplay

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: StDenis, Tom > Sent: Tuesday, February 14, 2017 11:58 AM > To: Deucher, Alexander; 'Samuel Pitoiset'; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre- > powerplay chips > > On 14/

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Tom St Denis
/amdgpu: expose amdgpu_sensors on pre-powerplay chips Totally untested but as long as read_sensor() has been recently implemented for dpm based boards, amdgpu_sensors can now be exposed. Cc: Tom St Denis Signed-off-by: Samuel Pitoiset --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 5 + 1

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Samuel Pitoiset > Sent: Monday, February 13, 2017 7:23 PM > To: amd-gfx@lists.freedesktop.org > Cc: StDenis, Tom; Samuel Pitoiset > Subject: [PATCH] drm/amdgpu: expose

RE: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Samuel Pitoiset > Sent: Tuesday, February 14, 2017 9:45 AM > To: StDenis, Tom; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Samuel Pitoiset
On 02/14/2017 03:04 PM, Tom St Denis wrote: On 13/02/17 07:23 PM, Samuel Pitoiset wrote: Totally untested but as long as read_sensor() has been recently implemented for dpm based boards, amdgpu_sensors can now be exposed. Cc: Tom St Denis Signed-off-by: Samuel Pitoiset --- drivers/gpu/drm/

Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-14 Thread Tom St Denis
On 13/02/17 07:23 PM, Samuel Pitoiset wrote: Totally untested but as long as read_sensor() has been recently implemented for dpm based boards, amdgpu_sensors can now be exposed. Cc: Tom St Denis Signed-off-by: Samuel Pitoiset --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 5 + 1 file ch

[PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips

2017-02-13 Thread Samuel Pitoiset
Totally untested but as long as read_sensor() has been recently implemented for dpm based boards, amdgpu_sensors can now be exposed. Cc: Tom St Denis Signed-off-by: Samuel Pitoiset --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --g