Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
From:   Peng Fan <peng.fan@nxp.com>
To:     Shawn Guo <shawnguo@kernel.org>,
        "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
CC:     "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>
Subject: RE: [PATCH V2 0/3] arm64: add i.MX93 support
Thread-Topic: [PATCH V2 0/3] arm64: add i.MX93 support
Thread-Index: AQHYXulHF+y34Ra6sk+O6vv1l7ZMaK0P9baAgAAAvkA=
Date:   Thu, 5 May 2022 08:29:18 +0000
Message-ID: <DU0PR04MB94173D505B2428714FD0968588C29@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220503122951.4147636-1-peng.fan@oss.nxp.com>
 <20220505082607.GE14615@dragon>
In-Reply-To: <20220505082607.GE14615@dragon>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?nveXB/zj+RC4wNpYcnQd522S4QNsraFyrUq3Od6qajPFgyAhmLv6zg8cD6Vz?=
 =?us-ascii?Q?1M4li68fueFHuNSffFNwzK44nLINgIb/hHitSSEAveRuW8yXXbFYOsm6IFaT?=
 =?us-ascii?Q?vDjyd3CKHZ9Z4VwHiupWadNESuxdrmPQfDV61FlAYVV71/0rA7FEIunOVjGv?=
 =?us-ascii?Q?KJbXjTyCTWI+E9JFOcsfLuXW3ntHRPwOPUTC3Iq94ix+qB721pesbP9XFQ51?=
 =?us-ascii?Q?6RHNvRP2OFR7Jcvtmc5HDwN/TrRs7y0yrzJ2wH2XIrt9UyEO1FHG0hnuAzhc?=
 =?us-ascii?Q?N5s6uqksEh71tbbZU57NdHYvS4lZKOwr4x02F2bXLRmqqEHW9Dl0ubZprHgo?=
 =?us-ascii?Q?O3OCMue2ztzxes0APnxhkRIqDtKQOUk6QWadAYO/wYYKvc+XKLm442XbkCfY?=
 =?us-ascii?Q?iL4R33e0p2SVbSbB7G2K4FZbu53uikgEdR88wGu569++wJAE+lS8Mc2wF59H?=
 =?us-ascii?Q?oTg6uwrdiUfS+OwKpcSZ7opv8JJH2kWWYLu09asYBsl4H7cRCzBH7s9V6BFl?=
 =?us-ascii?Q?jkb8SFICm8Gz8RBlUV2o+qHPDF0YWMPWRjN6FPNzg1bvI9VSeiwXhncPD4Ox?=
 =?us-ascii?Q?18m/AHg8SaQQmmH11wInX3HP9HhpXRtQC+CJeppOXkidrY1KdvStcXOlx8X+?=
 =?us-ascii?Q?YGXG1HyLxZ1PVb+iEQwP7GHsfhKhVqFVltmZ04i/maPP51HTJHpxq38n87CZ?=
 =?us-ascii?Q?sxTYYde/GTYGC3NsPM4W6NNvSbvfhBIE1MY4VjjLfRDen/hhgzCIKROYGon+?=
 =?us-ascii?Q?MSWoMXrevCCYkMqk1y2/F5Sp2NsgaxYU3IKL3mpL4xwsU2HJ+4yUh0p2jIo9?=
 =?us-ascii?Q?kwowvgE4rx6Z5f6V9OFxIMD5VyfHQeajV7I1qY7EAZcnzCf5ZEFuWQsKYZXV?=
 =?us-ascii?Q?uV3va5zu0F0apQ9C/MYKoAiZVL9npoFh+8HXAdXinzA5mXVOOuZDOWXibZ4G?=
 =?us-ascii?Q?DWZ8r0hvcS5T4ADJakoY84kYNTu+3IU/nnLWvL+Vb4iolzuWMISnujwoGMIz?=
 =?us-ascii?Q?SEF8tkgsXVAJOLDS4LE2AYG2XhqHKzmho4rdarbB7e4n5VAk9JKeAuW9zsrT?=
 =?us-ascii?Q?iRWSVeW6qBg2Okb3nE5TfzbF9kuDrLaa3YznRy+65eiHqy7uL9oFcmvUwVrO?=
 =?us-ascii?Q?f0TYab5SDxlSQpsmNosGExS3wQW4Sj8DntGuxotwuHwyn1bbyql5cunIJZEK?=
 =?us-ascii?Q?1MmoTfrO67VUjSnzW81S0d9LoHsyO3EA6TqaIy9srFJDMm5SVAte6uppM0gE?=
 =?us-ascii?Q?lk6iGZxQQdWLqLEBjW0zj6kEOwH/YsHvU1YpSPWmoh0BJz+AoHCFJWZvOKFl?=
 =?us-ascii?Q?HpGwzWJXrUC2gO2PEodo4JAX74JvhZYdN7daLE8+BQWK5/4QsGFHRizLqp+X?=
 =?us-ascii?Q?PeV/0OVprUnhniibg5BkGb/AO75q2ltJq89tHd+zs5ofg4ifIpW8omOd+UEp?=
 =?us-ascii?Q?8rYi8c2M4wTjgH9uFiZcIgdw697BTNbW/5Nan1Oop07IKh9LUHXKErOyGxNm?=
 =?us-ascii?Q?m0cNZe+YcpVZwKHpSnYiJxdyaL2Yh9mx5D/ozLQnV4RL0SHWt19owqcyOWz6?=
 =?us-ascii?Q?zNpr/qKwBTTwD1xqo6D/cDjI++nMjjqbFXd1wjduev1B14NwVXhHBUZ3ncBc?=
 =?us-ascii?Q?q302Dchlqyl5RyYrxwAVO/77Z8Svg4qg0vNh6fS7ZE0Wa2QMjeh9ISELkaG/?=
 =?us-ascii?Q?fuX3H2FRKD+hMZRwziHYe//Vsd+ROR1ThntiGZzTrmiJBvxYnuqnYlKA2HKl?=
 =?us-ascii?Q?AYYLNWuMCg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6eed2b70-d461-4813-f633-08da2e7158f4
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 May 2022 08:29:18.4664
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: asOwwDXA2utobpbJ1p6mfMYqw/J9oReP9eMOvXr/uoV2/d9OGjNMnO9hRMMrQrHUOGOmmGbVO5SrodbX2DErxw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6982
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Shawn,

> Subject: Re: [PATCH V2 0/3] arm64: add i.MX93 support
>=20
> Hi Peng,
>=20
> On Tue, May 03, 2022 at 08:29:48PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> >
> > V2:
> >  Update License
> >
> > Add i.MX93 dtsi and 11x11 evk board support
> >
> > The required bindings and drivers has already been in tree.
> > There arem checkpatch error for the pin header files as other i.MX pin
> > pin header files.
> >
> > Peng Fan (3):
> >   arm64: dts: freescale: Add i.MX93 dtsi support
> >   arm64: dts: freescale: add i.MX93 11x11 EVK basic support
> >   arm64: defconfig: enable i.MX93 clk & pinctrl
>=20
> Could you check if we can eliminate some (if not all) of the dtbs_check i=
ssues
> below?

Sorry for this. Will fix in v3.

Thanks,
Peng.

>=20
> Shawn
>=20
>=20
>   DTC     arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb
>   CHECK   arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: /: 'compatible' is a
> required property
> 	From schema: .local/lib/python3.8/site-
> packages/dtschema/schemas/root-node.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: /: 'model' is a requir=
ed
> property
> 	From schema: .local/lib/python3.8/site-
> packages/dtschema/schemas/root-node.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@44380000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema:
> repos/korg/imx/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@44000000/serial@44380000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@44390000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema:
> repos/korg/imx/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@44000000/serial@44390000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: pinctrl@443c0000:
> 'pinctrl-0' is a dependency of 'pinctrl-names'
> 	From schema: .local/lib/python3.8/site-
> packages/dtschema/schemas/pinctrl/pinctrl-consumer.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@44000000/anatop@44480000: failed to match any schema with
> compatible: ['fsl,imx93-anatop', 'syscon']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@42570000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@42570000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@42580000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@42580000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@42590000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@42590000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@425a0000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@425a0000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@42690000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@42690000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: serial@426a0000:
> compatible: 'oneOf' conditional failed, one must be fixed:
> 	['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart'] is too long
> 	'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuar=
t',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,im=
xrt1050-
> lpuart']
> 	'fsl,imx8ulp-lpuart' was expected
> 	'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpua=
rt']
> 	'fsl,imx8qxp-lpuart' was expected
> 	From schema: Documentation/devicetree/bindings/serial/fsl-
> lpuart.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb:0:0:
> /soc@0/bus@42000000/serial@426a0000: failed to match any schema with
> compatible: ['fsl,imx93-lpuart', 'fsl,imx7ulp-lpuart']
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: gpio@43810000:
> interrupts: [[0, 57, 4], [0, 58, 4]] is too long
> 	From schema: Documentation/devicetree/bindings/gpio/gpio-
> vf610.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: gpio@43820000:
> interrupts: [[0, 59, 4], [0, 60, 4]] is too long
> 	From schema: Documentation/devicetree/bindings/gpio/gpio-
> vf610.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: gpio@43830000:
> interrupts: [[0, 189, 4], [0, 190, 4]] is too long
> 	From schema: Documentation/devicetree/bindings/gpio/gpio-
> vf610.yaml
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: gpio@47400000:
> interrupts: [[0, 10, 4], [0, 11, 4]] is too long
> 	From schema: Documentation/devicetree/bindings/gpio/gpio-
> vf610.yaml