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;
From:   Peng Fan <peng.fan@nxp.com>
To:     Rob Herring <robh@kernel.org>,
        "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
CC:     "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        Aisheng Dong <aisheng.dong@nxp.com>,
        "festevam@gmail.com" <festevam@gmail.com>,
        Jacky Bai <ping.bai@nxp.com>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 0/9] dt-bindings: pinctrl: imx: use minItems
Thread-Topic: [PATCH 0/9] dt-bindings: pinctrl: imx: use minItems
Thread-Index: AQHYnnhx6HQy9b/Z1UqXbelGtR85J62Ph3EAgADX9WA=
Date:   Tue, 26 Jul 2022 09:03:32 +0000
Message-ID: <DU0PR04MB94171571EAF51759531AB7E888949@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220723094335.3577048-1-peng.fan@oss.nxp.com>
 <20220725200740.GA2610621-robh@kernel.org>
In-Reply-To: <20220725200740.GA2610621-robh@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?XB9fOYvBbVOQNOvvQovJFl7P+SgVgFMAUtnvc/VjvrY2AeBvFv6bfYVwqQvn?=
 =?us-ascii?Q?mj7PjLYENmFTRwtNBMndFVuMFeJRI3zbwBkmGToS1tPQSEeCU42dCgN5Vcqi?=
 =?us-ascii?Q?5N3oeudYCUJysjfLUBiY1FsvJW6L4Hbf8/RiZ6iIuHMb0RZWhY0XnAJVPnwq?=
 =?us-ascii?Q?8kA6KpLO9k7kn1PjKjmk9FQiYsMsVFlpxdjilsemm+FageCnGZGD4ffIRf/y?=
 =?us-ascii?Q?niqiN2dPxHwJt0BpRuochWdAaudUCkBQ4E+Vu85IBAw9tZOJE1wEJYsSAKgn?=
 =?us-ascii?Q?s4/MeCDIFV4NBH6j81F76k5md0DL/5gro0i0XuC+3k96QIOzD5PjIfIicY8x?=
 =?us-ascii?Q?CGQ1x/aBeiaZUqxmv+iihrW37ZNeAVrdzh9rRUiIak0L8oB9jkvheGCWE/wW?=
 =?us-ascii?Q?78TOX1LzK2u7thza9EyT8VfODi6V0k8quUdcMWZq/6QceU/mnQf0fOb0jf6l?=
 =?us-ascii?Q?hxH0KvLV+NH7GEU0usj9aCeG09yiWDKVy00D8adFT77vvm5AqvfSQraNXkx5?=
 =?us-ascii?Q?tJA+Dby7JhkiYmJuEMxd0WNxKAdi9hBrmPQfDKCL0EszTYql7O0QdhxKHoyI?=
 =?us-ascii?Q?N+EdAZ3Gaoyyw5RfQUqe5v2Ax54HIhxnW7PgFhmGot+fyRaz2y1cDUiAbhJX?=
 =?us-ascii?Q?HjpR0+ai7xjmqyq9qc9hG9CIABPbTlM0iCBmW1RMBfoCm84JO30e737Kg3TR?=
 =?us-ascii?Q?ZiA66cefUxOJZ9nBu9u6UmrplexAEa6MV301f5m8xGaaJN7OrnZODs0VYY6a?=
 =?us-ascii?Q?jJWhxXlLvjTtJgHVQ4l3gVI9HoRM0f1Od0Zq6SfndBbXNl1LBH+L+4SeVrps?=
 =?us-ascii?Q?OeTbem5QwrtlRRtkW5gVNI8CjULvvkuYCNfGEIl0owYjKBIyI7Tq7MzmJbtT?=
 =?us-ascii?Q?V0t9wvQYIDYKyUYz1wHagrWbDUMUSrNB56HAxHIXZLxBZCC1gbn0UxUCF5TB?=
 =?us-ascii?Q?58n4d9kOwMcaUOVwFDg5/jW4MrpLVls024a7lrKxx0I3bYjYas8HHKRaZ3I0?=
 =?us-ascii?Q?+/eHtqwo13TTe1+EF6Gph3cpvAR6sklV8qlcEkC93hJF/o81JYFPOFOMwe7E?=
 =?us-ascii?Q?sDb+7QhBFb8xTxe/q6+bNPT1BdauI98QZaSt+luJgTl3IHPfXdFKTnIN8OOJ?=
 =?us-ascii?Q?85N5yGzK/2zq+5Y0vPSsRf++keE/GQBqlE+aaGB3AHKQICa1XsecvkZe4WF+?=
 =?us-ascii?Q?SByPk2V9MiGq/0y+sytqJshxQ0HzKM8GVf59998wZ0NQULVMrNTWrDrkJdRh?=
 =?us-ascii?Q?9fR3AxgrBThJKYwZmArDojTqZhFviklewrfftgktkQthUsAUpMB9oBMf/HT6?=
 =?us-ascii?Q?lnwIKokguzKSbKIZ/KxKZw97Df8puypICzcM0woUgadfrshmcmvqNYfY46Mm?=
 =?us-ascii?Q?2EYV4ROwWWOXTeQmIG7YEfQFbHzG3DMZ4Hk2Kp+PELjYqAC2H5wa8RS51NVK?=
 =?us-ascii?Q?MNqsTISwG80YEIcemUx8w2yiHvb2eE3HrpWTvUsnquw1ylzlrbUerfK9DqOQ?=
 =?us-ascii?Q?EFQFyLLCu04ukBeuqSgccWU8TG/CgGpNbPjjqjTyG/YKBXtFPxfOLDY3phS0?=
 =?us-ascii?Q?zt9gsEMtoWIi6yacyK8=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: 9651435b-cce8-4a2a-5324-08da6ee5b70e
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Jul 2022 09:03:32.3457
 (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: kN63893SD1m9r6UMSRrYTj4feMqvg6bFucu4YkU7k/sWiHArZSGe5DGQujyjpkE4Gu85urkJus4CSVtwMO9ORQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8443
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        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

> Subject: Re: [PATCH 0/9] dt-bindings: pinctrl: imx: use minItems
>=20
> On Sat, Jul 23, 2022 at 05:43:26PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > There are many warnings when do dtbs_check: fsl,pins are too long, so
> > add minItems to address that.
>=20
> A single cell is not valid though, right?
>=20
> This particular property is problematic because each entry depending on t=
he
> platform has different number of cells (5, 6, etc.). A single cell (minIt=
ems: 1)
> is not valid though, right?

I thought no maxItems means no limitation on maxItems (:
Then I should add maxItems.
>=20
> There's a fix in dtschema min branch which should fix the warnings.
> Unfortunately, it just strips any bounds checking.

ok, let me update dtschema to rerun the dtbs_check.
Thanks,
Peng.
>=20
> Rob