From owner-freebsd-arm@freebsd.org  Tue Oct 25 20:39:09 2016
Return-Path: <owner-freebsd-arm@freebsd.org>
Delivered-To: freebsd-arm@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 6535AC1CEFA
 for <freebsd-arm@mailman.ysv.freebsd.org>;
 Tue, 25 Oct 2016 20:39:09 +0000 (UTC)
 (envelope-from jmcneill@invisible.ca)
Received: from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net
 [IPv6:2001:4b98:c:538::194])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id 2F1DB9CC
 for <freebsd-arm@freebsd.org>; Tue, 25 Oct 2016 20:39:09 +0000 (UTC)
 (envelope-from jmcneill@invisible.ca)
Received: from mfilter46-d.gandi.net (mfilter46-d.gandi.net [217.70.178.177])
 by relay2-d.mail.gandi.net (Postfix) with ESMTP id 99425C5A6C;
 Tue, 25 Oct 2016 22:38:59 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at mfilter46-d.gandi.net
Received: from relay2-d.mail.gandi.net ([IPv6:::ffff:217.70.183.194])
 by mfilter46-d.gandi.net (mfilter46-d.gandi.net [::ffff:10.0.15.180])
 (amavisd-new, port 10024)
 with ESMTP id 0fvpq1t-J0h7; Tue, 25 Oct 2016 22:38:58 +0200 (CEST)
X-Originating-IP: 142.163.95.249
Received: from dis
 (stjhnbsu1kw-142163095249.dhcp-dynamic.FibreOP.nb.bellaliant.net
 [142.163.95.249]) (Authenticated sender: jmcneill@invisible.ca)
 by relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 90112C5A67;
 Tue, 25 Oct 2016 22:38:57 +0200 (CEST)
Date: Tue, 25 Oct 2016 17:38:55 -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
In-Reply-To: <20161025213913.310b502e@zeta.dino.sk>
Message-ID: <alpine.DEB.2.11.1610251724430.641@dis.invisible.ca>
References: <20161024165820.16e6dd6f@zeta.dino.sk>
 <alpine.DEB.2.11.1610251058480.641@dis.invisible.ca>
 <alpine.DEB.2.11.1610251208390.641@dis.invisible.ca>
 <20161025180314.38ea1e96@zeta.dino.sk> <20161025202609.0958c55d@zeta.dino.sk>
 <20161025213913.310b502e@zeta.dino.sk>
User-Agent: Alpine 2.11 (DEB 23 2013-08-11)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-BeenThere: freebsd-arm@freebsd.org
X-Mailman-Version: 2.1.23
Precedence: list
List-Id: "Porting FreeBSD to ARM processors." <freebsd-arm.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/freebsd-arm>,
 <mailto:freebsd-arm-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arm/>
List-Post: <mailto:freebsd-arm@freebsd.org>
List-Help: <mailto:freebsd-arm-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/freebsd-arm>,
 <mailto:freebsd-arm-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Oct 2016 20:39:09 -0000

On Tue, 25 Oct 2016, Milan Obuch wrote:

> One more observation: booting verbose shows following
> aw_thermal0: <Allwinner Thermal Sensor Controller> mem 0x1c25000-0x1c253ff irq 29 on simplebus0
> aw_thermal0: #0: alarm 42C hyst 15C shut 65C
>
> which is for me wrong - shutdown temperature 65 degrees is unacceptably
> low.

Agreed. Those numbers are the SoC default values.

It looks like the thresholds should be defined in a thermal-zones node in 
the dts, but there is none defined in sun8i-h3.dtsi. We'll have to come up 
with some reasonable defaults, add them to our dts, and then add support 
for reading the trip points to the aw_thermal driver.

Cheers,
Jared