[PATCH 5/6] arm64: dts: rockchip: fix pwm-fan node for Radxa ROCK 5B

[FUKAUMI Naoki]

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 at radxa.com> wrote:
>>
>> fix properties to match with vendor kernel.
>>
>> Signed-off-by: FUKAUMI Naoki <naoki at radxa.com>
>> ---
>>   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
>