2024-04-24 05:02:18

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node names

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 <[email protected]>
Closes: https://lore.kernel.org/all/CAL_JsqJfT-jui5P56CO4Fr37kr5iNN8dpxt8ecKeFmdVGnRYbA@mail.gmail.com/
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../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>;
};
--
2.43.0



2024-04-24 19:07:36

by Jernej Škrabec

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node names

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 <[email protected]>
> Closes: https://lore.kernel.org/all/CAL_JsqJfT-jui5P56CO4Fr37kr5iNN8dpxt8ecKeFmdVGnRYbA@mail.gmail.com/
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Jernej Skrabec <[email protected]>

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>;
> };
>





2024-04-24 20:22:29

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node names


On Wed, 24 Apr 2024 06:55:21 +0200, Krzysztof Kozlowski wrote:
> 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 <[email protected]>
> Closes: https://lore.kernel.org/all/CAL_JsqJfT-jui5P56CO4Fr37kr5iNN8dpxt8ecKeFmdVGnRYbA@mail.gmail.com/
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../bindings/mfd/allwinner,sun6i-a31-prcm.yaml | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>

Acked-by: Rob Herring (Arm) <[email protected]>


2024-05-02 17:01:03

by Lee Jones

[permalink] [raw]
Subject: Re: (subset) [PATCH] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node names

On Wed, 24 Apr 2024 06:55:21 +0200, Krzysztof Kozlowski wrote:
> 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:
>
> [...]

Applied, thanks!

[1/1] dt-bindings: mfd: allwinner,sun6i-a31-prcm: Use hyphens in node names
commit: 334fbae01f66a81bcbeed2dd0ac954660beeb4aa

--
Lee Jones [李琼斯]