[PATCH 4/8] arm64: dts: amlogic: meson-g12b-radxa-zero2: fix pwm clock names

Neil Armstrong posted 8 patches 2 years, 7 months ago
[PATCH 4/8] arm64: dts: amlogic: meson-g12b-radxa-zero2: fix pwm clock names
Posted by Neil Armstrong 2 years, 7 months ago
Fixes the following bindings check error:
 - pwm@2000: clock-names: 'oneOf' conditional failed, one must be fixed:
	['clkin4'] is too short
	'clkin4' is not one of ['clkin0', 'clkin1']
	'clkin0' was expected
 - pwm@7000: clock-names: 'oneOf' conditional failed, one must be fixed:
	['clkin3'] is too short
	'clkin3' is not one of ['clkin0', 'clkin1']
	'clkin0' was expected
 - pwm@19000: clock-names: 'oneOf' conditional failed, one must be fixed:
	['clkin2'] is too short
	'clkin2' is not one of ['clkin0', 'clkin1']
	'clkin0' was expected

Fixes: d747e7f76a5f ("arm64: dts: meson: add support for Radxa Zero2")
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts
index 9a60c5ec2072..890f5bfebb03 100644
--- a/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dts
@@ -360,7 +360,7 @@ &pwm_ef {
 	pinctrl-0 = <&pwm_e_pins>;
 	pinctrl-names = "default";
 	clocks = <&xtal>;
-	clock-names = "clkin2";
+	clock-names = "clkin0";
 	status = "okay";
 };
 
@@ -368,7 +368,7 @@ &pwm_AO_ab {
 	pinctrl-0 = <&pwm_ao_a_pins>;
 	pinctrl-names = "default";
 	clocks = <&xtal>;
-	clock-names = "clkin3";
+	clock-names = "clkin0";
 	status = "okay";
 };
 
@@ -376,7 +376,7 @@ &pwm_AO_cd {
 	pinctrl-0 = <&pwm_ao_d_e_pins>;
 	pinctrl-names = "default";
 	clocks = <&xtal>;
-	clock-names = "clkin4";
+	clock-names = "clkin1";
 	status = "okay";
 };
 

-- 
2.34.1
Re: [PATCH 4/8] arm64: dts: amlogic: meson-g12b-radxa-zero2: fix pwm clock names
Posted by Martin Blumenstingl 2 years, 7 months ago
On Tue, Feb 7, 2023 at 4:08 PM Neil Armstrong <neil.armstrong@linaro.org> wrote:
>
> Fixes the following bindings check error:
>  - pwm@2000: clock-names: 'oneOf' conditional failed, one must be fixed:
>         ['clkin4'] is too short
>         'clkin4' is not one of ['clkin0', 'clkin1']
>         'clkin0' was expected
>  - pwm@7000: clock-names: 'oneOf' conditional failed, one must be fixed:
>         ['clkin3'] is too short
>         'clkin3' is not one of ['clkin0', 'clkin1']
>         'clkin0' was expected
>  - pwm@19000: clock-names: 'oneOf' conditional failed, one must be fixed:
>         ['clkin2'] is too short
>         'clkin2' is not one of ['clkin0', 'clkin1']
>         'clkin0' was expected
>
> Fixes: d747e7f76a5f ("arm64: dts: meson: add support for Radxa Zero2")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>