aw_thermal breakage on Allwinner H3 SoC

[Jared McNeill]

Does this patch help? It uses the same values as the BSP for sun8iw7p1 
found here:
https://github.com/friendlyarm/h3_lichee/blob/master/linux-3.4/drivers/thermal/sunxi-temperature.c#L1287

With this patch my NanoPi NEO shows around 39C when idle.

Cheers,
Jared

On Tue, 25 Oct 2016, Jared McNeill wrote:

> Hi Milan --
>
> There appears to be an inconsistency between the temperature conversion 
> formula in the H3 datasheet and the BSP. I'm looking into it..
>
> Cheers,
> Jared
>
>
> On Mon, 24 Oct 2016, Milan Obuch wrote:
>
>> Hi,
>> 
>> today I svnup'ped sources for 12-CURRENT I use to test on Orange Pi One
>> to svn revision 307846 and there was no aw_thermal device anymore.
>> After some fiddling I found reason is in aw_sid.c, namely in function
>> aw_sid_read_tscalib. I managed to get it working, somehow, giving the
>> patch in attachment. This way "sun8i-h3-sid" is defined and attaches,
>> which in turn makes aw_thermal attached and working, a bit.
>> 
>> Almost identical patch, without the last chunk was necessary in past,
>> but now behavior changed a bit - namely, sysctl dev.aw_thermal.0.cpu
>> shows 99C right after start, now, running for almost six hours idle,
>> 100C. This is surely wrong. SoC is not that hot given I can put my
>> finger on it without risk of being immediatelly burned. Before this
>> change it showed some 49 degrees right after power on and rises slowly
>> up to 60 - 63 degrees. This values were not correct, too, temperature
>> is maybe 45 degrees or so according to my finger integrated
>> thermometer :)
>> 
>> Did anybody test this on H3 based board? If yes, is there anything
>> special to be put into FDT definition file?
>> 
>> Has anybody a clue what's going on? If someone has an idea what to
>> check, I am ready to check it, but currently have not much experience
>> with sensors in SoC.
>> 
>> Regards,
>> Milan
>> 
> _______________________________________________
> freebsd-arm at freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/freebsd-arm
> To unsubscribe, send any mail to "freebsd-arm-unsubscribe at freebsd.org"
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: aw_thermal.patch
Type: text/x-diff
Size: 530 bytes
Desc: 
URL: <http://lists.freebsd.org/pipermail/freebsd-arm/attachments/20161025/be12fca3/attachment.patch>