[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8d29ddb9-359c-df91-abb6-acdb6927fa95@socionext.com>
Date: Fri, 9 Dec 2022 00:09:41 +0900
From: Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
Cc: Masami Hiramatsu <mhiramat@...nel.org>, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 01/16] dt-bindings: clock: Fix node descriptions in
uniphier-clock example
On 2022/12/08 17:06, Krzysztof Kozlowski wrote:
> On 07/12/2022 06:53, Kunihiko Hayashi wrote:
>> Prior to adding dt-bindings for SoC-dependent controllers, rename the
>> clock nodes their parent nodes to the generic names in the example.
>>
>> And also fix the missing compatible string.
>>
>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>
>> ---
>> .../clock/socionext,uniphier-clock.yaml | 21 +++++++++++--------
>> 1 file changed, 12 insertions(+), 9 deletions(-)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
>> b/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
>> index 9a0cc7341630..672450a1ecda 100644
>> ---
>> a/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
>> +++
>> b/Documentation/devicetree/bindings/clock/socionext,uniphier-clock.yaml
>> @@ -61,11 +61,12 @@ required:
>>
>> examples:
>> - |
>> - sysctrl@...40000 {
>> - compatible = "socionext,uniphier-sysctrl", "simple-mfd",
>> "syscon";
>> + syscon@...40000 {
>> + compatible = "socionext,uniphier-ld11-sysctrl",
>> + "simple-mfd", "syscon";
>> reg = <0x61840000 0x4000>;
>>
>> - clock {
>> + clock-controller {
>> compatible = "socionext,uniphier-ld11-clock";
>> #clock-cells = <1>;
>> };
>> @@ -74,11 +75,12 @@ examples:
>> };
>>
>> - |
>> - mioctrl@...10000 {
>> - compatible = "socionext,uniphier-mioctrl", "simple-mfd",
>> "syscon";
>> + syscon@...10000 {
>> + compatible = "socionext,uniphier-ld11-mioctrl",
>> + "simple-mfd", "syscon";
>> reg = <0x59810000 0x800>;
>>
>> - clock {
>> + clock-controller {
>> compatible = "socionext,uniphier-ld11-mio-clock";
>> #clock-cells = <1>;
>> };
>> @@ -87,11 +89,12 @@ examples:
>> };
>>
>> - |
>> - perictrl@...20000 {
>> - compatible = "socionext,uniphier-perictrl", "simple-mfd",
>> "syscon";
>> + syscon@...20000 {
>> + compatible = "socionext,uniphier-ld11-perictrl",
>> + "simple-mfd", "syscon";
>> reg = <0x59820000 0x200>;
>>
>> - clock {
>
> These are all three same examples (no differences except compatibles),
> so you can as well keep only one. You can also skip parent node,
> otherwise you might keep getting warnings for this schema. The parent is
> not that relevant here and its bindings (e.g.
> socionext,uniphier-sysctrl.yaml) should include complete example -
> parent with all the children.
I understand.
I'll keep one and drop the parent node from the example.
Thank you,
---
Best Regards
Kunihiko Hayashi
Powered by blists - more mailing lists