linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles
@ 2020-08-24 16:26 Krzysztof Kozlowski
  2020-08-24 16:26 ` [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
                   ` (18 more replies)
  0 siblings, 19 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8 SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
    compatible:0: 'fsl,imx8mm-gpio' is not one of ['fsl,imx1-gpio', 'fsl,imx21-gpio', 'fsl,imx31-gpio', 'fsl,imx35-gpio', 'fsl,imx7d-gpio']
    From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
    compatible: ['fsl,imx8mm-gpio', 'fsl,imx35-gpio'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
    compatible: Additional items are not allowed ('fsl,imx35-gpio' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../bindings/gpio/fsl-imx-gpio.yaml           | 21 +++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
index 0b223abe8cfb..454db20c2d1a 100644
--- a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
@@ -11,12 +11,21 @@ maintainers:
 
 properties:
   compatible:
-    enum:
-      - fsl,imx1-gpio
-      - fsl,imx21-gpio
-      - fsl,imx31-gpio
-      - fsl,imx35-gpio
-      - fsl,imx7d-gpio
+    oneOf:
+      - enum:
+          - fsl,imx1-gpio
+          - fsl,imx21-gpio
+          - fsl,imx31-gpio
+          - fsl,imx35-gpio
+          - fsl,imx7d-gpio
+      - items:
+          - enum:
+              - fsl,imx8mm-gpio
+              - fsl,imx8mn-gpio
+              - fsl,imx8mp-gpio
+              - fsl,imx8mq-gpio
+              - fsl,imx8qxp-gpio
+          - const: fsl,imx35-gpio
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:32   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
                   ` (17 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

The GPIO controller node can have gpio-ranges property.  This fixes
dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000: 'gpio-ranges' does not match any of the regexes: 'pinctrl-[0-9]+'
    From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Remove maxItems
---
 Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
index 454db20c2d1a..dffd9171ea66 100644
--- a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
@@ -51,6 +51,8 @@ properties:
 
   gpio-controller: true
 
+  gpio-ranges: true
+
 required:
   - compatible
   - reg
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
  2020-08-24 16:26 ` [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:33   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
                   ` (16 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

Allow parsing GPIO controller children nodes with GPIO hogs to fix
warning:

  arch/arm64/boot/dts/freescale/imx8mq-evk.dt.yaml: gpio@30240000: 'wl-reg-on' does not match any of the regexes: 'pinctrl-[0-9]+'
    From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/gpio/fsl-imx-gpio.yaml  | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
index dffd9171ea66..620a52f944e8 100644
--- a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
@@ -53,6 +53,23 @@ properties:
 
   gpio-ranges: true
 
+patternProperties:
+  "^(hog-[0-9]+|.+-hog(-[0-9]+)?)$":
+    type: object
+    properties:
+      gpio-hog: true
+      gpios: true
+      input: true
+      output-high: true
+      output-low: true
+      line-name: true
+
+    required:
+      - gpio-hog
+      - gpios
+
+    additionalProperties: false
+
 required:
   - compatible
   - reg
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
  2020-08-24 16:26 ` [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
  2020-08-24 16:26 ` [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:33   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
                   ` (15 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

Parse also optional power-domains property to fix dtbs_check warnings
like:

  arch/arm64/boot/dts/freescale/imx8qxp-ai_ml.dt.yaml: gpio@5d080000: 'power-domains' does not match any of the regexes: 'pinctrl-[0-9]+'
    From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
index 620a52f944e8..de0b9b5f6a70 100644
--- a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
@@ -53,6 +53,9 @@ properties:
 
   gpio-ranges: true
 
+  power-domains:
+    maxItems: 1
+
 patternProperties:
   "^(hog-[0-9]+|.+-hog(-[0-9]+)?)$":
     type: object
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (2 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:33   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
                   ` (14 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
    compatible:0: 'fsl,imx8mm-ddr-pmu' is not one of ['fsl,imx8-ddr-pmu', 'fsl,imx8m-ddr-pmu', 'fsl,imx8mp-ddr-pmu']
    From schema: Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
    compatible: ['fsl,imx8mm-ddr-pmu', 'fsl,imx8m-ddr-pmu'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
    compatible: Additional items are not allowed ('fsl,imx8m-ddr-pmu' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Handle also fsl,imx8mp-ddr-pmu
---
 .../devicetree/bindings/perf/fsl-imx-ddr.yaml    | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml b/Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml
index 9ed8f44adabe..5aad9f4e0b2a 100644
--- a/Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml
+++ b/Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml
@@ -11,10 +11,18 @@ maintainers:
 
 properties:
   compatible:
-    enum:
-      - fsl,imx8-ddr-pmu
-      - fsl,imx8m-ddr-pmu
-      - fsl,imx8mp-ddr-pmu
+    oneOf:
+      - enum:
+          - fsl,imx8-ddr-pmu
+          - fsl,imx8m-ddr-pmu
+          - fsl,imx8mp-ddr-pmu
+      - items:
+          - enum:
+              - fsl,imx8mm-ddr-pmu
+              - fsl,imx8mn-ddr-pmu
+              - fsl,imx8mq-ddr-pmu
+              - fsl,imx8mp-ddr-pmu
+          - const: fsl,imx8m-ddr-pmu
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (3 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:34   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
                   ` (13 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm']
    From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
index 01df06777cba..473863eb67e5 100644
--- a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
+++ b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
@@ -19,9 +19,17 @@ properties:
       - 3
 
   compatible:
-    enum:
-      - fsl,imx1-pwm
-      - fsl,imx27-pwm
+    oneOf:
+      - enum:
+          - fsl,imx1-pwm
+          - fsl,imx27-pwm
+      - items:
+          - enum:
+              - fsl,imx8mm-pwm
+              - fsl,imx8mn-pwm
+              - fsl,imx8mp-pwm
+              - fsl,imx8mq-pwm
+          - const: fsl,imx27-pwm
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (4 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:34   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
                   ` (12 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm']
    From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
    compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Fix subject prefix
---
 Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
index cba3f83ccd5f..3d896173b3b0 100644
--- a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
+++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml
@@ -36,6 +36,10 @@ properties:
             - fsl,imx6sx-uart
             - fsl,imx6ul-uart
             - fsl,imx7d-uart
+            - fsl,imx8mm-uart
+            - fsl,imx8mn-uart
+            - fsl,imx8mp-uart
+            - fsl,imx8mq-uart
           - const: fsl,imx6q-uart
 
   reg:
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (5 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:35   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
                   ` (11 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
    compatible:0: 'fsl,imx8mm-wdt' is not one of ['fsl,imx21-wdt']
    From schema: Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml

  arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
    compatible: ['fsl,imx8mm-wdt', 'fsl,imx21-wdt'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
    compatible: Additional items are not allowed ('fsl,imx21-wdt' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/watchdog/fsl-imx-wdt.yaml     | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml
index d96b93b11fad..991b4e33486e 100644
--- a/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml
@@ -14,8 +14,15 @@ allOf:
 
 properties:
   compatible:
-    enum:
-      - fsl,imx21-wdt
+    oneOf:
+      - const: fsl,imx21-wdt
+      - items:
+          - enum:
+              - fsl,imx8mm-wdt
+              - fsl,imx8mn-wdt
+              - fsl,imx8mp-wdt
+              - fsl,imx8mq-wdt
+          - const: fsl,imx21-wdt
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (6 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:35   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
                   ` (10 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
    compatible:0: 'fsl,imx8mm-gpmi-nand' is not one of ['fsl,imx23-gpmi-nand', 'fsl,imx28-gpmi-nand', 'fsl,imx6q-gpmi-nand', 'fsl,imx6sx-gpmi-nand', 'fsl,imx7d-gpmi-nand']
    From schema: Documentation/devicetree/bindings/mtd/gpmi-nand.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
    compatible: ['fsl,imx8mm-gpmi-nand', 'fsl,imx7d-gpmi-nand'] is too long

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
    compatible: Additional items are not allowed ('fsl,imx7d-gpmi-nand' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/mtd/gpmi-nand.yaml     | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml b/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
index 3201372b7f85..28ff8c581837 100644
--- a/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
+++ b/Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
@@ -20,12 +20,18 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - fsl,imx23-gpmi-nand
-      - fsl,imx28-gpmi-nand
-      - fsl,imx6q-gpmi-nand
-      - fsl,imx6sx-gpmi-nand
-      - fsl,imx7d-gpmi-nand
+    oneOf:
+      - enum:
+          - fsl,imx23-gpmi-nand
+          - fsl,imx28-gpmi-nand
+          - fsl,imx6q-gpmi-nand
+          - fsl,imx6sx-gpmi-nand
+          - fsl,imx7d-gpmi-nand
+      - items:
+          - enum:
+              - fsl,imx8mm-gpmi-nand
+              - fsl,imx8mn-gpmi-nand
+          - const: fsl,imx7d-gpmi-nand
 
   reg:
     items:
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 10/19] dt-bindings: reset: fsl,imx7-src: Add i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (7 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:36   ` [PATCH v2 10/19] dt-bindings: reset: fsl, imx7-src: " Rob Herring
  2020-08-24 16:26 ` [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
                   ` (9 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
    compatible:0: 'fsl,imx8mm-src' is not one of ['fsl,imx7d-src', 'fsl,imx8mq-src', 'fsl,imx8mp-src']
    From schema: Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
    compatible:1: 'syscon' was expected

  arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
    compatible: ['fsl,imx8mm-src', 'fsl,imx8mq-src', 'syscon'] is too long

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../bindings/reset/fsl,imx7-src.yaml          | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml b/Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml
index 569cd3bd3a70..00430e2eabc8 100644
--- a/Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml
+++ b/Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml
@@ -22,12 +22,19 @@ description: |
 
 properties:
   compatible:
-    items:
-      - enum:
-          - fsl,imx7d-src
-          - fsl,imx8mq-src
-          - fsl,imx8mp-src
-      - const: syscon
+    oneOf:
+      - items:
+          - enum:
+              - fsl,imx7d-src
+              - fsl,imx8mq-src
+              - fsl,imx8mp-src
+          - const: syscon
+      - items:
+          - enum:
+              - fsl,imx8mm-src
+              - fsl,imx8mn-src
+          - const: fsl,imx8mq-src
+          - const: syscon
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (8 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:36   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
                   ` (8 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
to fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
    compatible:0: 'fsl,imx8mn-tmu' is not one of ['fsl,imx8mm-tmu', 'fsl,imx8mp-tmu']
    From schema: Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
    compatible: ['fsl,imx8mn-tmu', 'fsl,imx8mm-tmu'] is too long

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
    compatible: Additional items are not allowed ('fsl,imx8mm-tmu' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/thermal/imx8mm-thermal.yaml    | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml b/Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml
index 38852877b8e3..89c54e08ee61 100644
--- a/Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml
+++ b/Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml
@@ -18,9 +18,13 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - fsl,imx8mm-tmu
-      - fsl,imx8mp-tmu
+    oneOf:
+      - enum:
+          - fsl,imx8mm-tmu
+          - fsl,imx8mp-tmu
+      - items:
+          - const: fsl,imx8mn-tmu
+          - const: fsl,imx8mm-tmu
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (9 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:38   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
                   ` (7 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

The i.MX 8 DTSes use two compatibles so update the binding to fix
dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
    compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long
    From schema: Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
    compatible: Additional items are not allowed ('fsl,imx7d-usdhc' was unexpected)

  arch/arm64/boot/dts/freescale/imx8mn-ddr4-evk.dt.yaml: mmc@30b40000:
    compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
---
 .../bindings/mmc/fsl-imx-esdhc.yaml           | 40 +++++++++++--------
 1 file changed, 23 insertions(+), 17 deletions(-)

diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
index 10b45966f1b8..54f05e2a4246 100644
--- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
+++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
@@ -21,23 +21,29 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - fsl,imx25-esdhc
-      - fsl,imx35-esdhc
-      - fsl,imx51-esdhc
-      - fsl,imx53-esdhc
-      - fsl,imx6q-usdhc
-      - fsl,imx6sl-usdhc
-      - fsl,imx6sx-usdhc
-      - fsl,imx6ull-usdhc
-      - fsl,imx7d-usdhc
-      - fsl,imx7ulp-usdhc
-      - fsl,imx8mq-usdhc
-      - fsl,imx8mm-usdhc
-      - fsl,imx8mn-usdhc
-      - fsl,imx8mp-usdhc
-      - fsl,imx8qm-usdhc
-      - fsl,imx8qxp-usdhc
+    oneOf:
+      - enum:
+          - fsl,imx25-esdhc
+          - fsl,imx35-esdhc
+          - fsl,imx51-esdhc
+          - fsl,imx53-esdhc
+          - fsl,imx6q-usdhc
+          - fsl,imx6sl-usdhc
+          - fsl,imx6sx-usdhc
+          - fsl,imx6ull-usdhc
+          - fsl,imx7d-usdhc
+          - fsl,imx7ulp-usdhc
+          - fsl,imx8mq-usdhc
+          - fsl,imx8mm-usdhc
+          - fsl,imx8qxp-usdhc
+      - items:
+          - enum:
+              - fsl,imx8mm-usdhc
+              - fsl,imx8mn-usdhc
+              - fsl,imx8mp-usdhc
+              - fsl,imx8mq-usdhc
+              - fsl,imx8qxp-usdhc
+          - const: fsl,imx7d-usdhc
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (10 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:39   ` Rob Herring
  2020-08-25  4:46   ` Sascha Hauer
  2020-08-24 16:26 ` [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
                   ` (6 subsequent siblings)
  18 siblings, 2 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

DTSes with new i.MX 8M SoCs use two compatibles so update the binding to
fix dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000: compatible:1: 'syscon' was expected
    From schema: Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
    compatible: ['fsl,imx8mn-ocotp', 'fsl,imx8mm-ocotp', 'syscon'] is too long

  arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
    compatible: Additional items are not allowed ('syscon' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 39 ++++++++++++-------
 1 file changed, 24 insertions(+), 15 deletions(-)

diff --git a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
index 1c9d7f05f173..b5b250185afd 100644
--- a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
+++ b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
@@ -19,21 +19,30 @@ allOf:
 
 properties:
   compatible:
-    items:
-      - enum:
-          - fsl,imx6q-ocotp
-          - fsl,imx6sl-ocotp
-          - fsl,imx6sx-ocotp
-          - fsl,imx6ul-ocotp
-          - fsl,imx6ull-ocotp
-          - fsl,imx7d-ocotp
-          - fsl,imx6sll-ocotp
-          - fsl,imx7ulp-ocotp
-          - fsl,imx8mq-ocotp
-          - fsl,imx8mm-ocotp
-          - fsl,imx8mn-ocotp
-          - fsl,imx8mp-ocotp
-      - const: syscon
+    oneOf:
+      - items:
+          - enum:
+              - fsl,imx6q-ocotp
+              - fsl,imx6sl-ocotp
+              - fsl,imx6sx-ocotp
+              - fsl,imx6ul-ocotp
+              - fsl,imx6ull-ocotp
+              - fsl,imx7d-ocotp
+              - fsl,imx6sll-ocotp
+              - fsl,imx7ulp-ocotp
+              - fsl,imx8mq-ocotp
+              - fsl,imx8mm-ocotp
+              - fsl,imx8mn-ocotp
+              - fsl,imx8mp-ocotp
+          - const: syscon
+      - items:
+          # The devices are not really compatible with fsl,imx8mm-ocotp, however
+          # the code for getting SoC revision depends on fsl,imx8mm-ocotp compatible.
+          - enum:
+              - fsl,imx8mn-ocotp
+              - fsl,imx8mp-ocotp
+          - const: fsl,imx8mm-ocotp
+          - const: syscon
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (11 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:39   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
                   ` (5 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

The Toradex Colibri i.MX 8 Evaluation board has two Toradex compatibles
so it needs separate entry.  This fixes dtbs_check warning:

  arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dt.yaml: /:
    compatible: ['toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x', 'fsl,imx8qxp'] is not valid under any of the given schemas (Possible causes of the failure):
    arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dt.yaml: /: compatible: ['toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x', 'fsl,imx8qxp'] is too long

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 Documentation/devicetree/bindings/arm/fsl.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 37592e7bfee9..377fc2a4c159 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -379,7 +379,13 @@ properties:
               - einfochips,imx8qxp-ai_ml  # i.MX8QXP AI_ML Board
               - fsl,imx8qxp-mek           # i.MX8QXP MEK Board
               - toradex,colibri-imx8x         # Colibri iMX8X Module
+          - const: fsl,imx8qxp
+
+      - description: Toradex Colibri i.MX8 Evaluation Board
+        items:
+          - enum:
               - toradex,colibri-imx8x-eval-v3 # Colibri iMX8X Module on Colibri Evaluation Board V3
+          - const: toradex,colibri-imx8x
           - const: fsl,imx8qxp
 
       - description:
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (12 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:40   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
                   ` (4 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

Document the binding for Zodiac Inflight Innovations Ultra Boards.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 Documentation/devicetree/bindings/arm/fsl.yaml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 377fc2a4c159..b48dbf924cfe 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -373,6 +373,14 @@ properties:
               - technexion,pico-pi-imx8m  # TechNexion PICO-PI-8M evk
           - const: fsl,imx8mq
 
+      - description: Zodiac Inflight Innovations Ultra Boards
+        items:
+          - enum:
+              - zii,imx8mq-ultra-rmb3
+              - zii,imx8mq-ultra-zest
+          - const: zii,imx8mq-ultra
+          - const: fsl,imx8mq
+
       - description: i.MX8QXP based Boards
         items:
           - enum:
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (13 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:40   ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl, irqsteer: " Rob Herring
  2020-08-24 16:26 ` [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
                   ` (3 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

The i.MX 8M DTSes use two compatibles so update the binding to fix
dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8mq-thor96.dt.yaml: interrupt-controller@32e2d000:
    compatible: ['fsl,imx8m-irqsteer', 'fsl,imx-irqsteer'] is too long
    From schema: Domentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml

  arch/arm64/boot/dts/freescale/imx8mq-thor96.dt.yaml: interrupt-controller@32e2d000:
    compatible: Additional items are not allowed ('fsl,imx-irqsteer' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../bindings/interrupt-controller/fsl,irqsteer.yaml       | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml
index 360a575ef8b0..3b11a1a15398 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml
@@ -11,9 +11,11 @@ maintainers:
 
 properties:
   compatible:
-    enum:
-      - fsl,imx8m-irqsteer
-      - fsl,imx-irqsteer
+    oneOf:
+      - const: fsl,imx-irqsteer
+      - items:
+          - const: fsl,imx8m-irqsteer
+          - const: fsl,imx-irqsteer
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: Fix compatible matching
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (14 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:42   ` Rob Herring
  2020-08-24 16:26 ` [PATCH v2 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
                   ` (2 subsequent siblings)
  18 siblings, 1 reply; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

The i.MX 8QXP DTSes use two compatibles so update the binding to fix
dtbs_check warnings like:

  arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
    compatible: ['fsl,imx8qxp-lpuart', 'fsl,imx7ulp-lpuart'] is too long
    From schema: Documentation/devicetree/bindings/serial/fsl-lpuart.yaml

  arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
    compatible: Additional items are not allowed ('fsl,imx7ulp-lpuart' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. New patch.
---
 .../devicetree/bindings/serial/fsl-lpuart.yaml | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
index e82c2cf9fef7..8ee651f2ef0b 100644
--- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
+++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
@@ -14,13 +14,17 @@ allOf:
 
 properties:
   compatible:
-    enum:
-      - fsl,vf610-lpuart
-      - fsl,ls1021a-lpuart
-      - fsl,ls1028a-lpuart
-      - fsl,imx7ulp-lpuart
-      - fsl,imx8qxp-lpuart
-      - fsl,imx8qm-lpuart
+    oneOf:
+      - enum:
+          - fsl,vf610-lpuart
+          - fsl,ls1021a-lpuart
+          - fsl,ls1028a-lpuart
+          - fsl,imx7ulp-lpuart
+          - fsl,imx8qxp-lpuart
+          - fsl,imx8qm-lpuart
+      - items:
+          - const: fsl,imx8qxp-lpuart
+          - const: fsl,imx7ulp-lpuart
 
   reg:
     maxItems: 1
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (15 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-24 16:26 ` [PATCH v2 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
  2020-08-25  2:31 ` [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Rob Herring
  18 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

According to device tree specification, device node names should be
somewhat generic and reflecting the function of the device so add the
"hog" suffix to wl-reg-on GPIO hog.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 arch/arm64/boot/dts/freescale/imx8mq-evk.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-evk.dts b/arch/arm64/boot/dts/freescale/imx8mq-evk.dts
index 8aa9cd8e495a..a088831d2e24 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-evk.dts
@@ -157,7 +157,7 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_wifi_reset>;
 
-	wl-reg-on {
+	wl-reg-on-hog {
 		gpio-hog;
 		gpios = <29 GPIO_ACTIVE_HIGH>;
 		output-high;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* [PATCH v2 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (16 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
@ 2020-08-24 16:26 ` Krzysztof Kozlowski
  2020-08-25  2:31 ` [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Rob Herring
  18 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-24 16:26 UTC (permalink / raw)
  To: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Srinivas Kandagatla,
	Will Deacon, Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog
  Cc: Krzysztof Kozlowski

According to device tree specification, device node names should be
somewhat generic and reflecting the function of the device so add the
"hog" suffixes to all GPIO hog nodes.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
index 0d1088dcaa02..fa7a041ffcfd 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
@@ -229,28 +229,28 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_gpio3_hog>;
 
-	usb-emulation {
+	usb-emulation-hog {
 		gpio-hog;
 		gpios = <10 GPIO_ACTIVE_HIGH>;
 		output-low;
 		line-name = "usb-emulation";
 	};
 
-	usb-mode1 {
+	usb-mode1-hog {
 		gpio-hog;
 		gpios = <11 GPIO_ACTIVE_HIGH>;
 		output-high;
 		line-name = "usb-mode1";
 	};
 
-	usb-pwr {
+	usb-pwr-hog {
 		gpio-hog;
 		gpios = <12 GPIO_ACTIVE_LOW>;
 		output-high;
 		line-name = "usb-pwr-ctrl-en-n";
 	};
 
-	usb-mode2 {
+	usb-mode2-hog {
 		gpio-hog;
 		gpios = <13 GPIO_ACTIVE_HIGH>;
 		output-high;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles
  2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (17 preceding siblings ...)
  2020-08-24 16:26 ` [PATCH v2 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
@ 2020-08-25  2:31 ` Rob Herring
  18 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:31 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, linux-pm, Frank Li, Shawn Guo, Han Xu,
	Bartosz Golaszewski, Will Deacon, linux-gpio, linux-arm-kernel,
	NXP Linux Team, Ulf Hansson, Mark Rutland,
	Pengutronix Kernel Team, linux-serial, linux-watchdog, linux-mtd,
	Philipp Zabel, Thierry Reding, devicetree, Srinivas Kandagatla,
	Uwe Kleine-König, linux-pwm, linux-kernel, Li Yang,
	linux-mmc, Sascha Hauer, Fabio Estevam, Guenter Roeck,
	Rob Herring, Miquel Raynal, Fugang Duan

On Mon, 24 Aug 2020 18:26:34 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8 SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
>     compatible:0: 'fsl,imx8mm-gpio' is not one of ['fsl,imx1-gpio', 'fsl,imx21-gpio', 'fsl,imx31-gpio', 'fsl,imx35-gpio', 'fsl,imx7d-gpio']
>     From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
>     compatible: ['fsl,imx8mm-gpio', 'fsl,imx35-gpio'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000:
>     compatible: Additional items are not allowed ('fsl,imx35-gpio' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../bindings/gpio/fsl-imx-gpio.yaml           | 21 +++++++++++++------
>  1 file changed, 15 insertions(+), 6 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property
  2020-08-24 16:26 ` [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
@ 2020-08-25  2:32   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:32 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Ulf Hansson, Linus Walleij, devicetree, Will Deacon,
	Mark Rutland, Guenter Roeck, linux-pm, linux-mmc, Fugang Duan,
	Li Yang, Han Xu, linux-kernel, Sascha Hauer, Miquel Raynal,
	linux-watchdog, Srinivas Kandagatla, linux-serial, Shawn Guo,
	Uwe Kleine-König, linux-gpio, linux-arm-kernel, Frank Li,
	linux-mtd, Thierry Reding, Pengutronix Kernel Team,
	Philipp Zabel, Rob Herring, linux-pwm, NXP Linux Team,
	Fabio Estevam, Bartosz Golaszewski

On Mon, 24 Aug 2020 18:26:35 +0200, Krzysztof Kozlowski wrote:
> The GPIO controller node can have gpio-ranges property.  This fixes
> dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@30200000: 'gpio-ranges' does not match any of the regexes: 'pinctrl-[0-9]+'
>     From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. Remove maxItems
> ---
>  Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  2020-08-24 16:26 ` [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
@ 2020-08-25  2:33   ` Rob Herring
  2020-08-25  2:41     ` Rob Herring
  0 siblings, 1 reply; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:33 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 06:26:36PM +0200, Krzysztof Kozlowski wrote:
> Allow parsing GPIO controller children nodes with GPIO hogs to fix
> warning:
> 
>   arch/arm64/boot/dts/freescale/imx8mq-evk.dt.yaml: gpio@30240000: 'wl-reg-on' does not match any of the regexes: 'pinctrl-[0-9]+'
>     From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml

The below change isn't going to fix 'wl-reg-on'?

> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../devicetree/bindings/gpio/fsl-imx-gpio.yaml  | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> index dffd9171ea66..620a52f944e8 100644
> --- a/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> @@ -53,6 +53,23 @@ properties:
>  
>    gpio-ranges: true
>  
> +patternProperties:
> +  "^(hog-[0-9]+|.+-hog(-[0-9]+)?)$":
> +    type: object
> +    properties:
> +      gpio-hog: true
> +      gpios: true
> +      input: true
> +      output-high: true
> +      output-low: true
> +      line-name: true
> +
> +    required:
> +      - gpio-hog
> +      - gpios
> +
> +    additionalProperties: false
> +
>  required:
>    - compatible
>    - reg
> -- 
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains
  2020-08-24 16:26 ` [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
@ 2020-08-25  2:33   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:33 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-mmc, linux-mtd, Guenter Roeck, Pengutronix Kernel Team,
	linux-serial, Bartosz Golaszewski, Thierry Reding, Frank Li,
	Sascha Hauer, Li Yang, linux-kernel, linux-gpio, linux-pwm,
	Rob Herring, linux-pm, Fugang Duan, Mark Rutland, Linus Walleij,
	Han Xu, NXP Linux Team, Will Deacon, linux-arm-kernel,
	Ulf Hansson, Srinivas Kandagatla, linux-watchdog, Shawn Guo,
	Fabio Estevam, Uwe Kleine-König, devicetree, Philipp Zabel,
	Miquel Raynal

On Mon, 24 Aug 2020 18:26:37 +0200, Krzysztof Kozlowski wrote:
> Parse also optional power-domains property to fix dtbs_check warnings
> like:
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-ai_ml.dt.yaml: gpio@5d080000: 'power-domains' does not match any of the regexes: 'pinctrl-[0-9]+'
>     From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-25  2:33   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:33 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-pm, linux-kernel, devicetree, Han Xu, linux-gpio,
	Fugang Duan, Bartosz Golaszewski, linux-serial, Guenter Roeck,
	linux-arm-kernel, Linus Walleij, Philipp Zabel, linux-watchdog,
	Miquel Raynal, Pengutronix Kernel Team, Fabio Estevam,
	Srinivas Kandagatla, Frank Li, linux-mtd, linux-mmc,
	Mark Rutland, Will Deacon, Uwe Kleine-König, Thierry Reding,
	Shawn Guo, NXP Linux Team, Li Yang, Sascha Hauer, Rob Herring,
	linux-pwm, Ulf Hansson

On Mon, 24 Aug 2020 18:26:38 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
>     compatible:0: 'fsl,imx8mm-ddr-pmu' is not one of ['fsl,imx8-ddr-pmu', 'fsl,imx8m-ddr-pmu', 'fsl,imx8mp-ddr-pmu']
>     From schema: Documentation/devicetree/bindings/perf/fsl-imx-ddr.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
>     compatible: ['fsl,imx8mm-ddr-pmu', 'fsl,imx8m-ddr-pmu'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: ddr-pmu@3d800000:
>     compatible: Additional items are not allowed ('fsl,imx8m-ddr-pmu' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. Handle also fsl,imx8mp-ddr-pmu
> ---
>  .../devicetree/bindings/perf/fsl-imx-ddr.yaml    | 16 ++++++++++++----
>  1 file changed, 12 insertions(+), 4 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
@ 2020-08-25  2:34   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:34 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-serial, Thierry Reding, linux-pwm, Han Xu, Fabio Estevam,
	Linus Walleij, NXP Linux Team, Philipp Zabel, Will Deacon,
	Miquel Raynal, devicetree, linux-arm-kernel, Srinivas Kandagatla,
	Shawn Guo, Sascha Hauer, Bartosz Golaszewski, Fugang Duan,
	Frank Li, Ulf Hansson, Guenter Roeck, Li Yang, linux-watchdog,
	linux-gpio, Uwe Kleine-König, Pengutronix Kernel Team,
	linux-mmc, linux-mtd, Mark Rutland, Rob Herring, linux-kernel,
	linux-pm

On Mon, 24 Aug 2020 18:26:39 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm']
>     From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
@ 2020-08-25  2:34   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:34 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-gpio, linux-pm, linux-pwm, Frank Li, linux-kernel,
	linux-arm-kernel, Miquel Raynal, Will Deacon, Li Yang,
	Rob Herring, Thierry Reding, Sascha Hauer, Shawn Guo,
	Philipp Zabel, Bartosz Golaszewski, linux-mmc, Fabio Estevam,
	Srinivas Kandagatla, Fugang Duan, Linus Walleij, Han Xu,
	Ulf Hansson, NXP Linux Team, Guenter Roeck, linux-mtd,
	Pengutronix Kernel Team, devicetree, linux-serial,
	linux-watchdog, Uwe Kleine-König, Mark Rutland

On Mon, 24 Aug 2020 18:26:40 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm']
>     From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000:
>     compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. Fix subject prefix
> ---
>  Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
@ 2020-08-25  2:35   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:35 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-pm, NXP Linux Team, Pengutronix Kernel Team,
	linux-watchdog, Li Yang, Uwe Kleine-König, linux-gpio,
	linux-serial, linux-kernel, linux-mtd, Philipp Zabel, Frank Li,
	Linus Walleij, linux-pwm, Thierry Reding, Miquel Raynal,
	Fabio Estevam, Rob Herring, Shawn Guo, Ulf Hansson, Will Deacon,
	Fugang Duan, linux-arm-kernel, Sascha Hauer, Han Xu, devicetree,
	Mark Rutland, linux-mmc, Bartosz Golaszewski,
	Srinivas Kandagatla, Guenter Roeck

On Mon, 24 Aug 2020 18:26:41 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
>     compatible:0: 'fsl,imx8mm-wdt' is not one of ['fsl,imx21-wdt']
>     From schema: Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
>     compatible: ['fsl,imx8mm-wdt', 'fsl,imx21-wdt'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-var-som-symphony.dt.yaml: watchdog@30280000:
>     compatible: Additional items are not allowed ('fsl,imx21-wdt' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../devicetree/bindings/watchdog/fsl-imx-wdt.yaml     | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
@ 2020-08-25  2:35   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:35 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Srinivas Kandagatla, Pengutronix Kernel Team, Guenter Roeck,
	Mark Rutland, Miquel Raynal, devicetree, linux-pwm, Rob Herring,
	Frank Li, Ulf Hansson, Fabio Estevam, Shawn Guo, Han Xu,
	linux-mtd, linux-kernel, Linus Walleij, Sascha Hauer,
	NXP Linux Team, Philipp Zabel, linux-watchdog, Li Yang, linux-pm,
	Uwe Kleine-König, Thierry Reding, linux-gpio, Will Deacon,
	linux-arm-kernel, Fugang Duan, linux-mmc, linux-serial,
	Bartosz Golaszewski

On Mon, 24 Aug 2020 18:26:42 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
>     compatible:0: 'fsl,imx8mm-gpmi-nand' is not one of ['fsl,imx23-gpmi-nand', 'fsl,imx28-gpmi-nand', 'fsl,imx6q-gpmi-nand', 'fsl,imx6sx-gpmi-nand', 'fsl,imx7d-gpmi-nand']
>     From schema: Documentation/devicetree/bindings/mtd/gpmi-nand.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
>     compatible: ['fsl,imx8mm-gpmi-nand', 'fsl,imx7d-gpmi-nand'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: nand-controller@33002000:
>     compatible: Additional items are not allowed ('fsl,imx7d-gpmi-nand' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../devicetree/bindings/mtd/gpmi-nand.yaml     | 18 ++++++++++++------
>  1 file changed, 12 insertions(+), 6 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 10/19] dt-bindings: reset: fsl, imx7-src: Add i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
@ 2020-08-25  2:36   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:36 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Uwe Kleine-König, Srinivas Kandagatla, Philipp Zabel,
	Frank Li, Fabio Estevam, linux-pm, linux-gpio, Fugang Duan,
	linux-pwm, Will Deacon, Rob Herring, linux-arm-kernel,
	Sascha Hauer, Miquel Raynal, Shawn Guo, linux-mtd, linux-mmc,
	Han Xu, Guenter Roeck, Thierry Reding, linux-watchdog,
	linux-kernel, Li Yang, linux-serial, devicetree, Linus Walleij,
	Mark Rutland, NXP Linux Team, Ulf Hansson,
	Pengutronix Kernel Team, Bartosz Golaszewski

On Mon, 24 Aug 2020 18:26:43 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
>     compatible:0: 'fsl,imx8mm-src' is not one of ['fsl,imx7d-src', 'fsl,imx8mq-src', 'fsl,imx8mp-src']
>     From schema: Documentation/devicetree/bindings/reset/fsl,imx7-src.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
>     compatible:1: 'syscon' was expected
> 
>   arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: reset-controller@30390000:
>     compatible: ['fsl,imx8mm-src', 'fsl,imx8mq-src', 'syscon'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../bindings/reset/fsl,imx7-src.yaml          | 19 +++++++++++++------
>  1 file changed, 13 insertions(+), 6 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible
  2020-08-24 16:26 ` [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
@ 2020-08-25  2:36   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:36 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-watchdog, Guenter Roeck, linux-serial,
	Uwe Kleine-König, Sascha Hauer, Li Yang, Shawn Guo,
	devicetree, Philipp Zabel, Frank Li, Miquel Raynal, Rob Herring,
	Fugang Duan, Mark Rutland, linux-gpio, linux-pwm, Fabio Estevam,
	Will Deacon, linux-kernel, linux-pm, linux-mmc,
	Srinivas Kandagatla, Thierry Reding, Pengutronix Kernel Team,
	linux-mtd, linux-arm-kernel, Han Xu, Bartosz Golaszewski,
	NXP Linux Team, Ulf Hansson, Linus Walleij

On Mon, 24 Aug 2020 18:26:44 +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs introduce their own compatibles so add them
> to fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
>     compatible:0: 'fsl,imx8mn-tmu' is not one of ['fsl,imx8mm-tmu', 'fsl,imx8mp-tmu']
>     From schema: Documentation/devicetree/bindings/thermal/imx8mm-thermal.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
>     compatible: ['fsl,imx8mn-tmu', 'fsl,imx8mm-tmu'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: tmu@30260000:
>     compatible: Additional items are not allowed ('fsl,imx8mm-tmu' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../devicetree/bindings/thermal/imx8mm-thermal.yaml    | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-24 16:26 ` [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
@ 2020-08-25  2:38   ` Rob Herring
  2020-08-25  7:08     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:38 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 06:26:45PM +0200, Krzysztof Kozlowski wrote:
> The i.MX 8 DTSes use two compatibles so update the binding to fix
> dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
>     compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long
>     From schema: Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
>     compatible: Additional items are not allowed ('fsl,imx7d-usdhc' was unexpected)
> 
>   arch/arm64/boot/dts/freescale/imx8mn-ddr4-evk.dt.yaml: mmc@30b40000:
>     compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
> ---
>  .../bindings/mmc/fsl-imx-esdhc.yaml           | 40 +++++++++++--------
>  1 file changed, 23 insertions(+), 17 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> index 10b45966f1b8..54f05e2a4246 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> @@ -21,23 +21,29 @@ description: |
>  
>  properties:
>    compatible:
> -    enum:
> -      - fsl,imx25-esdhc
> -      - fsl,imx35-esdhc
> -      - fsl,imx51-esdhc
> -      - fsl,imx53-esdhc
> -      - fsl,imx6q-usdhc
> -      - fsl,imx6sl-usdhc
> -      - fsl,imx6sx-usdhc
> -      - fsl,imx6ull-usdhc
> -      - fsl,imx7d-usdhc
> -      - fsl,imx7ulp-usdhc
> -      - fsl,imx8mq-usdhc
> -      - fsl,imx8mm-usdhc
> -      - fsl,imx8mn-usdhc
> -      - fsl,imx8mp-usdhc
> -      - fsl,imx8qm-usdhc
> -      - fsl,imx8qxp-usdhc
> +    oneOf:
> +      - enum:
> +          - fsl,imx25-esdhc
> +          - fsl,imx35-esdhc
> +          - fsl,imx51-esdhc
> +          - fsl,imx53-esdhc
> +          - fsl,imx6q-usdhc
> +          - fsl,imx6sl-usdhc
> +          - fsl,imx6sx-usdhc
> +          - fsl,imx6ull-usdhc
> +          - fsl,imx7d-usdhc
> +          - fsl,imx7ulp-usdhc

> +          - fsl,imx8mq-usdhc
> +          - fsl,imx8mm-usdhc
> +          - fsl,imx8qxp-usdhc

These should be dropped.

> +      - items:
> +          - enum:
> +              - fsl,imx8mm-usdhc
> +              - fsl,imx8mn-usdhc
> +              - fsl,imx8mp-usdhc
> +              - fsl,imx8mq-usdhc
> +              - fsl,imx8qxp-usdhc
> +          - const: fsl,imx7d-usdhc
>  
>    reg:
>      maxItems: 1
> -- 
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-25  2:39   ` Rob Herring
  2020-08-25  7:19     ` Krzysztof Kozlowski
  2020-08-25  4:46   ` Sascha Hauer
  1 sibling, 1 reply; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:39 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 06:26:46PM +0200, Krzysztof Kozlowski wrote:
> DTSes with new i.MX 8M SoCs use two compatibles so update the binding to
> fix dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000: compatible:1: 'syscon' was expected
>     From schema: Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
>     compatible: ['fsl,imx8mn-ocotp', 'fsl,imx8mm-ocotp', 'syscon'] is too long
> 
>   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
>     compatible: Additional items are not allowed ('syscon' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 39 ++++++++++++-------
>  1 file changed, 24 insertions(+), 15 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> index 1c9d7f05f173..b5b250185afd 100644
> --- a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> +++ b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> @@ -19,21 +19,30 @@ allOf:
>  
>  properties:
>    compatible:
> -    items:
> -      - enum:
> -          - fsl,imx6q-ocotp
> -          - fsl,imx6sl-ocotp
> -          - fsl,imx6sx-ocotp
> -          - fsl,imx6ul-ocotp
> -          - fsl,imx6ull-ocotp
> -          - fsl,imx7d-ocotp
> -          - fsl,imx6sll-ocotp
> -          - fsl,imx7ulp-ocotp
> -          - fsl,imx8mq-ocotp
> -          - fsl,imx8mm-ocotp
> -          - fsl,imx8mn-ocotp
> -          - fsl,imx8mp-ocotp
> -      - const: syscon
> +    oneOf:
> +      - items:
> +          - enum:
> +              - fsl,imx6q-ocotp
> +              - fsl,imx6sl-ocotp
> +              - fsl,imx6sx-ocotp
> +              - fsl,imx6ul-ocotp
> +              - fsl,imx6ull-ocotp
> +              - fsl,imx7d-ocotp
> +              - fsl,imx6sll-ocotp
> +              - fsl,imx7ulp-ocotp
> +              - fsl,imx8mq-ocotp
> +              - fsl,imx8mm-ocotp

> +              - fsl,imx8mn-ocotp
> +              - fsl,imx8mp-ocotp

Should be dropped.

> +          - const: syscon
> +      - items:
> +          # The devices are not really compatible with fsl,imx8mm-ocotp, however
> +          # the code for getting SoC revision depends on fsl,imx8mm-ocotp compatible.
> +          - enum:
> +              - fsl,imx8mn-ocotp
> +              - fsl,imx8mp-ocotp
> +          - const: fsl,imx8mm-ocotp
> +          - const: syscon
>  
>    reg:
>      maxItems: 1
> -- 
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  2020-08-24 16:26 ` [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
@ 2020-08-25  2:39   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:39 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-pwm, linux-serial, Bartosz Golaszewski, linux-watchdog,
	linux-gpio, devicetree, Miquel Raynal, Guenter Roeck,
	Thierry Reding, Sascha Hauer, Uwe Kleine-König,
	Fabio Estevam, Srinivas Kandagatla, linux-kernel, linux-mtd,
	Will Deacon, linux-arm-kernel, Fugang Duan, Rob Herring,
	Shawn Guo, Linus Walleij, Han Xu, Pengutronix Kernel Team,
	Li Yang, linux-pm, Frank Li, NXP Linux Team, Ulf Hansson,
	Mark Rutland, Philipp Zabel, linux-mmc

On Mon, 24 Aug 2020 18:26:47 +0200, Krzysztof Kozlowski wrote:
> The Toradex Colibri i.MX 8 Evaluation board has two Toradex compatibles
> so it needs separate entry.  This fixes dtbs_check warning:
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dt.yaml: /:
>     compatible: ['toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x', 'fsl,imx8qxp'] is not valid under any of the given schemas (Possible causes of the failure):
>     arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dt.yaml: /: compatible: ['toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x', 'fsl,imx8qxp'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding
  2020-08-24 16:26 ` [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
@ 2020-08-25  2:40   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-mmc, Philipp Zabel, Guenter Roeck, Pengutronix Kernel Team,
	Linus Walleij, Rob Herring, Ulf Hansson, linux-serial,
	Sascha Hauer, Han Xu, Bartosz Golaszewski, devicetree,
	Mark Rutland, linux-watchdog, Li Yang, linux-arm-kernel,
	Frank Li, linux-pm, Srinivas Kandagatla, Will Deacon, Shawn Guo,
	Uwe Kleine-König, Fabio Estevam, Miquel Raynal, linux-gpio,
	Thierry Reding, NXP Linux Team, linux-pwm, Fugang Duan,
	linux-kernel, linux-mtd

On Mon, 24 Aug 2020 18:26:48 +0200, Krzysztof Kozlowski wrote:
> Document the binding for Zodiac Inflight Innovations Ultra Boards.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl, irqsteer: Fix compatible matching
  2020-08-24 16:26 ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
@ 2020-08-25  2:40   ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Srinivas Kandagatla, Han Xu, Rob Herring, linux-pwm,
	Pengutronix Kernel Team, linux-mtd, Linus Walleij, linux-kernel,
	linux-arm-kernel, Bartosz Golaszewski, Thierry Reding, linux-mmc,
	Sascha Hauer, Mark Rutland, linux-gpio, Ulf Hansson,
	linux-watchdog, Will Deacon, Uwe Kleine-König, linux-pm,
	Shawn Guo, Guenter Roeck, linux-serial, Fabio Estevam,
	devicetree, Frank Li, Li Yang, Fugang Duan, Miquel Raynal,
	NXP Linux Team, Philipp Zabel

On Mon, 24 Aug 2020 18:26:49 +0200, Krzysztof Kozlowski wrote:
> The i.MX 8M DTSes use two compatibles so update the binding to fix
> dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8mq-thor96.dt.yaml: interrupt-controller@32e2d000:
>     compatible: ['fsl,imx8m-irqsteer', 'fsl,imx-irqsteer'] is too long
>     From schema: Domentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8mq-thor96.dt.yaml: interrupt-controller@32e2d000:
>     compatible: Additional items are not allowed ('fsl,imx-irqsteer' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  .../bindings/interrupt-controller/fsl,irqsteer.yaml       | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  2020-08-25  2:33   ` Rob Herring
@ 2020-08-25  2:41     ` Rob Herring
  0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:41 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 08:33:06PM -0600, Rob Herring wrote:
> On Mon, Aug 24, 2020 at 06:26:36PM +0200, Krzysztof Kozlowski wrote:
> > Allow parsing GPIO controller children nodes with GPIO hogs to fix
> > warning:
> > 
> >   arch/arm64/boot/dts/freescale/imx8mq-evk.dt.yaml: gpio@30240000: 'wl-reg-on' does not match any of the regexes: 'pinctrl-[0-9]+'
> >     From schema: Documentation/devicetree/bindings/gpio/fsl-imx-gpio.yaml
> 
> The below change isn't going to fix 'wl-reg-on'?

Never mind, saw you changed it.

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: Fix compatible matching
  2020-08-24 16:26 ` [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
@ 2020-08-25  2:42   ` Rob Herring
  2020-08-25  7:55     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 42+ messages in thread
From: Rob Herring @ 2020-08-25  2:42 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 06:26:50PM +0200, Krzysztof Kozlowski wrote:
> The i.MX 8QXP DTSes use two compatibles so update the binding to fix
> dtbs_check warnings like:
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
>     compatible: ['fsl,imx8qxp-lpuart', 'fsl,imx7ulp-lpuart'] is too long
>     From schema: Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> 
>   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
>     compatible: Additional items are not allowed ('fsl,imx7ulp-lpuart' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> ---
> 
> Changes since v1:
> 1. New patch.
> ---
>  .../devicetree/bindings/serial/fsl-lpuart.yaml | 18 +++++++++++-------
>  1 file changed, 11 insertions(+), 7 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> index e82c2cf9fef7..8ee651f2ef0b 100644
> --- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> +++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> @@ -14,13 +14,17 @@ allOf:
>  
>  properties:
>    compatible:
> -    enum:
> -      - fsl,vf610-lpuart
> -      - fsl,ls1021a-lpuart
> -      - fsl,ls1028a-lpuart
> -      - fsl,imx7ulp-lpuart
> -      - fsl,imx8qxp-lpuart
> -      - fsl,imx8qm-lpuart
> +    oneOf:
> +      - enum:
> +          - fsl,vf610-lpuart
> +          - fsl,ls1021a-lpuart
> +          - fsl,ls1028a-lpuart
> +          - fsl,imx7ulp-lpuart
> +          - fsl,imx8qxp-lpuart

This should be dropped.

> +          - fsl,imx8qm-lpuart
> +      - items:
> +          - const: fsl,imx8qxp-lpuart
> +          - const: fsl,imx7ulp-lpuart
>  
>    reg:
>      maxItems: 1
> -- 
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-24 16:26 ` [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
  2020-08-25  2:39   ` Rob Herring
@ 2020-08-25  4:46   ` Sascha Hauer
  2020-08-25  7:17     ` Krzysztof Kozlowski
  1 sibling, 1 reply; 42+ messages in thread
From: Sascha Hauer @ 2020-08-25  4:46 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 06:26:46PM +0200, Krzysztof Kozlowski wrote:
> +    oneOf:
> +      - items:
> +          - enum:
> +              - fsl,imx6q-ocotp
> +              - fsl,imx6sl-ocotp
> +              - fsl,imx6sx-ocotp
> +              - fsl,imx6ul-ocotp
> +              - fsl,imx6ull-ocotp
> +              - fsl,imx7d-ocotp
> +              - fsl,imx6sll-ocotp
> +              - fsl,imx7ulp-ocotp
> +              - fsl,imx8mq-ocotp
> +              - fsl,imx8mm-ocotp
> +              - fsl,imx8mn-ocotp
> +              - fsl,imx8mp-ocotp
> +          - const: syscon
> +      - items:
> +          # The devices are not really compatible with fsl,imx8mm-ocotp, however
> +          # the code for getting SoC revision depends on fsl,imx8mm-ocotp compatible.

Shouldn't this be fixed? It seems strange to justify a binding with
existing code.

Sascha

> +          - enum:
> +              - fsl,imx8mn-ocotp
> +              - fsl,imx8mp-ocotp
> +          - const: fsl,imx8mm-ocotp
> +          - const: syscon
>  
>    reg:
>      maxItems: 1
> -- 
> 2.17.1
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-25  2:38   ` Rob Herring
@ 2020-08-25  7:08     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25  7:08 UTC (permalink / raw)
  To: Rob Herring
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 08:38:01PM -0600, Rob Herring wrote:
> On Mon, Aug 24, 2020 at 06:26:45PM +0200, Krzysztof Kozlowski wrote:
> > The i.MX 8 DTSes use two compatibles so update the binding to fix
> > dtbs_check warnings like:
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
> >     compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long
> >     From schema: Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: mmc@30b40000:
> >     compatible: Additional items are not allowed ('fsl,imx7d-usdhc' was unexpected)
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-ddr4-evk.dt.yaml: mmc@30b40000:
> >     compatible: ['fsl,imx8mn-usdhc', 'fsl,imx7d-usdhc'] is too long
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > 
> > ---
> > 
> > Changes since v1:
> > 1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
> > ---
> >  .../bindings/mmc/fsl-imx-esdhc.yaml           | 40 +++++++++++--------
> >  1 file changed, 23 insertions(+), 17 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> > index 10b45966f1b8..54f05e2a4246 100644
> > --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> > +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> > @@ -21,23 +21,29 @@ description: |
> >  
> >  properties:
> >    compatible:
> > -    enum:
> > -      - fsl,imx25-esdhc
> > -      - fsl,imx35-esdhc
> > -      - fsl,imx51-esdhc
> > -      - fsl,imx53-esdhc
> > -      - fsl,imx6q-usdhc
> > -      - fsl,imx6sl-usdhc
> > -      - fsl,imx6sx-usdhc
> > -      - fsl,imx6ull-usdhc
> > -      - fsl,imx7d-usdhc
> > -      - fsl,imx7ulp-usdhc
> > -      - fsl,imx8mq-usdhc
> > -      - fsl,imx8mm-usdhc
> > -      - fsl,imx8mn-usdhc
> > -      - fsl,imx8mp-usdhc
> > -      - fsl,imx8qm-usdhc
> > -      - fsl,imx8qxp-usdhc
> > +    oneOf:
> > +      - enum:
> > +          - fsl,imx25-esdhc
> > +          - fsl,imx35-esdhc
> > +          - fsl,imx51-esdhc
> > +          - fsl,imx53-esdhc
> > +          - fsl,imx6q-usdhc
> > +          - fsl,imx6sl-usdhc
> > +          - fsl,imx6sx-usdhc
> > +          - fsl,imx6ull-usdhc
> > +          - fsl,imx7d-usdhc
> > +          - fsl,imx7ulp-usdhc
> 
> > +          - fsl,imx8mq-usdhc
> > +          - fsl,imx8mm-usdhc
> > +          - fsl,imx8qxp-usdhc
> 
> These should be dropped.

You're right, thanks.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-25  4:46   ` Sascha Hauer
@ 2020-08-25  7:17     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25  7:17 UTC (permalink / raw)
  To: Sascha Hauer
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Tue, Aug 25, 2020 at 06:46:53AM +0200, Sascha Hauer wrote:
> On Mon, Aug 24, 2020 at 06:26:46PM +0200, Krzysztof Kozlowski wrote:
> > +    oneOf:
> > +      - items:
> > +          - enum:
> > +              - fsl,imx6q-ocotp
> > +              - fsl,imx6sl-ocotp
> > +              - fsl,imx6sx-ocotp
> > +              - fsl,imx6ul-ocotp
> > +              - fsl,imx6ull-ocotp
> > +              - fsl,imx7d-ocotp
> > +              - fsl,imx6sll-ocotp
> > +              - fsl,imx7ulp-ocotp
> > +              - fsl,imx8mq-ocotp
> > +              - fsl,imx8mm-ocotp
> > +              - fsl,imx8mn-ocotp
> > +              - fsl,imx8mp-ocotp
> > +          - const: syscon
> > +      - items:
> > +          # The devices are not really compatible with fsl,imx8mm-ocotp, however
> > +          # the code for getting SoC revision depends on fsl,imx8mm-ocotp compatible.
> 
> Shouldn't this be fixed? It seems strange to justify a binding with
> existing code.

The fsl,imx8mn-ocotp is actually compatible with "mm", only the
fsl,imx8mp-ocotp is not (although someone might argue that subset of
"mp" falls into "mn").

The problem is that drivers/soc/imx/soc-imx8m.c checks revision only
against the fsl,imx8mm-ocotp compatible, not the others. It's the
pattern used in that driver also for other i.MX 8. Therefore it is a
binding implemented and actually used, regardless whether it is correct
from hardware point of view or not.

If that's preferred, I could adjust the drivers/soc/imx/soc-imx8m.c for
the incompatible "mp". It should not break anything.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-25  2:39   ` Rob Herring
@ 2020-08-25  7:19     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25  7:19 UTC (permalink / raw)
  To: Rob Herring
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 08:39:04PM -0600, Rob Herring wrote:
> On Mon, Aug 24, 2020 at 06:26:46PM +0200, Krzysztof Kozlowski wrote:
> > DTSes with new i.MX 8M SoCs use two compatibles so update the binding to
> > fix dtbs_check warnings like:
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000: compatible:1: 'syscon' was expected
> >     From schema: Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
> >     compatible: ['fsl,imx8mn-ocotp', 'fsl,imx8mm-ocotp', 'syscon'] is too long
> > 
> >   arch/arm64/boot/dts/freescale/imx8mn-evk.dt.yaml: efuse@30350000:
> >     compatible: Additional items are not allowed ('syscon' was unexpected)
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >  .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 39 ++++++++++++-------
> >  1 file changed, 24 insertions(+), 15 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> > index 1c9d7f05f173..b5b250185afd 100644
> > --- a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> > +++ b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
> > @@ -19,21 +19,30 @@ allOf:
> >  
> >  properties:
> >    compatible:
> > -    items:
> > -      - enum:
> > -          - fsl,imx6q-ocotp
> > -          - fsl,imx6sl-ocotp
> > -          - fsl,imx6sx-ocotp
> > -          - fsl,imx6ul-ocotp
> > -          - fsl,imx6ull-ocotp
> > -          - fsl,imx7d-ocotp
> > -          - fsl,imx6sll-ocotp
> > -          - fsl,imx7ulp-ocotp
> > -          - fsl,imx8mq-ocotp
> > -          - fsl,imx8mm-ocotp
> > -          - fsl,imx8mn-ocotp
> > -          - fsl,imx8mp-ocotp
> > -      - const: syscon
> > +    oneOf:
> > +      - items:
> > +          - enum:
> > +              - fsl,imx6q-ocotp
> > +              - fsl,imx6sl-ocotp
> > +              - fsl,imx6sx-ocotp
> > +              - fsl,imx6ul-ocotp
> > +              - fsl,imx6ull-ocotp
> > +              - fsl,imx7d-ocotp
> > +              - fsl,imx6sll-ocotp
> > +              - fsl,imx7ulp-ocotp
> > +              - fsl,imx8mq-ocotp
> > +              - fsl,imx8mm-ocotp
> 
> > +              - fsl,imx8mn-ocotp
> > +              - fsl,imx8mp-ocotp
> 
> Should be dropped.

OK

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 42+ messages in thread

* Re: [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: Fix compatible matching
  2020-08-25  2:42   ` Rob Herring
@ 2020-08-25  7:55     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25  7:55 UTC (permalink / raw)
  To: Rob Herring
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Srinivas Kandagatla, Will Deacon,
	Mark Rutland, Thierry Reding, Uwe Kleine-König,
	Philipp Zabel, Guenter Roeck, Li Yang, Han Xu, Frank Li,
	Fugang Duan, devicetree, linux-kernel, linux-gpio,
	linux-arm-kernel, linux-mmc, linux-mtd, linux-pwm, linux-serial,
	linux-pm, linux-watchdog

On Mon, Aug 24, 2020 at 08:42:26PM -0600, Rob Herring wrote:
> On Mon, Aug 24, 2020 at 06:26:50PM +0200, Krzysztof Kozlowski wrote:
> > The i.MX 8QXP DTSes use two compatibles so update the binding to fix
> > dtbs_check warnings like:
> > 
> >   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
> >     compatible: ['fsl,imx8qxp-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> >     From schema: Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > 
> >   arch/arm64/boot/dts/freescale/imx8qxp-mek.dt.yaml: serial@5a060000:
> >     compatible: Additional items are not allowed ('fsl,imx7ulp-lpuart' was unexpected)
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > 
> > ---
> > 
> > Changes since v1:
> > 1. New patch.
> > ---
> >  .../devicetree/bindings/serial/fsl-lpuart.yaml | 18 +++++++++++-------
> >  1 file changed, 11 insertions(+), 7 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > index e82c2cf9fef7..8ee651f2ef0b 100644
> > --- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > +++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > @@ -14,13 +14,17 @@ allOf:
> >  
> >  properties:
> >    compatible:
> > -    enum:
> > -      - fsl,vf610-lpuart
> > -      - fsl,ls1021a-lpuart
> > -      - fsl,ls1028a-lpuart
> > -      - fsl,imx7ulp-lpuart
> > -      - fsl,imx8qxp-lpuart
> > -      - fsl,imx8qm-lpuart
> > +    oneOf:
> > +      - enum:
> > +          - fsl,vf610-lpuart
> > +          - fsl,ls1021a-lpuart
> > +          - fsl,ls1028a-lpuart
> > +          - fsl,imx7ulp-lpuart
> > +          - fsl,imx8qxp-lpuart
> 
> This should be dropped.

Right.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 42+ messages in thread

end of thread, other threads:[~2020-08-25  7:55 UTC | newest]

Thread overview: 42+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-24 16:26 [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
2020-08-24 16:26 ` [PATCH v2 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
2020-08-25  2:32   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
2020-08-25  2:33   ` Rob Herring
2020-08-25  2:41     ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
2020-08-25  2:33   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
2020-08-25  2:33   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
2020-08-25  2:34   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
2020-08-25  2:34   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
2020-08-25  2:35   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
2020-08-25  2:35   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
2020-08-25  2:36   ` [PATCH v2 10/19] dt-bindings: reset: fsl, imx7-src: " Rob Herring
2020-08-24 16:26 ` [PATCH v2 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
2020-08-25  2:36   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
2020-08-25  2:38   ` Rob Herring
2020-08-25  7:08     ` Krzysztof Kozlowski
2020-08-24 16:26 ` [PATCH v2 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
2020-08-25  2:39   ` Rob Herring
2020-08-25  7:19     ` Krzysztof Kozlowski
2020-08-25  4:46   ` Sascha Hauer
2020-08-25  7:17     ` Krzysztof Kozlowski
2020-08-24 16:26 ` [PATCH v2 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
2020-08-25  2:39   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
2020-08-25  2:40   ` Rob Herring
2020-08-24 16:26 ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
2020-08-25  2:40   ` [PATCH v2 16/19] dt-bindings: interrupt-controller: fsl, irqsteer: " Rob Herring
2020-08-24 16:26 ` [PATCH v2 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
2020-08-25  2:42   ` Rob Herring
2020-08-25  7:55     ` Krzysztof Kozlowski
2020-08-24 16:26 ` [PATCH v2 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
2020-08-24 16:26 ` [PATCH v2 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
2020-08-25  2:31 ` [PATCH v2 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Rob Herring

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 NNTP newsgroup(s).