Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <5fcaf73e-50a6-9530-8aeb-180219dc6573@linaro.org>
Date:   Tue, 16 Aug 2022 13:11:54 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.12.0
Subject: Re: [PATCH v4 2/2] arm64: dts: imx8mp-msc-sm2s: Add device trees for
 MSC SM2S-IMX8PLUS SoM and carrier board
Content-Language: en-US
To:     Martyn Welch <martyn.welch@collabora.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>
Cc:     kernel@collabora.com, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
References: <20220812084120.376042-1-martyn.welch@collabora.com>
 <20220812084120.376042-2-martyn.welch@collabora.com>
 <8962b7ed-a21c-0b7f-7a6d-5db3db84e4cb@linaro.org>
 <65a094d5d03ad8f7b35196c9dff6ffc6cf0ea151.camel@collabora.com>
 <15ddd798-873e-d90d-11e9-c6dd46ca03f4@linaro.org>
 <c874e1db8526bfa915baca1f0bb28d0c5f5a1feb.camel@collabora.com>
 <993a4559-c9f4-c6e2-8854-d2018180fd6a@linaro.org>
 <dc944acacf64f19b46e1d865d9608853b44a686a.camel@collabora.com>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <dc944acacf64f19b46e1d865d9608853b44a686a.camel@collabora.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 15/08/2022 10:45, Martyn Welch wrote:
> 	From schema: /home/martyn/Documents/geh0003/linux-
> upstream/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml
> /home/martyn/Documents/geh0003/linux-
> upstream/arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb:
> usb@32f10108: 'power-domains' does not match any of the regexes:
> '^usb@[0-9a-f]+$', 'pinctrl-[0-9]+'
> 	From schema: /home/martyn/Documents/geh0003/linux-
> upstream/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml
> arch/arm64/boot/dts/freescale/imx8mp-msc-sm2s-ep1.dtb:0:0: /extcon-
> usb0: failed to match any schema with compatible: ['linux,extcon-usb-
> gpio']
> 
> 
> Unless I'm missing something, that's all from dtsi I'm including, not
> from the files I'm trying to add (with the possible exception of the
> tools not understanding that pinctrl nodes might have optional entities
> in them.
> 
> Do you have patch 1 of the series in your tree?:
> 
> https://lore.kernel.org/linux-devicetree/20220812084120.376042-1-martyn.welch@collabora.com/
> 
> I thought that was exactly what that was doing... 

I have it and when you read it you should notice it won't work for your
case. You define one set of compatibles and implement something a bit
else. Regardless whether you see the error or not, the bindings must
match your DTS. Here anyway the DTS will be gone, so no errors...

Best regards,
Krzysztof