Hi Alexey Charkov, On 12/2/24 22:39, Alexey Charkov wrote:
Hi Naoki, On Mon, Dec 2, 2024 at 5:35 PM FUKAUMI Naoki <naoki@xxxxxxxxx> wrote:fix properties to match with vendor kernel. Signed-off-by: FUKAUMI Naoki <naoki@xxxxxxxxx> --- arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts index 7c8359fe712a..6a02febaefb7 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts @@ -64,10 +64,10 @@ led-0 { fan: pwm-fan { compatible = "pwm-fan"; - cooling-levels = <0 120 150 180 210 240 255>; - fan-supply = <&vcc5v0_sys>; - pwms = <&pwm1 0 50000 0>; #cooling-cells = <2>; + cooling-levels = <0 64 128 192 255>;Could you please share your testing setup for this part? In my experience using Radxa 4010 fan, it didn't spin up reliably below 120, so the value of 64 looks not helpful here.
I'm sorry, I have only 2513, it doesn't work on 5B with/without this patch. so I couldn't test this part. (btw I'll get 4012 soon)
then, thank you for your test report, I'll keep old value. Best regards, -- FUKAUMI Naoki Radxa Computer (Shenzhen) Co., Ltd.
Best regards, Alexey
