Re: [PATCH] arm64: tegra: Enable PWM fan on the Jetson TX1 Devkit

[Tomasz Maciej Nowak <tmn505@gmail.com>]

Hi.

W dniu 21.04.2025 o 00:42, Aaron Kling via B4 Relay pisze:
> From: Aaron Kling <webgeek1234@gmail.com>
> 
> This is based on 6f78a94, which enabled added the fan and thermal zones
> for the Jetson Nano Devkit. The fan and thermal characteristics of the
> two devkits are similar, so usng the same configuration.

Does this work on Your DevKit? Doesn't on mine, the fan won't budge. Maybe the
revision difference? What I'm using ATM is [1] and [2]. Because inverted polarity
of PWM, not submitted since that'll need the driver changes [3],[4].

1. https://github.com/tmn505/linux/commit/a78c520ec94aeab2c9dc8e1f46597c4174ff957d
2. https://github.com/tmn505/linux/commit/99beee4f0cd5d3a6f30e1829d823c11cb8b54bac
3. https://libera.irclog.whitequark.org/tegra/2024-07-19#36707118;
4. https://libera.irclog.whitequark.org/tegra/2024-10-14#37145211;

Regards

> 
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
>  arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi | 60 ++++++++++++++++++++++++++
>  1 file changed, 60 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi b/arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi
> index 83ed6ac2a8d8f403fb588edce83dc401065c162f..bc02f2eb14bcbd99627c58b398bbf43061c8110b 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi
> +++ b/arch/arm64/boot/dts/nvidia/tegra210-p2597.dtsi
> @@ -1623,6 +1623,14 @@ key-volume-up {
>  		};
>  	};
>  
> +	fan: pwm-fan {
> +		compatible = "pwm-fan";
> +		pwms = <&pwm 3 45334>;
> +
> +		cooling-levels = <0 64 128 255>;
> +		#cooling-cells = <2>;
> +	};
> +
>  	vdd_sys_mux: regulator-vdd-sys-mux {
>  		compatible = "regulator-fixed";
>  		regulator-name = "VDD_SYS_MUX";
> @@ -1778,4 +1786,56 @@ vdd_usb_vbus_otg: regulator-vdd-usb-vbus-otg {
>  		enable-active-high;
>  		vin-supply = <&vdd_5v0_sys>;
>  	};
> +
> +	thermal-zones {
> +		cpu-thermal {
> +			trips {
> +				cpu_trip_critical: critical {
> +					temperature = <96500>;
> +					hysteresis = <0>;
> +					type = "critical";
> +				};
> +
> +				cpu_trip_hot: hot {
> +					temperature = <70000>;
> +					hysteresis = <2000>;
> +					type = "hot";
> +				};
> +
> +				cpu_trip_active: active {
> +					temperature = <50000>;
> +					hysteresis = <2000>;
> +					type = "active";
> +				};
> +
> +				cpu_trip_passive: passive {
> +					temperature = <30000>;
> +					hysteresis = <2000>;
> +					type = "passive";
> +				};
> +			};
> +
> +			cooling-maps {
> +				cpu-critical {
> +					cooling-device = <&fan 3 3>;
> +					trip = <&cpu_trip_critical>;
> +				};
> +
> +				cpu-hot {
> +					cooling-device = <&fan 2 2>;
> +					trip = <&cpu_trip_hot>;
> +				};
> +
> +				cpu-active {
> +					cooling-device = <&fan 1 1>;
> +					trip = <&cpu_trip_active>;
> +				};
> +
> +				cpu-passive {
> +					cooling-device = <&fan 0 0>;
> +					trip = <&cpu_trip_passive>;
> +				};
> +			};
> +		};
> +	};
>  };
> 
> ---
> base-commit: 9c32cda43eb78f78c73aee4aa344b777714e259b
> change-id: 20250420-tx1-therm-9fb3c30fa43f
> 
> Best regards,
-- 
TMN