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

From: "Michel Dänzer" <michel-otUistvHUpPR7s880joybQ@public.gmane.org>
To: "Deucher,
	Alexander" <Alexander.Deucher-5C7GfCeVMHo@public.gmane.org>,
	'Samuel Pitoiset'
	<samuel.pitoiset-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"StDenis, Tom" <Tom.StDenis-5C7GfCeVMHo@public.gmane.org>
Cc: "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
Subject: Re: [PATCH] drm/amdgpu: expose amdgpu_sensors on pre-powerplay chips
Date: Wed, 15 Feb 2017 11:51:59 +0900	[thread overview]
Message-ID: <35369b6f-b008-543a-fc0d-aa51fd0abe85@daenzer.net> (raw)
In-Reply-To: <BN6PR12MB1652DD1BE92DB032AEC64C76F7580-/b2+HYfkarQqUD6E6FAiowdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>

On 15/02/17 01:17 AM, Deucher, Alexander wrote:
>> From: amd-gfx [mailto:amd-gfx-bounces@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 code is reading the wrong register to get the temp but
>>> we can fix that in another change later on.
>>
>> No idea. It was just a copy-n-paste. Maybe the initial code is buggy?
> 
> I don’t know that the GPU temp sensor on CI based APUs actually works
> properly.  I think the package temperature is exposed via the CPU
> thermals.

Seems to work fine with radeon on my two Kaveri laptops, FWIW.

On my desktop Kaveri, unplausibly low values (around 9-10C) are reported
at idle for both GPU and CPU. With load, both go up to around 30C, so it
seems like the values make sense in a relative sense but not in an
absolute one.

-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx