[PATCH v2 2/4] ARM: dts: dove-cubox: fix si5351 node names

Alvin Šipraga posted 4 patches 2 years, 2 months ago
There is a newer version of this series
[PATCH v2 2/4] ARM: dts: dove-cubox: fix si5351 node names
Posted by Alvin Šipraga 2 years, 2 months ago
From: Alvin Šipraga <alsi@bang-olufsen.dk>

Correct the device tree to conform with the bindings. The node name and
index should be separated with an @.

Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Alvin Šipraga <alsi@bang-olufsen.dk>
---
 arch/arm/boot/dts/marvell/dove-cubox.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/marvell/dove-cubox.dts b/arch/arm/boot/dts/marvell/dove-cubox.dts
index bfde99486a87..bcaaf8320c45 100644
--- a/arch/arm/boot/dts/marvell/dove-cubox.dts
+++ b/arch/arm/boot/dts/marvell/dove-cubox.dts
@@ -101,7 +101,7 @@ si5351: clock-generator@60 {
 		/* connect xtal input as source of pll0 and pll1 */
 		silabs,pll-source = <0 0>, <1 0>;
 
-		clkout0 {
+		clkout@0 {
 			reg = <0>;
 			silabs,drive-strength = <8>;
 			silabs,multisynth-source = <0>;
@@ -109,7 +109,7 @@ clkout0 {
 			silabs,pll-master;
 		};
 
-		clkout2 {
+		clkout@2 {
 			reg = <2>;
 			silabs,drive-strength = <8>;
 			silabs,multisynth-source = <1>;
-- 
2.42.0

Re: [PATCH v2 2/4] ARM: dts: dove-cubox: fix si5351 node names
Posted by Krzysztof Kozlowski 2 years, 2 months ago
On 04/10/2023 08:35, Alvin Šipraga wrote:
> From: Alvin Šipraga <alsi@bang-olufsen.dk>
> 
> Correct the device tree to conform with the bindings. The node name and
> index should be separated with an @.
> 
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Alvin Šipraga <alsi@bang-olufsen.dk>
> ---

This is independent change, please do not mix it with this series. DTS
should go via ARM SOC, not drivers subsystem tree.

Best regards,
Krzysztof

Re: (subset) [PATCH v2 2/4] ARM: dts: dove-cubox: fix si5351 node names
Posted by Krzysztof Kozlowski 1 year, 10 months ago
On Wed, 04 Oct 2023 08:35:28 +0200, Alvin Šipraga wrote:
> Correct the device tree to conform with the bindings. The node name and
> index should be separated with an @.
> 
> 

Applied, thanks!

[2/4] ARM: dts: dove-cubox: fix si5351 node names
      https://git.kernel.org/krzk/linux-dt/c/2df26223650027602d017368f4e8dc1eff90404e

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>