ALL WINNER high temp. stop

[Daniel Braniss]

> On 21 Sep 2016, at 14:13, Daniel Braniss <danny at cs.huji.ac.il> wrote:
> 
> 
>> On 21 Sep 2016, at 13:49, Jared McNeill <jmcneill at invisible.ca> wrote:
>> 
>> CPU frequency scaling is supported now. Have you added operating points to the dts? Without a heatsink or fan, you need to set a reasonable set of operating points.
>> 
>> 64C does seem quite low, the thermal driver uses the power-on default temperature for the shutdown temperature though (which should be > 100C).
>> 
>> Are you sure you are using the correct compat string for the thermal driver in your dts? Different SoCs use a different formula for reading the temperature.
> 
> I’m using what you sent me :-)
> https://github.com/jaredmcneill/freebsd/blob/allwinner-h3/sys/boot/fdt/dts/arm/orangepi-plus-2e.dts#L12 <https://github.com/jaredmcneill/freebsd/blob/allwinner-h3/sys/boot/fdt/dts/arm/orangepi-plus-2e.dts#L121>
> 

here is my dts for the orange-one:
/*-
 * Copyright (c) 2016 Jared McNeill <jmcneill at invisible.ca>
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
 * 1. Redistributions of source code must retain the above copyright
 *    notice, this list of conditions and the following disclaimer.
 * 2. Redistributions in binary form must reproduce the above copyright
 *    notice, this list of conditions and the following disclaimer in the
 *    documentation and/or other materials provided with the distribution.
 *
 * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
 * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
 * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
 * SUCH DAMAGE.
 *
 * $FreeBSD$
 */
#include "sun8i-h3-orangepi-one.dts"

/ {
        clocks {
	       ths_clk: clk at 1c20074 {
	       	       #clock-cells = <0>;
		       compatible = "allwinner,sun8i-h3-ths-clk";
		       reg = <0x01c20074 0x4>;
		       clocks = <&osc24M>;
		       clock-output-names = "ths";
		};
        };
	soc {
		emac: ethernet at 1c30000 {
			compatible = "allwinner,sun8i-h3-emac";
			reg = <0x01c30000 0x104>, <0x01c00030 0x4>;
			reg-names = "emac", "syscon";
			interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_HIGH>;
			resets = <&ahb_rst 17>;
			reset-names = "ahb";
			clocks = <&bus_gates 17>;
			clock-names = "ahb";
			#address-cells = <1>;
			#size-cells = <0>;
			status = "disabled";
		};
		
		sid: eeprom at 01c14000 {
			compatible = "allwinner,sun8i-h3-sid";
			reg = <0x01c14000 0x400>;
		};

		rtp: rtp at 1c25000 {
			compatible = "allwinner,sun8i-h3-ts";
			reg = <0x01c25000 0x400>;
			interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>;
			clocks = <&bus_gates 72>,<&ths_clk>;
			clock-names = "ahb", "ths";
			resets = <&apb1_rst 8>;
			#thermal-sensor-cells = <0>;
		};

	};
};

&mmc0_pins_a {
	allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
};

&pio {
        emac_pins_rgmii_a: emac_rgmii at 0 {
                allwinner,pins = "PD0", "PD1", "PD2", "PD3", "PD4", "PD5",
                		 "PD7", "PD8", "PD9", "PD10", "PD12", "PD13",
				 "PD15", "PD16", "PD17";
                allwinner,function = "emac";
                allwinner,drive = <SUN4I_PINCTRL_40_MA>;
                allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
        };

	emac_phy_reset_pin: emac_phy_reset_pin at 0 {
		allwinner,pins = "PD6";
		allwinner,function = "gpio_out";
		allwinner,drive = <SUN4I_PINCTRL_10_MA>;
		allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
	};
};


/*
 * Board-specific stuff here
 */

/ {
	/*
	model = "Xunlong Orange Pi One";
	compatible = "xunlong,orangepi-one", "allwinner,sun8i-h3";
	*/
	reg_gmac_3v3: gmac-3v3 {
		compatible = "regulator-fixed";
		pinctrl-names = "default";
		pinctrl-0 = <&emac_phy_reset_pin>;
		regulator-name = "gmac-3v3";
		regulator-min-microvolt = <3300000>;
		regulator-max-microvolt = <3300000>;
		startup-delay-us = <100000>;
		enable-active-high;
		gpio = <&pio 3 6 GPIO_ACTIVE_HIGH>;
	};
};

&emac {
	phy-supply = <&reg_gmac_3v3>;
	phy-mode = "mii";
	phy = <&phy1>;

	allwinner,leds-active-low;
	status = "okay";

	allwinner,use-internal-phy;
	resets = <&ahb_rst 17>,<&ahb_rst 66>;
	reset-names = "ahb", "ephy";
	clocks = <&bus_gates 17>,<&bus_gates 128>;
	clock-names = "ahb","ephy";

	phy1: ethernet-phy at 1 {
		reg = <1>;
	};
};

&ehci2 {
	status = "okay";
};


> 
>> 
>> Cheers,
>> Jared
>> 
>> On Wed, 21 Sep 2016, Daniel Braniss wrote:
>> 
>>> hi all,
>>> now that there is thermal control, trying to compile e.g. from ports
>>> portmaster, heats up the cpu, which somewhere around 64C
>>> decides to halt.
>>> Now, I remember some weeks ago, with a kernel version
>>> without the thermal stuff compiling python and all went ok,
>>> so
>>> Q: what is the thermal high water mark?
>>> Q: are the latest changes overheating the cpu, or is the thermal driver over
>>>   cautious?
>>> 
>>> danny
>>> 
>>> 
>>> 
> 
> _______________________________________________
> 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"