Commit 201fe12e authored by Johan Jonker's avatar Johan Jonker Committed by Thierry Reding

ARM: dts: rockchip: Remove clock-names from PWM nodes

A test with the command below gives this error:

/arch/arm/boot/dts/rk3036-evb.dt.yaml:
pwm@20050030: 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=arm dtbs_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml
Signed-off-by: default avatarJohan Jonker <jbx6244@gmail.com>
Acked-by: default avatarHeiko Stuebner <heiko@sntech.de>
Signed-off-by: default avatarThierry Reding <thierry.reding@gmail.com>
parent 78e7da2c
...@@ -355,7 +355,6 @@ pwm0: pwm@20050000 { ...@@ -355,7 +355,6 @@ pwm0: pwm@20050000 {
reg = <0x20050000 0x10>; reg = <0x20050000 0x10>;
#pwm-cells = <3>; #pwm-cells = <3>;
clocks = <&cru PCLK_PWM>; clocks = <&cru PCLK_PWM>;
clock-names = "pwm";
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm0_pin>; pinctrl-0 = <&pwm0_pin>;
status = "disabled"; status = "disabled";
...@@ -366,7 +365,6 @@ pwm1: pwm@20050010 { ...@@ -366,7 +365,6 @@ pwm1: pwm@20050010 {
reg = <0x20050010 0x10>; reg = <0x20050010 0x10>;
#pwm-cells = <3>; #pwm-cells = <3>;
clocks = <&cru PCLK_PWM>; clocks = <&cru PCLK_PWM>;
clock-names = "pwm";
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm1_pin>; pinctrl-0 = <&pwm1_pin>;
status = "disabled"; status = "disabled";
...@@ -377,7 +375,6 @@ pwm2: pwm@20050020 { ...@@ -377,7 +375,6 @@ pwm2: pwm@20050020 {
reg = <0x20050020 0x10>; reg = <0x20050020 0x10>;
#pwm-cells = <3>; #pwm-cells = <3>;
clocks = <&cru PCLK_PWM>; clocks = <&cru PCLK_PWM>;
clock-names = "pwm";
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm2_pin>; pinctrl-0 = <&pwm2_pin>;
status = "disabled"; status = "disabled";
...@@ -388,7 +385,6 @@ pwm3: pwm@20050030 { ...@@ -388,7 +385,6 @@ pwm3: pwm@20050030 {
reg = <0x20050030 0x10>; reg = <0x20050030 0x10>;
#pwm-cells = <2>; #pwm-cells = <2>;
clocks = <&cru PCLK_PWM>; clocks = <&cru PCLK_PWM>;
clock-names = "pwm";
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm3_pin>; pinctrl-0 = <&pwm3_pin>;
status = "disabled"; status = "disabled";
......
...@@ -679,7 +679,6 @@ pwm0: pwm@ff680000 { ...@@ -679,7 +679,6 @@ pwm0: pwm@ff680000 {
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm0_pin>; pinctrl-0 = <&pwm0_pin>;
clocks = <&cru PCLK_RKPWM>; clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled"; status = "disabled";
}; };
...@@ -690,7 +689,6 @@ pwm1: pwm@ff680010 { ...@@ -690,7 +689,6 @@ pwm1: pwm@ff680010 {
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm1_pin>; pinctrl-0 = <&pwm1_pin>;
clocks = <&cru PCLK_RKPWM>; clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled"; status = "disabled";
}; };
...@@ -701,7 +699,6 @@ pwm2: pwm@ff680020 { ...@@ -701,7 +699,6 @@ pwm2: pwm@ff680020 {
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm2_pin>; pinctrl-0 = <&pwm2_pin>;
clocks = <&cru PCLK_RKPWM>; clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled"; status = "disabled";
}; };
...@@ -712,7 +709,6 @@ pwm3: pwm@ff680030 { ...@@ -712,7 +709,6 @@ pwm3: pwm@ff680030 {
pinctrl-names = "default"; pinctrl-names = "default";
pinctrl-0 = <&pwm3_pin>; pinctrl-0 = <&pwm3_pin>;
clocks = <&cru PCLK_RKPWM>; clocks = <&cru PCLK_RKPWM>;
clock-names = "pwm";
status = "disabled"; status = "disabled";
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment