Received-SPF: pass (google.com: domain of linux-kernel+bounces-157569-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: Jernej =?utf-8?B?xaBrcmFiZWM=?= <jernej.skrabec@gmail.com>
To: Lee Jones <lee@kernel.org>, Rob Herring <robh@kernel.org>,
 Krzysztof Kozlowski <krzk+dt@kernel.org>, Conor Dooley <conor+dt@kernel.org>,
 Chen-Yu Tsai <wens@csie.org>, Samuel Holland <samuel@sholland.org>,
 Maxime Ripard <mripard@kernel.org>, devicetree@vger.kernel.org,
 linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev,
 linux-kernel@vger.kernel.org,
 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Subject:
 Re: [PATCH] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node
 names
Date: Wed, 24 Apr 2024 20:58:13 +0200
Message-ID: <2320577.ElGaqSPkdT@jernej-laptop>
In-Reply-To: <20240424045521.31857-1-krzysztof.kozlowski@linaro.org>
References: <20240424045521.31857-1-krzysztof.kozlowski@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="utf-8"

Dne sreda, 24. april 2024 ob 06:55:21 GMT +2 je Krzysztof Kozlowski napisal(a):
> Underscores should not be used in node names (dtc with W=2 warns about
> them), so replace them with hyphens.  This should have no impact on
> known users: Linux MFD driver does not care about children node names.
> DTS was already adjusted in commit 0f47ef3ff1bd ("arm: dts: allwinner: drop
> underscore in node names"), so without this change, we observe
> dtbs_check warnings:
> 
>   sun6i-a31s-colorfly-e708-q1.dtb: prcm@1f01400: 'ahb0-clk', 'apb0-clk', 'apb0-gates-clk', 'apb0-rst', 'ar100-clk', 'ir-clk' do not match any of the regexes: '^.*_(clk|rst)$', 'pinctrl-[0-9]+'
> 
> Reported-by: Rob Herring <robh@kernel.org>
> Closes: https://lore.kernel.org/all/CAL_JsqJfT-jui5P56CO4Fr37kr5iNN8dpxt8ecKeFmdVGnRYbA@mail.gmail.com/
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej

> ---
>  .../bindings/mfd/allwinner,sun6i-a31-prcm.yaml     | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/allwinner,sun6i-a31-prcm.yaml b/Documentation/devicetree/bindings/mfd/allwinner,sun6i-a31-prcm.yaml
> index 8789e3639ff7..ca0e9f1f2354 100644
> --- a/Documentation/devicetree/bindings/mfd/allwinner,sun6i-a31-prcm.yaml
> +++ b/Documentation/devicetree/bindings/mfd/allwinner,sun6i-a31-prcm.yaml
> @@ -20,7 +20,7 @@ properties:
>      maxItems: 1
>  
>  patternProperties:
> -  "^.*_(clk|rst)$":
> +  "^.*-(clk|rst)$":
>      type: object
>      unevaluatedProperties: false
>  
> @@ -171,7 +171,7 @@ examples:
>          compatible = "allwinner,sun6i-a31-prcm";
>          reg = <0x01f01400 0x200>;
>  
> -        ar100: ar100_clk {
> +        ar100: ar100-clk {
>              compatible = "allwinner,sun6i-a31-ar100-clk";
>              #clock-cells = <0>;
>              clocks = <&rtc 0>, <&osc24M>,
> @@ -180,7 +180,7 @@ examples:
>              clock-output-names = "ar100";
>          };
>  
> -        ahb0: ahb0_clk {
> +        ahb0: ahb0-clk {
>              compatible = "fixed-factor-clock";
>              #clock-cells = <0>;
>              clock-div = <1>;
> @@ -189,14 +189,14 @@ examples:
>              clock-output-names = "ahb0";
>          };
>  
> -        apb0: apb0_clk {
> +        apb0: apb0-clk {
>              compatible = "allwinner,sun6i-a31-apb0-clk";
>              #clock-cells = <0>;
>              clocks = <&ahb0>;
>              clock-output-names = "apb0";
>          };
>  
> -        apb0_gates: apb0_gates_clk {
> +        apb0_gates: apb0-gates-clk {
>              compatible = "allwinner,sun6i-a31-apb0-gates-clk";
>              #clock-cells = <1>;
>              clocks = <&apb0>;
> @@ -206,14 +206,14 @@ examples:
>                                   "apb0_i2c";
>          };
>  
> -        ir_clk: ir_clk {
> +        ir_clk: ir-clk {
>              #clock-cells = <0>;
>              compatible = "allwinner,sun4i-a10-mod0-clk";
>              clocks = <&rtc 0>, <&osc24M>;
>              clock-output-names = "ir";
>          };
>  
> -        apb0_rst: apb0_rst {
> +        apb0_rst: apb0-rst {
>              compatible = "allwinner,sun6i-a31-clock-reset";
>              #reset-cells = <1>;
>          };
>