2022-12-13 08:42:12

by Kunihiko Hayashi

Subject: [PATCH v3 01/17] dt-bindings: clock: Fix node descriptions in uniphier-clock example

Prior to adding dt-bindings for SoC-dependent controllers, rename the
clock nodes to the generic names in the example.

And drop redundant examples and a parent node of the clock as it is not
directly necessary.

Signed-off-by: Kunihiko Hayashi <[email protected]>
---
.../clock/socionext,uniphier-clock.yaml | 39 ++-----------------
1 file changed, 3 insertions(+), 36 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml b/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
index 9a0cc7341630..4e82582fb2f3 100644
--- a/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
@@ -61,40 +61,7 @@ required:

examples:
- |
- sysctrl@61840000 {
- compatible = "socionext,uniphier-sysctrl", "simple-mfd", "syscon";
- reg = <0x61840000 0x4000>;
-
- clock {
- compatible = "socionext,uniphier-ld11-clock";
- #clock-cells = <1>;
- };
-
- // other nodes ...
- };
-
- - |
- mioctrl@59810000 {
- compatible = "socionext,uniphier-mioctrl", "simple-mfd", "syscon";
- reg = <0x59810000 0x800>;
-
- clock {
- compatible = "socionext,uniphier-ld11-mio-clock";
- #clock-cells = <1>;
- };
-
- // other nodes ...
- };
-
- - |
- perictrl@59820000 {
- compatible = "socionext,uniphier-perictrl", "simple-mfd", "syscon";
- reg = <0x59820000 0x200>;
-
- clock {
- compatible = "socionext,uniphier-ld11-peri-clock";
- #clock-cells = <1>;
- };
-
- // other nodes ...
+ clock-controller {
+ compatible = "socionext,uniphier-ld11-clock";
+ #clock-cells = <1>;
};
--
2.25.1

2022-12-13 14:55:04

by Krzysztof Kozlowski

Subject: Re: [PATCH v3 01/17] dt-bindings: clock: Fix node descriptions in uniphier-clock example

On 13/12/2022 09:24, Kunihiko Hayashi wrote:
> Prior to adding dt-bindings for SoC-dependent controllers, rename the
> clock nodes to the generic names in the example.
>
> And drop redundant examples and a parent node of the clock as it is not
> directly necessary.
>
> Signed-off-by: Kunihiko Hayashi <[email protected]>
> ---


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof