[PATCH 0/2] imx8mp: spi: update binding and compatible

devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH 0/2] imx8mp: spi: update binding and compatible
@ 2022-10-20 10:31 Peng Fan (OSS)
  2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Peng Fan (OSS) @ 2022-10-20 10:31 UTC (permalink / raw)
  To: broonie, robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
  Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
	linux-arm-kernel, linux-spi, Peng Fan

From: Peng Fan <peng.fan@nxp.com>

i.MX8MP ECSPI reused from i.MX6UL, so update binding and dts

Peng Fan (2):
  dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
  arm64: dts: imx8mp: update ecspi compatible and clk

 .../devicetree/bindings/spi/fsl-imx-cspi.yaml     |  4 +++-
 arch/arm64/boot/dts/freescale/imx8mp.dtsi         | 15 ++++++++++++---
 2 files changed, 15 insertions(+), 4 deletions(-)

-- 
2.37.1


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

* [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
  2022-10-20 10:31 [PATCH 0/2] imx8mp: spi: update binding and compatible Peng Fan (OSS)
@ 2022-10-20 10:31 ` Peng Fan (OSS)
  2022-10-20 12:56   ` Rob Herring
  2022-10-21  2:23   ` Rob Herring
  2022-10-20 10:31 ` [PATCH 2/2] arm64: dts: imx8mp: update ecspi compatible and clk Peng Fan (OSS)
  2022-10-29  8:09 ` [PATCH 0/2] imx8mp: spi: update binding and compatible Shawn Guo
  2 siblings, 2 replies; 7+ messages in thread
From: Peng Fan (OSS) @ 2022-10-20 10:31 UTC (permalink / raw)
  To: broonie, robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
  Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
	linux-arm-kernel, linux-spi, Peng Fan

From: Peng Fan <peng.fan@nxp.com>

i.MX8MP ECSPI is derived from i.MX6UL, so update the binding.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml b/Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml
index 50df1a40bbe3..12cb76711000 100644
--- a/Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml
+++ b/Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml
@@ -22,6 +22,9 @@ properties:
       - const: fsl,imx35-cspi
       - const: fsl,imx51-ecspi
       - const: fsl,imx53-ecspi
+      - items:
+          - const: fsl,imx8mp-ecspi
+          - const: fsl,imx6ul-ecspi
       - items:
           - enum:
               - fsl,imx50-ecspi
@@ -34,7 +37,6 @@ properties:
               - fsl,imx8mq-ecspi
               - fsl,imx8mm-ecspi
               - fsl,imx8mn-ecspi
-              - fsl,imx8mp-ecspi
           - const: fsl,imx51-ecspi
 
   reg:
-- 
2.37.1


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

* Re: [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
  2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
@ 2022-10-20 12:56   ` Rob Herring
  2022-10-21  1:46     ` Peng Fan
  2022-10-21  2:23   ` Rob Herring
  1 sibling, 1 reply; 7+ messages in thread
From: Rob Herring @ 2022-10-20 12:56 UTC (permalink / raw)
  To: Peng Fan (OSS)
  Cc: Peng Fan, krzysztof.kozlowski+dt, kernel, linux-kernel, broonie,
	devicetree, shawnguo, festevam, linux-arm-kernel, linux-imx,
	linux-spi, s.hauer, robh+dt

On Thu, 20 Oct 2022 18:31:57 +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP ECSPI is derived from i.MX6UL, so update the binding.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/


spi@30630000: Unevaluated properties are not allowed ('num-chipselects' was unexpected)
	arch/arm/boot/dts/imx7d-flex-concentrator.dtb
	arch/arm/boot/dts/imx7d-flex-concentrator-mfg.dtb

spi@30820000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@30820000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@30830000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@30830000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@30830000: Unevaluated properties are not allowed ('num-chipselects' was unexpected)
	arch/arm/boot/dts/imx7d-flex-concentrator.dtb
	arch/arm/boot/dts/imx7d-flex-concentrator-mfg.dtb

spi@30840000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@30840000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-edimm2.2.dtb
	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dtb
	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb

spi@43fa4000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb

spi@43fa4000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb

spi@50004000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb

spi@50004000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb

spi@50010000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@50010000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-qvga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-svga.dtb
	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-vga.dtb
	arch/arm/boot/dts/imx25-karo-tx25.dtb
	arch/arm/boot/dts/imx25-pdk.dtb
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@63fac000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@63fac000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@63fc0000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx50-evk.dtb
	arch/arm/boot/dts/imx50-kobo-aura.dtb
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@63fc0000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx50-evk.dtb
	arch/arm/boot/dts/imx50-kobo-aura.dtb
	arch/arm/boot/dts/imx53-ard.dtb
	arch/arm/boot/dts/imx53-cx9020.dtb
	arch/arm/boot/dts/imx53-kp-ddc.dtb
	arch/arm/boot/dts/imx53-kp-hsc.dtb
	arch/arm/boot/dts/imx53-m53evk.dtb
	arch/arm/boot/dts/imx53-m53menlo.dtb
	arch/arm/boot/dts/imx53-mba53.dtb
	arch/arm/boot/dts/imx53-ppd.dtb
	arch/arm/boot/dts/imx53-qsb.dtb
	arch/arm/boot/dts/imx53-qsrb.dtb
	arch/arm/boot/dts/imx53-smd.dtb
	arch/arm/boot/dts/imx53-tx53-x03x.dtb
	arch/arm/boot/dts/imx53-tx53-x13x.dtb
	arch/arm/boot/dts/imx53-usbarmory.dtb
	arch/arm/boot/dts/imx53-voipac-bsb.dtb

spi@83fc0000: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm/boot/dts/imx51-apf51dev.dtb
	arch/arm/boot/dts/imx51-apf51.dtb
	arch/arm/boot/dts/imx51-babbage.dtb
	arch/arm/boot/dts/imx51-digi-connectcore-jsk.dtb
	arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dtb
	arch/arm/boot/dts/imx51-ts4800.dtb
	arch/arm/boot/dts/imx51-zii-rdu1.dtb
	arch/arm/boot/dts/imx51-zii-scu2-mezz.dtb
	arch/arm/boot/dts/imx51-zii-scu3-esb.dtb

spi@83fc0000: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/imx51-apf51dev.dtb
	arch/arm/boot/dts/imx51-apf51.dtb
	arch/arm/boot/dts/imx51-babbage.dtb
	arch/arm/boot/dts/imx51-digi-connectcore-jsk.dtb
	arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dtb
	arch/arm/boot/dts/imx51-ts4800.dtb
	arch/arm/boot/dts/imx51-zii-rdu1.dtb
	arch/arm/boot/dts/imx51-zii-scu2-mezz.dtb
	arch/arm/boot/dts/imx51-zii-scu3-esb.dtb


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

* RE: [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
  2022-10-20 12:56   ` Rob Herring
@ 2022-10-21  1:46     ` Peng Fan
  0 siblings, 0 replies; 7+ messages in thread
From: Peng Fan @ 2022-10-21  1:46 UTC (permalink / raw)
  To: Rob Herring, Peng Fan (OSS)
  Cc: krzysztof.kozlowski+dt@linaro.org, kernel@pengutronix.de,
	linux-kernel@vger.kernel.org, broonie@kernel.org,
	devicetree@vger.kernel.org, shawnguo@kernel.org,
	festevam@gmail.com, linux-arm-kernel@lists.infradead.org,
	dl-linux-imx, linux-spi@vger.kernel.org, s.hauer@pengutronix.de,
	robh+dt@kernel.org

Hi Rob,

> Subject: Re: [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP
> binding
> 
> On Thu, 20 Oct 2022 18:31:57 +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > i.MX8MP ECSPI is derived from i.MX6UL, so update the binding.
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> 
> Running 'make dtbs_check' with the schema in this patch gives the following
> warnings. Consider if they are expected or the schema is incorrect. These
> may not be new warnings.

Patch 2/2 updates the compatible string which could resolved the dtbs_check
failure for i.MX8MP boards.

Thanks,
Peng.

> 
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
> 
> Full log is available here:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> hwork.ozlabs.org%2Fpatch%2F&amp;data=05%7C01%7Cpeng.fan%40nxp.c
> om%7Cbd5f3e7566774254285308dab29a7eb5%7C686ea1d3bc2b4c6fa92cd
> 99c5c301635%7C0%7C1%7C638018673864797750%7CUnknown%7CTWFpb
> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI
> 6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=ZawGt%2BSCPrZkiNU3WYRy%
> 2F2JqCVY%2FtRWfVC%2BCTRcC0Fs%3D&amp;reserved=0
> 
> 
> spi@30630000: Unevaluated properties are not allowed ('num-chipselects'
> was unexpected)
> 	arch/arm/boot/dts/imx7d-flex-concentrator.dtb
> 	arch/arm/boot/dts/imx7d-flex-concentrator-mfg.dtb
> 
> spi@30820000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@30820000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@30830000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@30830000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@30830000: Unevaluated properties are not allowed ('num-chipselects'
> was unexpected)
> 	arch/arm/boot/dts/imx7d-flex-concentrator.dtb
> 	arch/arm/boot/dts/imx7d-flex-concentrator-mfg.dtb
> 
> spi@30840000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@30840000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm64/boot/dts/freescale/imx8mp-dhcom-pdk2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-evk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-icore-mx8mp-
> edimm2.2.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-
> mba8mpxl.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-nonwifi-dev.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dahlia.dtb
> 	arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi-dev.dtb
> 
> spi@43fa4000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 
> spi@43fa4000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 
> spi@50004000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 
> spi@50004000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 
> spi@50010000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@50010000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-cmo-
> qvga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> svga.dtb
> 	arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard-dvi-
> vga.dtb
> 	arch/arm/boot/dts/imx25-karo-tx25.dtb
> 	arch/arm/boot/dts/imx25-pdk.dtb
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@63fac000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@63fac000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@63fc0000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx50-evk.dtb
> 	arch/arm/boot/dts/imx50-kobo-aura.dtb
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@63fc0000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx50-evk.dtb
> 	arch/arm/boot/dts/imx50-kobo-aura.dtb
> 	arch/arm/boot/dts/imx53-ard.dtb
> 	arch/arm/boot/dts/imx53-cx9020.dtb
> 	arch/arm/boot/dts/imx53-kp-ddc.dtb
> 	arch/arm/boot/dts/imx53-kp-hsc.dtb
> 	arch/arm/boot/dts/imx53-m53evk.dtb
> 	arch/arm/boot/dts/imx53-m53menlo.dtb
> 	arch/arm/boot/dts/imx53-mba53.dtb
> 	arch/arm/boot/dts/imx53-ppd.dtb
> 	arch/arm/boot/dts/imx53-qsb.dtb
> 	arch/arm/boot/dts/imx53-qsrb.dtb
> 	arch/arm/boot/dts/imx53-smd.dtb
> 	arch/arm/boot/dts/imx53-tx53-x03x.dtb
> 	arch/arm/boot/dts/imx53-tx53-x13x.dtb
> 	arch/arm/boot/dts/imx53-usbarmory.dtb
> 	arch/arm/boot/dts/imx53-voipac-bsb.dtb
> 
> spi@83fc0000: compatible: 'oneOf' conditional failed, one must be fixed:
> 	arch/arm/boot/dts/imx51-apf51dev.dtb
> 	arch/arm/boot/dts/imx51-apf51.dtb
> 	arch/arm/boot/dts/imx51-babbage.dtb
> 	arch/arm/boot/dts/imx51-digi-connectcore-jsk.dtb
> 	arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dtb
> 	arch/arm/boot/dts/imx51-ts4800.dtb
> 	arch/arm/boot/dts/imx51-zii-rdu1.dtb
> 	arch/arm/boot/dts/imx51-zii-scu2-mezz.dtb
> 	arch/arm/boot/dts/imx51-zii-scu3-esb.dtb
> 
> spi@83fc0000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
> 	arch/arm/boot/dts/imx51-apf51dev.dtb
> 	arch/arm/boot/dts/imx51-apf51.dtb
> 	arch/arm/boot/dts/imx51-babbage.dtb
> 	arch/arm/boot/dts/imx51-digi-connectcore-jsk.dtb
> 	arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dtb
> 	arch/arm/boot/dts/imx51-ts4800.dtb
> 	arch/arm/boot/dts/imx51-zii-rdu1.dtb
> 	arch/arm/boot/dts/imx51-zii-scu2-mezz.dtb
> 	arch/arm/boot/dts/imx51-zii-scu3-esb.dtb


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

* Re: [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
  2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
  2022-10-20 12:56   ` Rob Herring
@ 2022-10-21  2:23   ` Rob Herring
  1 sibling, 0 replies; 7+ messages in thread
From: Rob Herring @ 2022-10-21  2:23 UTC (permalink / raw)
  To: Peng Fan (OSS)
  Cc: Peng Fan, linux-imx, linux-arm-kernel, broonie, shawnguo,
	krzysztof.kozlowski+dt, s.hauer, festevam, linux-kernel, kernel,
	devicetree, robh+dt, linux-spi

On Thu, 20 Oct 2022 18:31:57 +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP ECSPI is derived from i.MX6UL, so update the binding.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  Documentation/devicetree/bindings/spi/fsl-imx-cspi.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

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

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

* [PATCH 2/2] arm64: dts: imx8mp: update ecspi compatible and clk
  2022-10-20 10:31 [PATCH 0/2] imx8mp: spi: update binding and compatible Peng Fan (OSS)
  2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
@ 2022-10-20 10:31 ` Peng Fan (OSS)
  2022-10-29  8:09 ` [PATCH 0/2] imx8mp: spi: update binding and compatible Shawn Guo
  2 siblings, 0 replies; 7+ messages in thread
From: Peng Fan (OSS) @ 2022-10-20 10:31 UTC (permalink / raw)
  To: broonie, robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
  Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
	linux-arm-kernel, linux-spi, Peng Fan, Clark Wang

From: Peng Fan <peng.fan@nxp.com>

i.MX8MP ECSPI is derived from i.MX6UL, so update compatible
Add assigned-clocks settings

Signed-off-by: Clark Wang <xiaoning.wang@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mp.dtsi | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index 732a87179edd..315902fa34c3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -713,12 +713,15 @@ aips3: bus@30800000 {
 			ecspi1: spi@30820000 {
 				#address-cells = <1>;
 				#size-cells = <0>;
-				compatible = "fsl,imx8mp-ecspi", "fsl,imx51-ecspi";
+				compatible = "fsl,imx8mp-ecspi", "fsl,imx6ul-ecspi";
 				reg = <0x30820000 0x10000>;
 				interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>;
 				clocks = <&clk IMX8MP_CLK_ECSPI1_ROOT>,
 					 <&clk IMX8MP_CLK_ECSPI1_ROOT>;
 				clock-names = "ipg", "per";
+				assigned-clock-rates = <80000000>;
+				assigned-clocks = <&clk IMX8MP_CLK_ECSPI1>;
+				assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
 				dmas = <&sdma1 0 7 1>, <&sdma1 1 7 2>;
 				dma-names = "rx", "tx";
 				status = "disabled";
@@ -727,12 +730,15 @@ ecspi1: spi@30820000 {
 			ecspi2: spi@30830000 {
 				#address-cells = <1>;
 				#size-cells = <0>;
-				compatible = "fsl,imx8mp-ecspi", "fsl,imx51-ecspi";
+				compatible = "fsl,imx8mp-ecspi", "fsl,imx6ul-ecspi";
 				reg = <0x30830000 0x10000>;
 				interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>;
 				clocks = <&clk IMX8MP_CLK_ECSPI2_ROOT>,
 					 <&clk IMX8MP_CLK_ECSPI2_ROOT>;
 				clock-names = "ipg", "per";
+				assigned-clock-rates = <80000000>;
+				assigned-clocks = <&clk IMX8MP_CLK_ECSPI2>;
+				assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
 				dmas = <&sdma1 2 7 1>, <&sdma1 3 7 2>;
 				dma-names = "rx", "tx";
 				status = "disabled";
@@ -741,12 +747,15 @@ ecspi2: spi@30830000 {
 			ecspi3: spi@30840000 {
 				#address-cells = <1>;
 				#size-cells = <0>;
-				compatible = "fsl,imx8mp-ecspi", "fsl,imx51-ecspi";
+				compatible = "fsl,imx8mp-ecspi", "fsl,imx6ul-ecspi";
 				reg = <0x30840000 0x10000>;
 				interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>;
 				clocks = <&clk IMX8MP_CLK_ECSPI3_ROOT>,
 					 <&clk IMX8MP_CLK_ECSPI3_ROOT>;
 				clock-names = "ipg", "per";
+				assigned-clock-rates = <80000000>;
+				assigned-clocks = <&clk IMX8MP_CLK_ECSPI3>;
+				assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
 				dmas = <&sdma1 4 7 1>, <&sdma1 5 7 2>;
 				dma-names = "rx", "tx";
 				status = "disabled";
-- 
2.37.1


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

* Re: [PATCH 0/2] imx8mp: spi: update binding and compatible
  2022-10-20 10:31 [PATCH 0/2] imx8mp: spi: update binding and compatible Peng Fan (OSS)
  2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
  2022-10-20 10:31 ` [PATCH 2/2] arm64: dts: imx8mp: update ecspi compatible and clk Peng Fan (OSS)
@ 2022-10-29  8:09 ` Shawn Guo
  2 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2022-10-29  8:09 UTC (permalink / raw)
  To: Peng Fan (OSS)
  Cc: broonie, robh+dt, krzysztof.kozlowski+dt, s.hauer, kernel,
	festevam, linux-imx, devicetree, linux-kernel, linux-arm-kernel,
	linux-spi, Peng Fan

On Thu, Oct 20, 2022 at 06:31:56PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP ECSPI reused from i.MX6UL, so update binding and dts
> 
> Peng Fan (2):
>   dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding
>   arm64: dts: imx8mp: update ecspi compatible and clk

Applied both, thanks!

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

end of thread, other threads:[~2022-10-29  8:09 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-20 10:31 [PATCH 0/2] imx8mp: spi: update binding and compatible Peng Fan (OSS)
2022-10-20 10:31 ` [PATCH 1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding Peng Fan (OSS)
2022-10-20 12:56   ` Rob Herring
2022-10-21  1:46     ` Peng Fan
2022-10-21  2:23   ` Rob Herring
2022-10-20 10:31 ` [PATCH 2/2] arm64: dts: imx8mp: update ecspi compatible and clk Peng Fan (OSS)
2022-10-29  8:09 ` [PATCH 0/2] imx8mp: spi: update binding and compatible Shawn Guo

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