* [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings
@ 2023-04-14 9:19 Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 1/6] dt-bindings: serial: fsl-imx-uart: add missing properties Stefan Wahren
` (6 more replies)
0 siblings, 7 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren
This series tries to address some dtbs_check warnings on i.MX6ULL.
Changes in V3:
- add Krzysztof's Reviewed-Bys
- fix indentation in Patch 6 found by Krzysztof Kozlowski
Changes in V2:
- new patch to fix fsl-imx-uart warnings
- fixed GPC typo found by Fabio Estevam
- keep enum in bindings as suggested by Krzysztof Kozlowski
- make imx6ul GPT compatible to imx6sx
Stefan Wahren (6):
dt-bindings: serial: fsl-imx-uart: add missing properties
dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible
dt-bindings: imx-thermal: add imx6sll and imx6ul compatible
dt-bindings: imxgpt: add imx6ul compatible
ARM: dts: imx: Adjust dma-apbh node name
ARM: dts: imx6ul: Add clock and PGC node to GPC
.../devicetree/bindings/crypto/fsl-dcp.yaml | 12 ++++++---
.../bindings/serial/fsl-imx-uart.yaml | 25 +++++++++++++++++++
.../bindings/thermal/imx-thermal.yaml | 14 ++++++++---
.../devicetree/bindings/timer/fsl,imxgpt.yaml | 3 +++
arch/arm/boot/dts/imx23.dtsi | 2 +-
arch/arm/boot/dts/imx28.dtsi | 2 +-
arch/arm/boot/dts/imx6qdl.dtsi | 2 +-
arch/arm/boot/dts/imx6sx.dtsi | 2 +-
arch/arm/boot/dts/imx6ul.dtsi | 14 ++++++++++-
arch/arm/boot/dts/imx7s.dtsi | 2 +-
10 files changed, 65 insertions(+), 13 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH V3 1/6] dt-bindings: serial: fsl-imx-uart: add missing properties
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 2/6] dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible Stefan Wahren
` (5 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren, Krzysztof Kozlowski
Currently the dtbs_check for imx generates warnings like this:
serial@7000c000: Unevaluated properties are not allowed
('clock-names', 'clocks', 'dma-names', 'dmas' were unexpected)
So add the missing properties to the devicetree binding.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../bindings/serial/fsl-imx-uart.yaml | 25 +++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
index 4cbe76e1715b..06f4b02b0550 100644
--- a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
+++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
@@ -52,6 +52,24 @@ properties:
interrupts:
maxItems: 1
+ clocks:
+ items:
+ - description: ipg clock
+ - description: per clock
+
+ clock-names:
+ items:
+ - const: ipg
+ - const: per
+
+ dmas:
+ maxItems: 2
+
+ dma-names:
+ items:
+ - const: rx
+ - const: tx
+
fsl,dte-mode:
$ref: /schemas/types.yaml#/definitions/flag
description: |
@@ -87,11 +105,15 @@ required:
- compatible
- reg
- interrupts
+ - clocks
+ - clock-names
unevaluatedProperties: false
examples:
- |
+ #include <dt-bindings/clock/imx5-clock.h>
+
aliases {
serial0 = &uart1;
};
@@ -100,6 +122,9 @@ examples:
compatible = "fsl,imx51-uart", "fsl,imx21-uart";
reg = <0x73fbc000 0x4000>;
interrupts = <31>;
+ clocks = <&clks IMX5_CLK_UART1_IPG_GATE>,
+ <&clks IMX5_CLK_UART1_PER_GATE>;
+ clock-names = "ipg", "per";
uart-has-rtscts;
fsl,dte-mode;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH V3 2/6] dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 1/6] dt-bindings: serial: fsl-imx-uart: add missing properties Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 3/6] dt-bindings: imx-thermal: add imx6sll and imx6ul compatible Stefan Wahren
` (4 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren, Krzysztof Kozlowski
Currently the dtbs_check for imx6 generates warnings like this:
'fsl,imx6sl-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp']
['fsl,imx6sl-dcp', 'fsl,imx28-dcp'] is too long
or
'fsl,imx6ull-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp']
['fsl,imx6ull-dcp', 'fsl,imx28-dcp'] is too long
So add them to the devicetree binding.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/crypto/fsl-dcp.yaml | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml
index 99be01539fcd..8dd36c2f76fd 100644
--- a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml
+++ b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml
@@ -11,9 +11,15 @@ maintainers:
properties:
compatible:
- enum:
- - fsl,imx23-dcp
- - fsl,imx28-dcp
+ oneOf:
+ - enum:
+ - fsl,imx23-dcp
+ - fsl,imx28-dcp
+ - items:
+ - enum:
+ - fsl,imx6sl-dcp
+ - fsl,imx6ull-dcp
+ - const: fsl,imx28-dcp
reg:
maxItems: 1
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH V3 3/6] dt-bindings: imx-thermal: add imx6sll and imx6ul compatible
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 1/6] dt-bindings: serial: fsl-imx-uart: add missing properties Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 2/6] dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 4/6] dt-bindings: imxgpt: add " Stefan Wahren
` (3 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren, Krzysztof Kozlowski
Currently the dtbs_check for imx6 generates warnings like this:
['fsl,imx6sll-tempmon', 'fsl,imx6sx-tempmon'] is too long
So add them to the devicetree binding.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/thermal/imx-thermal.yaml | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/thermal/imx-thermal.yaml b/Documentation/devicetree/bindings/thermal/imx-thermal.yaml
index b22c8b59d5c7..8902e9499c96 100644
--- a/Documentation/devicetree/bindings/thermal/imx-thermal.yaml
+++ b/Documentation/devicetree/bindings/thermal/imx-thermal.yaml
@@ -12,10 +12,16 @@ maintainers:
properties:
compatible:
- enum:
- - fsl,imx6q-tempmon
- - fsl,imx6sx-tempmon
- - fsl,imx7d-tempmon
+ oneOf:
+ - enum:
+ - fsl,imx6q-tempmon
+ - fsl,imx6sx-tempmon
+ - fsl,imx7d-tempmon
+ - items:
+ - enum:
+ - fsl,imx6sll-tempmon
+ - fsl,imx6ul-tempmon
+ - const: fsl,imx6sx-tempmon
interrupts:
description: |
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH V3 4/6] dt-bindings: imxgpt: add imx6ul compatible
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
` (2 preceding siblings ...)
2023-04-14 9:19 ` [PATCH V3 3/6] dt-bindings: imx-thermal: add imx6sll and imx6ul compatible Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name Stefan Wahren
` (2 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren, Krzysztof Kozlowski
Currently the dtbs_check for imx6ul generates warnings like this:
['fsl,imx6ul-gpt', 'fsl,imx6sx-gpt'] is too long
According to the timer-imx-gpt driver all imx6 use the same imx6dl data,
but according to the existing DTS files the imx6ul GPT IP is derived from
imx6sx. So better follow the DTS files here and make the imx6ul GPT
compatible to the imx6sl one to fix the warning.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
index 716c6afcca1f..685137338ac9 100644
--- a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
+++ b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
@@ -34,6 +34,9 @@ properties:
- fsl,imxrt1050-gpt
- fsl,imxrt1170-gpt
- const: fsl,imx6dl-gpt
+ - items:
+ - const: fsl,imx6ul-gpt
+ - const: fsl,imx6sx-gpt
reg:
maxItems: 1
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
` (3 preceding siblings ...)
2023-04-14 9:19 ` [PATCH V3 4/6] dt-bindings: imxgpt: add " Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-05-14 2:30 ` Shawn Guo
2023-04-14 9:19 ` [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC Stefan Wahren
2023-05-19 8:03 ` [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
6 siblings, 1 reply; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren
Currently the dtbs_check generates warnings like this:
$nodename:0: 'dma-apbh@110000' does not match '^dma-controller(@.*)?$'
So fix all affected dma-apbh node names.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
arch/arm/boot/dts/imx23.dtsi | 2 +-
arch/arm/boot/dts/imx28.dtsi | 2 +-
arch/arm/boot/dts/imx6qdl.dtsi | 2 +-
arch/arm/boot/dts/imx6sx.dtsi | 2 +-
arch/arm/boot/dts/imx6ul.dtsi | 2 +-
arch/arm/boot/dts/imx7s.dtsi | 2 +-
6 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi
index d19508c8f9ed..a3668a0827fc 100644
--- a/arch/arm/boot/dts/imx23.dtsi
+++ b/arch/arm/boot/dts/imx23.dtsi
@@ -59,7 +59,7 @@ icoll: interrupt-controller@80000000 {
reg = <0x80000000 0x2000>;
};
- dma_apbh: dma-apbh@80004000 {
+ dma_apbh: dma-controller@80004000 {
compatible = "fsl,imx23-dma-apbh";
reg = <0x80004000 0x2000>;
interrupts = <0 14 20 0
diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi
index a8d3c3113e0f..29e37b1fae66 100644
--- a/arch/arm/boot/dts/imx28.dtsi
+++ b/arch/arm/boot/dts/imx28.dtsi
@@ -78,7 +78,7 @@ hsadc: hsadc@80002000 {
status = "disabled";
};
- dma_apbh: dma-apbh@80004000 {
+ dma_apbh: dma-controller@80004000 {
compatible = "fsl,imx28-dma-apbh";
reg = <0x80004000 0x2000>;
interrupts = <82 83 84 85
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index b72ec745f6d1..bda182edc589 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -150,7 +150,7 @@ soc: soc {
interrupt-parent = <&gpc>;
ranges;
- dma_apbh: dma-apbh@110000 {
+ dma_apbh: dma-controller@110000 {
compatible = "fsl,imx6q-dma-apbh", "fsl,imx28-dma-apbh";
reg = <0x00110000 0x2000>;
interrupts = <0 13 IRQ_TYPE_LEVEL_HIGH>,
diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
index 93ac2380ca1e..4233943a1cca 100644
--- a/arch/arm/boot/dts/imx6sx.dtsi
+++ b/arch/arm/boot/dts/imx6sx.dtsi
@@ -209,7 +209,7 @@ gpu: gpu@1800000 {
power-domains = <&pd_pu>;
};
- dma_apbh: dma-apbh@1804000 {
+ dma_apbh: dma-controller@1804000 {
compatible = "fsl,imx6sx-dma-apbh", "fsl,imx28-dma-apbh";
reg = <0x01804000 0x2000>;
interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>,
diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
index 3d9d0f823568..118764c50d92 100644
--- a/arch/arm/boot/dts/imx6ul.dtsi
+++ b/arch/arm/boot/dts/imx6ul.dtsi
@@ -164,7 +164,7 @@ intc: interrupt-controller@a01000 {
<0x00a06000 0x2000>;
};
- dma_apbh: dma-apbh@1804000 {
+ dma_apbh: dma-controller@1804000 {
compatible = "fsl,imx6q-dma-apbh", "fsl,imx28-dma-apbh";
reg = <0x01804000 0x2000>;
interrupts = <0 13 IRQ_TYPE_LEVEL_HIGH>,
diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arch/arm/boot/dts/imx7s.dtsi
index efe2525b62fa..54026c2c93fa 100644
--- a/arch/arm/boot/dts/imx7s.dtsi
+++ b/arch/arm/boot/dts/imx7s.dtsi
@@ -1257,7 +1257,7 @@ fec1: ethernet@30be0000 {
};
};
- dma_apbh: dma-apbh@33000000 {
+ dma_apbh: dma-controller@33000000 {
compatible = "fsl,imx7d-dma-apbh", "fsl,imx28-dma-apbh";
reg = <0x33000000 0x2000>;
interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>,
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
` (4 preceding siblings ...)
2023-04-14 9:19 ` [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name Stefan Wahren
@ 2023-04-14 9:19 ` Stefan Wahren
2023-05-14 2:32 ` Shawn Guo
2023-05-19 8:03 ` [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
6 siblings, 1 reply; 10+ messages in thread
From: Stefan Wahren @ 2023-04-14 9:19 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm,
Stefan Wahren, Jacky Bai
According to fsl,imx-gpc.yaml the General Power Control requires
a ipg clock and a Power Gating Control node. So add them to fix
the dtbs_check warnings on i.MX6UL boards:
gpc@20dc000: 'clocks' is a required property
gpc@20dc000: 'clock-names' is a required property
gpc@20dc000: 'pgc' is a required property
Suggested-by: Jacky Bai <ping.bai@nxp.com>
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
arch/arm/boot/dts/imx6ul.dtsi | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
index 118764c50d92..a8efaed69214 100644
--- a/arch/arm/boot/dts/imx6ul.dtsi
+++ b/arch/arm/boot/dts/imx6ul.dtsi
@@ -719,6 +719,18 @@ gpc: gpc@20dc000 {
#interrupt-cells = <3>;
interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
interrupt-parent = <&intc>;
+ clocks = <&clks IMX6UL_CLK_IPG>;
+ clock-names = "ipg";
+
+ pgc {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ power-domain@0 {
+ reg = <0>;
+ #power-domain-cells = <0>;
+ };
+ };
};
iomuxc: pinctrl@20e0000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name
2023-04-14 9:19 ` [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name Stefan Wahren
@ 2023-05-14 2:30 ` Shawn Guo
0 siblings, 0 replies; 10+ messages in thread
From: Shawn Guo @ 2023-05-14 2:30 UTC (permalink / raw
To: Stefan Wahren
Cc: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Sascha Hauer, kernel, Fabio Estevam,
linux-imx, Rafael J . Wysocki, Daniel Lezcano, Amit Kucheria,
Thomas Gleixner, linux-crypto, devicetree, linux-serial,
linux-arm-kernel, linux-pm
On Fri, Apr 14, 2023 at 11:19:46AM +0200, Stefan Wahren wrote:
> Currently the dtbs_check generates warnings like this:
>
> $nodename:0: 'dma-apbh@110000' does not match '^dma-controller(@.*)?$'
>
> So fix all affected dma-apbh node names.
>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Applied, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC
2023-04-14 9:19 ` [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC Stefan Wahren
@ 2023-05-14 2:32 ` Shawn Guo
0 siblings, 0 replies; 10+ messages in thread
From: Shawn Guo @ 2023-05-14 2:32 UTC (permalink / raw
To: Stefan Wahren
Cc: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Sascha Hauer, kernel, Fabio Estevam,
linux-imx, Rafael J . Wysocki, Daniel Lezcano, Amit Kucheria,
Thomas Gleixner, linux-crypto, devicetree, linux-serial,
linux-arm-kernel, linux-pm, Jacky Bai
On Fri, Apr 14, 2023 at 11:19:47AM +0200, Stefan Wahren wrote:
> According to fsl,imx-gpc.yaml the General Power Control requires
> a ipg clock and a Power Gating Control node. So add them to fix
> the dtbs_check warnings on i.MX6UL boards:
>
> gpc@20dc000: 'clocks' is a required property
> gpc@20dc000: 'clock-names' is a required property
> gpc@20dc000: 'pgc' is a required property
>
> Suggested-by: Jacky Bai <ping.bai@nxp.com>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Applied, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
` (5 preceding siblings ...)
2023-04-14 9:19 ` [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC Stefan Wahren
@ 2023-05-19 8:03 ` Stefan Wahren
6 siblings, 0 replies; 10+ messages in thread
From: Stefan Wahren @ 2023-05-19 8:03 UTC (permalink / raw
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Greg Kroah-Hartman, Shawn Guo, Sascha Hauer
Cc: kernel, Fabio Estevam, linux-imx, Rafael J . Wysocki,
Daniel Lezcano, Amit Kucheria, Thomas Gleixner, linux-crypto,
devicetree, linux-serial, linux-arm-kernel, linux-pm
Hi,
Am 14.04.23 um 11:19 schrieb Stefan Wahren:
> This series tries to address some dtbs_check warnings on i.MX6ULL.
>
> Changes in V3:
> - add Krzysztof's Reviewed-Bys
> - fix indentation in Patch 6 found by Krzysztof Kozlowski
>
> Changes in V2:
> - new patch to fix fsl-imx-uart warnings
> - fixed GPC typo found by Fabio Estevam
> - keep enum in bindings as suggested by Krzysztof Kozlowski
> - make imx6ul GPT compatible to imx6sx
>
> Stefan Wahren (6):
> dt-bindings: serial: fsl-imx-uart: add missing properties
> dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible
> dt-bindings: imx-thermal: add imx6sll and imx6ul compatible
> dt-bindings: imxgpt: add imx6ul compatible
> ARM: dts: imx: Adjust dma-apbh node name
> ARM: dts: imx6ul: Add clock and PGC node to GPC
>
currently patch 3, 5 and 6 has been applied. Should i resend the rest of
the series?
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-05-19 8:04 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-14 9:19 [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 1/6] dt-bindings: serial: fsl-imx-uart: add missing properties Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 2/6] dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 3/6] dt-bindings: imx-thermal: add imx6sll and imx6ul compatible Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 4/6] dt-bindings: imxgpt: add " Stefan Wahren
2023-04-14 9:19 ` [PATCH V3 5/6] ARM: dts: imx: Adjust dma-apbh node name Stefan Wahren
2023-05-14 2:30 ` Shawn Guo
2023-04-14 9:19 ` [PATCH V3 6/6] ARM: dts: imx6ul: Add clock and PGC node to GPC Stefan Wahren
2023-05-14 2:32 ` Shawn Guo
2023-05-19 8:03 ` [PATCH V3 0/6] ARM: dts: imx6ull: Fix dtbs_check warnings Stefan Wahren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).