Re: [PATCH v5 02/12] arm64: dts: rockchip: Change node name for pwm-fan for Radxa ROCK 5C

Dragan Simic
2024-12-16T12:43:34Z

On 2024-12-16 12:30, FUKAUMI Naoki wrote:
> Use more common name "pwm-fan" for pwm-fan node. No functinal change.
>
> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>

Looking good to me, as a preparatory patch. Please, feel free
to include:

Reviewed-by: Dragan Simic <dsimic@manjaro.org>

> ---
> Changes in v5
> - Reword commit message
> Changes in v4
> - new
> ---
> arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts
> b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts
> index 23e9b447b6f0..630f026d856c 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5c.dts
> @@ -68,7 +68,7 @@ led-1 {
> };
> };
>
> - fan {
> + pwm-fan {
> compatible = "pwm-fan";
> #cooling-cells = <2>;
> cooling-levels = <0 64 128 192 255>;