arm64: dts: rockchip: Remove clock-names from PWM nodes
authorJohan Jonker <jbx6244@gmail.com>
Mon, 12 Apr 2021 20:01:55 +0000 (22:01 +0200)
committerThierry Reding <thierry.reding@gmail.com>
Fri, 23 Apr 2021 16:53:08 +0000 (18:53 +0200)
A test with the command below gives this error:

/arch/arm64/boot/dts/rockchip/rk3368-evb-act8846.dt.yaml:
pwm@ff680030: clock-names: ['pwm'] is too short

Devices with only one PWM clock use it to both to derive the functional
clock for the device and as the bus clock. The driver does not need
"clock-names" to get a handle, so remove them all.

make ARCH=arm64 dtbs_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml

Signed-off-by: Johan Jonker <jbx6244@gmail.com>
Acked-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
arch/arm64/boot/dts/rockchip/rk3368.dtsi
arch/arm64/boot/dts/rockchip/rk3399.dtsi

index 7af68ec3feaec866bf4a816dd76d6399b4b42f61..4b2c67c28036556431c200ddb69d32fa07c66bb1 100644 (file)
                pinctrl-names = "default";
                pinctrl-0 = <&pwm0_pin>;
                clocks = <&cru PCLK_PWM1>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                pinctrl-names = "default";
                pinctrl-0 = <&pwm1_pin>;
                clocks = <&cru PCLK_PWM1>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                reg = <0x0 0xff680020 0x0 0x10>;
                #pwm-cells = <3>;
                clocks = <&cru PCLK_PWM1>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                pinctrl-names = "default";
                pinctrl-0 = <&pwm3_pin>;
                clocks = <&cru PCLK_PWM1>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
index edbbf35fe19e620db9137d82dc82788f8a739553..a79faac34e023a1d0506c33fb2c63497ca3ed65e 100644 (file)
                pinctrl-names = "default";
                pinctrl-0 = <&pwm0_pin>;
                clocks = <&pmucru PCLK_RKPWM_PMU>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                pinctrl-names = "default";
                pinctrl-0 = <&pwm1_pin>;
                clocks = <&pmucru PCLK_RKPWM_PMU>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                pinctrl-names = "default";
                pinctrl-0 = <&pwm2_pin>;
                clocks = <&pmucru PCLK_RKPWM_PMU>;
-               clock-names = "pwm";
                status = "disabled";
        };
 
                pinctrl-names = "default";
                pinctrl-0 = <&pwm3a_pin>;
                clocks = <&pmucru PCLK_RKPWM_PMU>;
-               clock-names = "pwm";
                status = "disabled";
        };