Date: Tue, 25 Oct 2016 11:01:06 -0300 (ADT) From: Jared McNeill <jmcneill@invisible.ca> To: Milan Obuch <freebsd-arm@dino.sk> Cc: freebsd-arm@freebsd.org Subject: Re: aw_thermal breakage on Allwinner H3 SoC Message-ID: <alpine.DEB.2.11.1610251058480.641@dis.invisible.ca> In-Reply-To: <20161024165820.16e6dd6f@zeta.dino.sk> References: <20161024165820.16e6dd6f@zeta.dino.sk>
next in thread | previous in thread | raw e-mail | index | archive | help
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 >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.DEB.2.11.1610251058480.641>