linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings
@ 2020-08-25 19:35 Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
                   ` (19 more replies)
  0 siblings, 20 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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

Hi,

This is a v3 of cleanup of i.XM 8 bindings and DTSes.

It is separate patchset from i.MX 8 pin configuration cleanup, which
also touch the bindings [1]. No dependencies (although in my tree this
comes first).

Merging
=======
There are no dependencies, so dt-bindings could go through Rob's tree,
DTS through SoC. I think there is no point to push dt-bindings changes
through subsystem maintainers (gpio, pwm, watchdog, mtd etc). Usually
Rob has been picking them up.

Changes since v2:
=================
1. Add Rob's review,
2. Correct things pointed during review (see individual patches and
their change logs).

[1] dt-bindings: mtd: gpmi-nand: Fix matching of clocks on different SoC

Best regards,
Krzysztof


Krzysztof Kozlowski (19):
  dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles
  dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property
  dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  dt-bindings: gpio: fsl-imx-gpio: Add power-domains
  dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles
  dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  dt-bindings: serial: fsl-imx-uart: Add i.MX 8M compatibles
  dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  dt-bindings: mtd: gpmi-nand: Add i.MX 8M compatibles
  dt-bindings: reset: fsl,imx7-src: Add i.MX 8M compatibles
  dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible
  dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  dt-bindings: arm: fsl: Add ZII Ultra boards binding
  dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible
    matching
  dt-bindings: serial: fsl-lpuart: Fix compatible matching
  arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on
  arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs

 .../devicetree/bindings/arm/fsl.yaml          | 14 ++++++
 .../bindings/gpio/fsl-imx-gpio.yaml           | 43 ++++++++++++++++---
 .../interrupt-controller/fsl,irqsteer.yaml    |  8 ++--
 .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++--------
 .../devicetree/bindings/mtd/gpmi-nand.yaml    | 18 +++++---
 .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 38 +++++++++-------
 .../devicetree/bindings/perf/fsl-imx-ddr.yaml | 16 +++++--
 .../devicetree/bindings/pwm/imx-pwm.yaml      | 14 ++++--
 .../bindings/reset/fsl,imx7-src.yaml          | 19 +++++---
 .../bindings/serial/fsl-imx-uart.yaml         |  4 ++
 .../bindings/serial/fsl-lpuart.yaml           | 17 +++++---
 .../bindings/thermal/imx8mm-thermal.yaml      | 10 +++--
 .../bindings/watchdog/fsl-imx-wdt.yaml        | 11 ++++-
 arch/arm64/boot/dts/freescale/imx8mq-evk.dts  |  2 +-
 .../boot/dts/freescale/imx8mq-zii-ultra.dtsi  |  8 ++--
 15 files changed, 182 insertions(+), 77 deletions(-)

-- 
2.17.1


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

* [PATCH v3 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
                   ` (18 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
                   ` (17 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
                   ` (16 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (2 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
                   ` (15 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (3 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
                   ` (14 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (4 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-09-23 11:52   ` Thierry Reding
  2020-08-25 19:35 ` [PATCH v3 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
                   ` (13 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 07/19] dt-bindings: serial: fsl-imx-uart: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (5 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
                   ` (12 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (6 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 22:21   ` Guenter Roeck
  2020-08-25 19:35 ` [PATCH v3 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
                   ` (11 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 09/19] dt-bindings: mtd: gpmi-nand: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (7 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
                   ` (10 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 10/19] dt-bindings: reset: fsl,imx7-src: Add i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (8 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
                   ` (9 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (9 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
                   ` (8 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (10 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-28  8:45   ` Ulf Hansson
  2020-08-28 21:38   ` Rob Herring
  2020-08-25 19:35 ` [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
                   ` (7 subsequent siblings)
  19 siblings, 2 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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 v2:
1. Remove moved compatibles.

Changes since v1:
1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
---
 .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++++---------
 1 file changed, 20 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..e71d13c2d109 100644
--- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
+++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
@@ -21,23 +21,26 @@ 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
+      - 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] 37+ messages in thread

* [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (11 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-28 22:26   ` Rob Herring
  2020-08-25 19:35 ` [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
                   ` (6 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>

---

Changes since v2:
1. Remove moved compatibles.
---
 .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 38 +++++++++++--------
 1 file changed, 23 insertions(+), 15 deletions(-)

diff --git a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
index 1c9d7f05f173..5a7284737229 100644
--- a/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
+++ b/Documentation/devicetree/bindings/nvmem/imx-ocotp.yaml
@@ -19,21 +19,29 @@ 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
+          - const: syscon
+      - items:
+          - enum:
+              - fsl,imx8mn-ocotp
+              # i.MX8MP not really compatible with fsl,imx8mm-ocotp, however
+              # the code for getting SoC revision depends on fsl,imx8mm-ocotp
+              # compatible.
+              - fsl,imx8mp-ocotp
+          - const: fsl,imx8mm-ocotp
+          - const: syscon
 
   reg:
     maxItems: 1
-- 
2.17.1


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

* [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (12 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-31  3:22   ` Shawn Guo
  2020-08-25 19:35 ` [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
                   ` (5 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (13 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-31  3:22   ` Shawn Guo
  2020-08-25 19:35 ` [PATCH v3 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
                   ` (4 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (14 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-25 19:35 ` [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
                   ` (3 subsequent siblings)
  19 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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>
Reviewed-by: Rob Herring <robh@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] 37+ messages in thread

* [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: Fix compatible matching
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (15 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-28 22:26   ` Rob Herring
  2020-08-25 19:35 ` [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
                   ` (2 subsequent siblings)
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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 v2:
1. Remove moved compatibles.

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

diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
index e82c2cf9fef7..bd21060d26e0 100644
--- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
+++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
@@ -14,13 +14,16 @@ 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,imx8qm-lpuart
+      - items:
+          - const: fsl,imx8qxp-lpuart
+          - const: fsl,imx7ulp-lpuart
 
   reg:
     maxItems: 1
-- 
2.17.1


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

* [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (16 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-31  3:24   ` Shawn Guo
  2020-08-25 19:35 ` [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
  2020-08-28 12:51 ` [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Linus Walleij
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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] 37+ messages in thread

* [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (17 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
@ 2020-08-25 19:35 ` Krzysztof Kozlowski
  2020-08-31  3:25   ` Shawn Guo
  2020-08-28 12:51 ` [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Linus Walleij
  19 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-25 19:35 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, Mark Rutland,
	Thierry Reding, Anson Huang, 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] 37+ messages in thread

* Re: [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  2020-08-25 19:35 ` [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
@ 2020-08-25 22:21   ` Guenter Roeck
  0 siblings, 0 replies; 37+ messages in thread
From: Guenter Roeck @ 2020-08-25 22:21 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring, Linus Walleij,
	Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, 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 8/25/20 12:35 PM, 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>
> Reviewed-by: Rob Herring <robh@kernel.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
>  .../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
> 


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

* Re: [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-25 19:35 ` [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
@ 2020-08-28  8:45   ` Ulf Hansson
  2020-08-28  9:12     ` Krzysztof Kozlowski
  2020-08-28 21:38   ` Rob Herring
  1 sibling, 1 reply; 37+ messages in thread
From: Ulf Hansson @ 2020-08-28  8:45 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring
  Cc: Linus Walleij, Bartosz Golaszewski, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Miquel Raynal, Mark Rutland, Thierry Reding, Anson Huang,
	Li Yang, Han Xu, Frank Li, Fugang Duan, DTML,
	Linux Kernel Mailing List, open list:GPIO SUBSYSTEM, Linux ARM,
	linux-mmc, linux-mtd, linux-pwm, linux-serial, Linux PM,
	linux-watchdog

On Tue, 25 Aug 2020 at 21:37, Krzysztof Kozlowski <krzk@kernel.org> 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>

Rob, Krzysztof - do you want me to pick this one?

Kind regards
Uffe

>
> ---
>
> Changes since v2:
> 1. Remove moved compatibles.
>
> Changes since v1:
> 1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
> ---
>  .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++++---------
>  1 file changed, 20 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..e71d13c2d109 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> @@ -21,23 +21,26 @@ 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
> +      - 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] 37+ messages in thread

* Re: [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-28  8:45   ` Ulf Hansson
@ 2020-08-28  9:12     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-28  9:12 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, Li Yang, Han Xu, Frank Li, Fugang Duan, DTML,
	Linux Kernel Mailing List, open list:GPIO SUBSYSTEM, Linux ARM,
	linux-mmc, linux-mtd, linux-pwm, linux-serial, Linux PM,
	linux-watchdog

On Fri, Aug 28, 2020 at 10:45:40AM +0200, Ulf Hansson wrote:
> On Tue, 25 Aug 2020 at 21:37, Krzysztof Kozlowski <krzk@kernel.org> 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>
> 
> Rob, Krzysztof - do you want me to pick this one?

dt-bindings are independent so they can be applied individually.

I don't mind you taking it but still Rob's ack/review would be needed.

Other choice is that entire dt-bindings series go through Rob's tree.

Rob, what's your preference?

Best regards,
Krzysztof

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

* Re: [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings
  2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
                   ` (18 preceding siblings ...)
  2020-08-25 19:35 ` [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
@ 2020-08-28 12:51 ` Linus Walleij
  2020-08-28 13:08   ` Krzysztof Kozlowski
  19 siblings, 1 reply; 37+ messages in thread
From: Linus Walleij @ 2020-08-28 12:51 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
  Cc: Rob Herring, Bartosz Golaszewski, Pengutronix Kernel Team,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Mark Rutland,
	Thierry Reding, Anson Huang, Li Yang, Han Xu, Frank Li,
	Fugang Duan,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-kernel, open list:GPIO SUBSYSTEM, Linux ARM, linux-mmc,
	linux-mtd, linux-pwm, open list:SERIAL DRIVERS, Linux PM list,
	LINUXWATCHDOG

On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:

> This is a v3 of cleanup of i.XM 8 bindings and DTSes.

If you are going to be working a lot on Freescale SoC code going forward
I wouldn't mind if you could add yourself as maintainer for the
Freescale pin controller and GPIO at least, I already have high trust
in you in general so if the Freescale maintainers also have that I think you
should just sign up as maintainer. This makes it easier to do pull requests
and things like that.

Yours,
Linus Walleij

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

* Re: [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings
  2020-08-28 12:51 ` [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Linus Walleij
@ 2020-08-28 13:08   ` Krzysztof Kozlowski
  2020-08-28 14:51     ` Linus Walleij
  0 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-28 13:08 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Shawn Guo, Sascha Hauer, Fabio Estevam, Rob Herring,
	Bartosz Golaszewski, Pengutronix Kernel Team, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, Li Yang, Han Xu, Frank Li, Fugang Duan,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-kernel, open list:GPIO SUBSYSTEM, Linux ARM, linux-mmc,
	linux-mtd, linux-pwm, open list:SERIAL DRIVERS, Linux PM list,
	LINUXWATCHDOG

On Fri, Aug 28, 2020 at 02:51:20PM +0200, Linus Walleij wrote:
> On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
> > This is a v3 of cleanup of i.XM 8 bindings and DTSes.
> 
> If you are going to be working a lot on Freescale SoC code going forward
> I wouldn't mind if you could add yourself as maintainer for the
> Freescale pin controller and GPIO at least, I already have high trust
> in you in general so if the Freescale maintainers also have that I think you
> should just sign up as maintainer. This makes it easier to do pull requests
> and things like that.

Thanks for encouragement.  Indeed I am planning to do more work around
i.MX 8M platforms from NXP/Freescale. However there are already four
maintainers for Freescale pin controller drivers so I doubt there is
need for fifth entry :).

Different question is the GPIO driver which apparently lacks entry in
Maintainers file.

Best regards,
Krzysztof


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

* Re: [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings
  2020-08-28 13:08   ` Krzysztof Kozlowski
@ 2020-08-28 14:51     ` Linus Walleij
  0 siblings, 0 replies; 37+ messages in thread
From: Linus Walleij @ 2020-08-28 14:51 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Shawn Guo, Sascha Hauer, Fabio Estevam, Rob Herring,
	Bartosz Golaszewski, Pengutronix Kernel Team, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, Li Yang, Han Xu, Frank Li, Fugang Duan,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-kernel, open list:GPIO SUBSYSTEM, Linux ARM, linux-mmc,
	linux-mtd, linux-pwm, open list:SERIAL DRIVERS, Linux PM list,
	LINUXWATCHDOG

On Fri, Aug 28, 2020 at 3:08 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> On Fri, Aug 28, 2020 at 02:51:20PM +0200, Linus Walleij wrote:
> > On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >
> > > This is a v3 of cleanup of i.XM 8 bindings and DTSes.
> >
> > If you are going to be working a lot on Freescale SoC code going forward
> > I wouldn't mind if you could add yourself as maintainer for the
> > Freescale pin controller and GPIO at least, I already have high trust
> > in you in general so if the Freescale maintainers also have that I think you
> > should just sign up as maintainer. This makes it easier to do pull requests
> > and things like that.
>
> Thanks for encouragement.  Indeed I am planning to do more work around
> i.MX 8M platforms from NXP/Freescale. However there are already four
> maintainers for Freescale pin controller drivers so I doubt there is
> need for fifth entry :).

I beg to differ. As subsystem maintainer it has happened to me more
than once that the i.MX pin control patches went unreviewed for weeks.
Everyone listed for this driver is a kitchen sink maintainer that get
way too much traffic and things fall over the edge.

> Different question is the GPIO driver which apparently lacks entry in
> Maintainers file.

That's probably just an oversight. Feel free to add it!

Yours,
Linus Walleij

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

* Re: [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-25 19:35 ` [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
  2020-08-28  8:45   ` Ulf Hansson
@ 2020-08-28 21:38   ` Rob Herring
  2020-08-29  6:24     ` Krzysztof Kozlowski
  1 sibling, 1 reply; 37+ messages in thread
From: Rob Herring @ 2020-08-28 21:38 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Fabio Estevam, Miquel Raynal, Fugang Duan, linux-pm,
	linux-kernel, linux-gpio, Thierry Reding, Anson Huang, Han Xu,
	linux-serial, Li Yang, Sascha Hauer, Rob Herring,
	linux-arm-kernel, Shawn Guo, Bartosz Golaszewski, Ulf Hansson,
	linux-mmc, linux-mtd, Linus Walleij, Pengutronix Kernel Team,
	NXP Linux Team, Frank Li, Mark Rutland, devicetree, linux-pwm,
	linux-watchdog

On Tue, 25 Aug 2020 21:35:29 +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 v2:
> 1. Remove moved compatibles.
> 
> Changes since v1:
> 1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
> ---
>  .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++++---------
>  1 file changed, 20 insertions(+), 17 deletions(-)
> 


My bot found errors running 'make dt_binding_check' on your patch:

/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: compatible: ['fsl,imx8qxp-usdhc'] is not valid under any of the given schemas (Possible causes of the failure):
	/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: compatible: ['fsl,imx8qxp-usdhc'] is too short
	/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: compatible:0: 'fsl,imx8qxp-usdhc' is not one of ['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']

	From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml


See https://patchwork.ozlabs.org/patch/1351360

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure dt-schema is up to date:

pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade

Please check and re-submit.


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

* Re: [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  2020-08-25 19:35 ` [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
@ 2020-08-28 22:26   ` Rob Herring
  0 siblings, 0 replies; 37+ messages in thread
From: Rob Herring @ 2020-08-28 22:26 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Shawn Guo, Fabio Estevam, Rob Herring, Han Xu, linux-kernel,
	Frank Li, Bartosz Golaszewski, linux-watchdog, Sascha Hauer,
	linux-pm, linux-mtd, linux-mmc, Li Yang, Linus Walleij,
	devicetree, Thierry Reding, linux-arm-kernel, Anson Huang,
	NXP Linux Team, Fugang Duan, Mark Rutland, Miquel Raynal,
	linux-gpio, Ulf Hansson, linux-serial, linux-pwm,
	Pengutronix Kernel Team

On Tue, 25 Aug 2020 21:35:30 +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>
> 
> ---
> 
> Changes since v2:
> 1. Remove moved compatibles.
> ---
>  .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 38 +++++++++++--------
>  1 file changed, 23 insertions(+), 15 deletions(-)
> 

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

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

* Re: [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: Fix compatible matching
  2020-08-25 19:35 ` [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
@ 2020-08-28 22:26   ` Rob Herring
  0 siblings, 0 replies; 37+ messages in thread
From: Rob Herring @ 2020-08-28 22:26 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-arm-kernel, Rob Herring, Miquel Raynal, linux-kernel,
	Fugang Duan, Thierry Reding, Mark Rutland, Frank Li,
	Linus Walleij, linux-pm, devicetree, linux-mtd, Sascha Hauer,
	NXP Linux Team, Pengutronix Kernel Team, linux-watchdog,
	linux-gpio, Ulf Hansson, Anson Huang, linux-pwm, Han Xu,
	Bartosz Golaszewski, linux-serial, linux-mmc, Fabio Estevam,
	Li Yang, Shawn Guo

On Tue, 25 Aug 2020 21:35:34 +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 v2:
> 1. Remove moved compatibles.
> 
> Changes since v1:
> 1. New patch.
> ---
>  .../devicetree/bindings/serial/fsl-lpuart.yaml  | 17 ++++++++++-------
>  1 file changed, 10 insertions(+), 7 deletions(-)
> 

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

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

* Re: [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  2020-08-28 21:38   ` Rob Herring
@ 2020-08-29  6:24     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-29  6:24 UTC (permalink / raw)
  To: Rob Herring
  Cc: Fabio Estevam, Miquel Raynal, Fugang Duan, linux-pm,
	linux-kernel, linux-gpio, Thierry Reding, Anson Huang, Han Xu,
	linux-serial, Li Yang, Sascha Hauer, Rob Herring,
	linux-arm-kernel, Shawn Guo, Bartosz Golaszewski, Ulf Hansson,
	linux-mmc, linux-mtd, Linus Walleij, Pengutronix Kernel Team,
	NXP Linux Team, Frank Li, Mark Rutland, devicetree, linux-pwm,
	linux-watchdog

On Fri, 28 Aug 2020 at 23:38, Rob Herring <robh@kernel.org> wrote:
>
> On Tue, 25 Aug 2020 21:35:29 +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 v2:
> > 1. Remove moved compatibles.
> >
> > Changes since v1:
> > 1. Handle also fsl,imx8mm-usdhc and fsl,imx8qxp-usdhc
> > ---
> >  .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++++---------
> >  1 file changed, 20 insertions(+), 17 deletions(-)
> >
>
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: compatible: ['fsl,imx8qxp-usdhc'] is not valid under any of the given schemas (Possible causes of the failure):
>         /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/imx8qxp-lpcg.example.dt.yaml: mmc@5b010000: compatible: ['fsl,imx8qxp-usdhc'] is too short

I will send a v4 just for this one patch. All others seem to be good.

Best regards,
Krzysztof

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

* Re: [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  2020-08-25 19:35 ` [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
@ 2020-08-31  3:22   ` Shawn Guo
  0 siblings, 0 replies; 37+ messages in thread
From: Shawn Guo @ 2020-08-31  3:22 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, 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 09:35:31PM +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>
> Reviewed-by: Rob Herring <robh@kernel.org>

Applied, thanks.

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

* Re: [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding
  2020-08-25 19:35 ` [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
@ 2020-08-31  3:22   ` Shawn Guo
  0 siblings, 0 replies; 37+ messages in thread
From: Shawn Guo @ 2020-08-31  3:22 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, 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 09:35:32PM +0200, Krzysztof Kozlowski wrote:
> Document the binding for Zodiac Inflight Innovations Ultra Boards.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Reviewed-by: Rob Herring <robh@kernel.org>

Applied, thanks.

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

* Re: [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on
  2020-08-25 19:35 ` [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
@ 2020-08-31  3:24   ` Shawn Guo
  0 siblings, 0 replies; 37+ messages in thread
From: Shawn Guo @ 2020-08-31  3:24 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, 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 09:35:35PM +0200, Krzysztof Kozlowski wrote:
> 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>

Applied, thanks.

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

* Re: [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs
  2020-08-25 19:35 ` [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
@ 2020-08-31  3:25   ` Shawn Guo
  0 siblings, 0 replies; 37+ messages in thread
From: Shawn Guo @ 2020-08-31  3:25 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Ulf Hansson, Miquel Raynal, Mark Rutland, Thierry Reding,
	Anson Huang, 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 09:35:36PM +0200, Krzysztof Kozlowski wrote:
> 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>

Applied, thanks.

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

* Re: [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-08-25 19:35 ` [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
@ 2020-09-23 11:52   ` Thierry Reding
  2020-09-23 14:08     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 37+ messages in thread
From: Thierry Reding @ 2020-09-23 11:52 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Mark Rutland,
	Anson Huang, 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

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

On Tue, Aug 25, 2020 at 09:35:23PM +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>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* Re: [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-09-23 11:52   ` Thierry Reding
@ 2020-09-23 14:08     ` Krzysztof Kozlowski
  2020-09-23 14:15       ` Thierry Reding
  0 siblings, 1 reply; 37+ messages in thread
From: Krzysztof Kozlowski @ 2020-09-23 14:08 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Mark Rutland,
	Anson Huang, 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 Wed, 23 Sep 2020 at 13:52, Thierry Reding <thierry.reding@gmail.com> wrote:
>
> On Tue, Aug 25, 2020 at 09:35:23PM +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>
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > ---
> >  Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 14 +++++++++++---
> >  1 file changed, 11 insertions(+), 3 deletions(-)
>
> Applied, thanks.

Thanks Thierry, but this was already picked up by Rob into DT tree.

Best regards,
Krzysztof

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

* Re: [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  2020-09-23 14:08     ` Krzysztof Kozlowski
@ 2020-09-23 14:15       ` Thierry Reding
  0 siblings, 0 replies; 37+ messages in thread
From: Thierry Reding @ 2020-09-23 14:15 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Linus Walleij, Bartosz Golaszewski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Ulf Hansson, Miquel Raynal, Mark Rutland,
	Anson Huang, 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

[-- Attachment #1: Type: text/plain, Size: 1445 bytes --]

On Wed, Sep 23, 2020 at 04:08:53PM +0200, Krzysztof Kozlowski wrote:
> On Wed, 23 Sep 2020 at 13:52, Thierry Reding <thierry.reding@gmail.com> wrote:
> >
> > On Tue, Aug 25, 2020 at 09:35:23PM +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>
> > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > ---
> > >  Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 14 +++++++++++---
> > >  1 file changed, 11 insertions(+), 3 deletions(-)
> >
> > Applied, thanks.
> 
> Thanks Thierry, but this was already picked up by Rob into DT tree.

I hadn't seen any email and since Rob had given a reviewed-by I assumed
this was supposed to be picked up into subsystem trees. I'll drop it
again.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2020-09-23 14:15 UTC | newest]

Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-25 19:35 [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 01/19] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 02/19] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 03/19] dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 04/19] dt-bindings: gpio: fsl-imx-gpio: Add power-domains Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 05/19] dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 06/19] dt-bindings: pwm: imx-pwm: " Krzysztof Kozlowski
2020-09-23 11:52   ` Thierry Reding
2020-09-23 14:08     ` Krzysztof Kozlowski
2020-09-23 14:15       ` Thierry Reding
2020-08-25 19:35 ` [PATCH v3 07/19] dt-bindings: serial: fsl-imx-uart: " Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 08/19] dt-bindings: watchdog: fsl-imx-wdt: " Krzysztof Kozlowski
2020-08-25 22:21   ` Guenter Roeck
2020-08-25 19:35 ` [PATCH v3 09/19] dt-bindings: mtd: gpmi-nand: " Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 10/19] dt-bindings: reset: fsl,imx7-src: " Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 11/19] dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 12/19] dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching Krzysztof Kozlowski
2020-08-28  8:45   ` Ulf Hansson
2020-08-28  9:12     ` Krzysztof Kozlowski
2020-08-28 21:38   ` Rob Herring
2020-08-29  6:24     ` Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 13/19] dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles Krzysztof Kozlowski
2020-08-28 22:26   ` Rob Herring
2020-08-25 19:35 ` [PATCH v3 14/19] dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding Krzysztof Kozlowski
2020-08-31  3:22   ` Shawn Guo
2020-08-25 19:35 ` [PATCH v3 15/19] dt-bindings: arm: fsl: Add ZII Ultra boards binding Krzysztof Kozlowski
2020-08-31  3:22   ` Shawn Guo
2020-08-25 19:35 ` [PATCH v3 16/19] dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible matching Krzysztof Kozlowski
2020-08-25 19:35 ` [PATCH v3 17/19] dt-bindings: serial: fsl-lpuart: " Krzysztof Kozlowski
2020-08-28 22:26   ` Rob Herring
2020-08-25 19:35 ` [PATCH v3 18/19] arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on Krzysztof Kozlowski
2020-08-31  3:24   ` Shawn Guo
2020-08-25 19:35 ` [PATCH v3 19/19] arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs Krzysztof Kozlowski
2020-08-31  3:25   ` Shawn Guo
2020-08-28 12:51 ` [PATCH v3 00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings Linus Walleij
2020-08-28 13:08   ` Krzysztof Kozlowski
2020-08-28 14:51     ` Linus Walleij

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).