| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CALHNRZ-1wY2D4FOauh7tD+2QKBfhtfdJcvpV_B9Y0tEpE1kTVA@mail.gmail.com>
Date: Tue, 22 Apr 2025 10:58:48 -0500
From: Aaron Kling <webgeek1234@...il.com>
To: Tomasz Maciej Nowak <tmn505@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Thierry Reding <thierry.reding@...il.com>,
Jonathan Hunter <jonathanh@...dia.com>, devicetree@...r.kernel.org,
linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: tegra: Enable PWM fan on the Jetson TX1 Devkit
On Tue, Apr 22, 2025 at 9:52 AM Tomasz Maciej Nowak <tmn505@...il.com> wrote:
>
> Hi.
>
> W dniu 21.04.2025 o 00:42, Aaron Kling via B4 Relay pisze:
> > From: Aaron Kling <webgeek1234@...il.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].
I would have sworn I verified this before sending it in. I've had the
patches for some time. But you are correct, this does not work as-is.
Maybe I lost something cleaning up for submission or just plain
misremembered the verification. I will send a v2 once I've fixed and
verified. Apologies to the list for the bad submission.
For inverted polarity, listing them backwards already has precedence
in mainline, see the Banana Pi R3 dt. This makes me want to double
check the existing Nano pwm-fan entry in mainline, though. Cause I
thought all the t210 devices were the same in regards to pwm fan
inversion. And it doesn't have reversed entries.
Sincerely,
Aaron Kling
>
> 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@...il.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
>
Powered by blists - more mailing lists