From owner-freebsd-arm@freebsd.org Fri Nov 4 09:31:26 2016 Return-Path: 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 AC7F7C2BADA for ; Fri, 4 Nov 2016 09:31:26 +0000 (UTC) (envelope-from jmcneill@invisible.ca) Received: from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net [IPv6:2001:4b98:c:538::195]) (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 7555511F7 for ; Fri, 4 Nov 2016 09:31:26 +0000 (UTC) (envelope-from jmcneill@invisible.ca) Received: from mfilter44-d.gandi.net (mfilter44-d.gandi.net [217.70.178.175]) by relay3-d.mail.gandi.net (Postfix) with ESMTP id 96030A80CE; Fri, 4 Nov 2016 10:31:24 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at mfilter44-d.gandi.net Received: from relay3-d.mail.gandi.net ([IPv6:::ffff:217.70.183.195]) by mfilter44-d.gandi.net (mfilter44-d.gandi.net [::ffff:10.0.15.180]) (amavisd-new, port 10024) with ESMTP id nS1FnmJodbOu; Fri, 4 Nov 2016 10:31:22 +0100 (CET) X-Originating-IP: 142.163.95.249 Received: from [192.168.2.69] (stjhnbsu1kw-142163095249.dhcp-dynamic.FibreOP.nb.bellaliant.net [142.163.95.249]) (Authenticated sender: jmcneill@invisible.ca) by relay3-d.mail.gandi.net (Postfix) with ESMTPSA id 06412A80F9; Fri, 4 Nov 2016 10:31:21 +0100 (CET) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (1.0) Subject: Re: aw_thermal breakage on Allwinner H3 SoC From: Jared McNeill X-Mailer: iPad Mail (14B72) In-Reply-To: <20161103123445.6bcc4799@zeta.dino.sk> Date: Fri, 4 Nov 2016 06:31:20 -0300 Cc: freebsd-arm@freebsd.org Content-Transfer-Encoding: quoted-printable Message-Id: References: <20161024165820.16e6dd6f@zeta.dino.sk> <20161025180314.38ea1e96@zeta.dino.sk> <20161025202609.0958c55d@zeta.dino.sk> <20161025213913.310b502e@zeta.dino.sk> <20161103123445.6bcc4799@zeta.dino.sk> To: Milan Obuch X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 04 Nov 2016 09:31:26 -0000 It would look similar to the thermal zone definition in sun7i-a20.dtsi. Mayb= e in the short term we can add some tunables with sane defaults in aw_therma= l. > On Nov 3, 2016, at 8:34 AM, Milan Obuch wrote: >=20 > On Tue, 25 Oct 2016 17:38:55 -0300 (ADT) > Jared McNeill wrote: >=20 >>> On Tue, 25 Oct 2016, Milan Obuch wrote: >>>=20 >>> One more observation: booting verbose shows following >>> aw_thermal0: mem >>> 0x1c25000-0x1c253ff irq 29 on simplebus0 aw_thermal0: #0: alarm 42C >>> hyst 15C shut 65C >>>=20 >>> which is for me wrong - shutdown temperature 65 degrees is >>> unacceptably low. =20 >>=20 >> Agreed. Those numbers are the SoC default values. >>=20 >> 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. >>=20 >> Cheers, >> Jared >=20 > How could this get done? Do we have any example (in some other dts) how > this definition should look like? Where do these vaules got read from? >=20 > Regards, > Milan