On Fri, Aug 27, 2021 at 02:42:59PM +0800, Richard Zhu wrote: > - ranges property should be grouped by region, with no functional > changes. Otherwise, schema dtbs_check would report the following errors. > "linux-imx/arch/arm/boot/dts/imx6qp-vicutp.dt.yaml: pcie@1ffc000: ranges: 'oneOf' conditional failed, one must be fixed: > linux-imx/arch/arm/boot/dts/imx6qp-vicutp.dt.yaml: pcie@1ffc000: ranges: 'oneOf' conditional failed, one must be fixed: > [[2164260864, 0, 0, 33030144, 0, 65536, 2181038080, 0, 16777216, 16777216, 0, 15728640]] is not of type 'boolean' > True was expected > [[2164260864, 0, 0, 33030144, 0, 65536, 2181038080, 0, 16777216, 16777216, 0, 15728640]] is not of type 'null' > [2164260864, 0, 0, 33030144, 0, 65536, 2181038080, 0, 16777216, 16777216, 0, 15728640] is too long > From schema: linux-imx/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml" > - refer to commit 281f1f99cf3a ("PCI: dwc: Detect number of iATU windows"). > The num-viewport is not required anymore, remove them totally. > - dt_binding_check complains "compatible: ['fsl,imx6qp-pcie', 'snps,dw-pcie'] > is too long", remove "snps,dw-pcie" from the compatible string. > > Signed-off-by: Richard Zhu <hongxing.zhu@xxxxxxx> > Reviewed-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx> Applied, thanks!