[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <2441efb1ebbfe1f54507a1cc73c52436@manjaro.org>
Date: Sun, 22 Sep 2024 14:05:33 +0200
From: Dragan Simic <dsimic@...jaro.org>
To: Florian Klink <flokli@...kli.de>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
Sebastian Reichel <sebastian.reichel@...labora.com>, Tamás Szűcs <tszucs@...tonmail.ch>, Ondrej Jirman
<megi@....cz>, Muhammed Efe Cetin <efectn@...tonmail.com>, FUKAUMI Naoki
<naoki@...xa.com>, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: rockchip: enable automatic fan control on
Orange Pi 5+
Hello Florian,
Please, see a comment below.
On 2024-09-22 13:21, Florian Klink wrote:
> This links the PWM fan on Orange Pi 5+ as an active cooling device
> managed automatically by the thermal subsystem, with a target SoC
> temperature of 65C and a minimum-spin interval from 55C to 65C to
> ensure airflow when the system gets warm.
>
> This is pretty much the same as '4a152231b050 ("arm64: dts: rockchip:
> enable automatic fan control on Rock 5B")', except for the Orange Pi
> 5+ board.
>
> Signed-off-by: Florian Klink <flokli@...kli.de>
> ---
> .../dts/rockchip/rk3588-orangepi-5-plus.dts | 30 +++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
> b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
> index e74871491ef5..d91438752006 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
> @@ -351,6 +351,36 @@ &i2s2m0_sdi
> status = "okay";
> };
>
> +&package_thermal {
> + polling-delay = <1000>;
> +
> + trips {
> + package_fan0: package-fan0 {
> + temperature = <55000>;
> + hysteresis = <2000>;
> + type = "active";
> + };
> +
> + package_fan1: package-fan1 {
> + temperature = <65000>;
> + hysteresis = <2000>;
> + type = "active";
> + };
> + };
> +
> + cooling-maps {
> + map0 {
> + trip = <&package_fan0>;
> + cooling-device = <&fan THERMAL_NO_LIMIT 1>;
> + };
> +
> + map1 {
> + trip = <&package_fan1>;
> + cooling-device = <&fan 2 THERMAL_NO_LIMIT>;
> + };
Thanks for renumbering these cooling maps. It's just that you
missed to label this patch as the v2, i.e. you should have used
"[PATCH v2]" in the patch subject.
Please, resend this patch with the v2 label, and please provide
a change log in the patch notes, i.e. below the "---" line.
Here's an example of an earlier v2 patch submission [1] that you
may find helpful.
In the v2 submission, please feel free to also include my
Reviewed-by: Dragan Simic <dsimic@...jaro.org>
[1]
https://lore.kernel.org/linux-rockchip/bdb60f1f793166cd65f58ab7aea025347076019c.1719679068.git.dsimic@manjaro.org/T/#u
> + };
> +};
> +
> /* phy1 - M.KEY socket */
> &pcie2x1l0 {
> reset-gpios = <&gpio4 RK_PA5 GPIO_ACTIVE_HIGH>;
Powered by blists - more mailing lists