diff mbox series

[1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding

Message ID 20221020103158.2273874-2-peng.fan@oss.nxp.com
State New
Headers show
Series [1/2] dt-bindings: spi: fsl-imx-cspi: update i.MX8MP binding | expand

Commit Message

Peng Fan (OSS) Oct. 20, 2022, 10:31 a.m. UTC
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(-)

Comments

Peng Fan Oct. 21, 2022, 1:46 a.m. UTC | #1
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
diff mbox series

Patch

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: