Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Richard Zhu <hongxing.zhu@nxp.com>
To:     Rob Herring <robh@kernel.org>
CC:     "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "linux-phy@lists.infradead.org" <linux-phy@lists.infradead.org>,
        "galak@kernel.crashing.org" <galak@kernel.crashing.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "tharvey@gateworks.com" <tharvey@gateworks.com>,
        "kishon@ti.com" <kishon@ti.com>, dl-linux-imx <linux-imx@nxp.com>,
        "l.stach@pengutronix.de" <l.stach@pengutronix.de>,
        "vkoul@kernel.org" <vkoul@kernel.org>
Subject: RE: [PATCH v3 2/9] dt-bindings: phy: add imx8 pcie phy driver support
Thread-Topic: [PATCH v3 2/9] dt-bindings: phy: add imx8 pcie phy driver
 support
Thread-Index: AQHXv0hlrBQmJmSLUUWUdasqgJ4mX6vPWLqAgACu40A=
Date:   Tue, 12 Oct 2021 23:46:25 +0000
Message-ID: <AS8PR04MB867657DBC4C2576930647EEE8CB69@AS8PR04MB8676.eurprd04.prod.outlook.com>
References: <1634028078-2387-1-git-send-email-hongxing.zhu@nxp.com>
 <1634028078-2387-3-git-send-email-hongxing.zhu@nxp.com>
 <1634044709.029189.2661233.nullmailer@robh.at.kernel.org>
In-Reply-To: <1634044709.029189.2661233.nullmailer@robh.at.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?evTR5BDXX9EDJf45pakTTFJiCQlwaor1ebL7lhllY4tjrR6l9HviXLmA9ckT?=
 =?us-ascii?Q?Scix4r5Ehp3Lhjmc+lL1WMfQjPy1hxrWoHAlK9LDmHE0ImF6C5Pe2Y6QF5XK?=
 =?us-ascii?Q?iHJ02z8znG4ZbGvZweEJK90nF75W3kw2xAiENgS+EjJCpBADj2SzaG/VqRg/?=
 =?us-ascii?Q?4xg/ptZG7gVPul4kVMdqe/b/02kwvoSxSl8osn97/TmZTsQ9ZDKLKlegphqo?=
 =?us-ascii?Q?w1RRW0s7/iX5DWEwgCsY74Q70cvcPLnwTGMBKkRG0GMgm34eSpx7sHrbF7Eg?=
 =?us-ascii?Q?ztOqH9P92GOiiGlyceFFtMrj+sNtbDX6S60+Q/BhZM8v4oRw6jz8DByTxHEG?=
 =?us-ascii?Q?G1TtjO8SsS0pfcbPe1RvBnDJNLTE+V3sxtwqsI2OsnfLUtWbfxtBDCT554IO?=
 =?us-ascii?Q?sYtASEqMH7lhUglleQrj4Jay/gbWXMl2zaNdLRDWNbiDCi5k6SuwIceNH5gb?=
 =?us-ascii?Q?eXkPQ9+Aw6Y/LHsaNyL+/wnplRhchnoFackfGqHE9JGWeJZztdGJ8fbwzH75?=
 =?us-ascii?Q?Ru0EUFVqfb900F7k7QBJU2MCnQ1Jv8F5iBUik5TIja1zKnZQVD4pWsTX9Xfj?=
 =?us-ascii?Q?vAjEn1/iN8Z35xoU6z8JDZwDOeRuB68HeBRPz2stCNlRfyengJ1AhQPNgYpM?=
 =?us-ascii?Q?G7Xcik1Adlom5NtRMY5ckv4iTW6zKZK9RWD0j9ajGkeSt6mOYiWVGXXzwoub?=
 =?us-ascii?Q?a6Ecg1QJ7W8P0uQsrzi3+jX6u+UZgaxCNhzYqqB6PkXD1pEqqOkQkl5m42qt?=
 =?us-ascii?Q?PJlbbjCJKpmQ4oN/Zh0FgiWSwDVhx3lqvjjz4owMw+04WPQ0XV3AnDNCf4rR?=
 =?us-ascii?Q?uulLp7IIDS4/TPM2gOVlmr3YcwDlKhdmAy+Ru74LoqlVOhqapYUmuuC4MBwi?=
 =?us-ascii?Q?Xw/KSm3pVCmn/MwxdrGwTtqdfYZszWO1JDppjqn+0aU29fGtjujMwrHzMSXA?=
 =?us-ascii?Q?8R9Qmp13E+4kXhgzK3lhBOhJ1OLO3wewAdAK7GnOYoeFt60ZlNDs/IElaAf2?=
 =?us-ascii?Q?jprOJO1Onw0Er9jnC0jbglZ+712l2/RSFDo0GH8IkoUfB0k6O0+servvT2FM?=
 =?us-ascii?Q?fyLidO4IfmTppDzF5Txc03zs52T5uQk8BRLn/ElBaV/1ihgnD/ceVLu3heIc?=
 =?us-ascii?Q?XfdLOmxGnEOvvKLiisd3uG76NM6GmGfn1Ff+LKeTMRNV46RuPA1nC3DXSHgV?=
 =?us-ascii?Q?Vt3shIHuNEYih1gjfNUmBxMjVltOGBYVZp2oY/Su2Ag2vaUMnfwXNYhXpL6m?=
 =?us-ascii?Q?qzL75y02pXCZCq4yy6xkiW1H83lRP2d9815Ue0i0FKotdozF44fU+xRdcE4j?=
 =?us-ascii?Q?xDyhPSgQklehGA4I/i8FSIYK?=
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: AS8PR04MB8676.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 420dfdff-a4f9-48ce-f329-08d98dda8111
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Oct 2021 23:46:25.6394
 (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: gzv6Uo0HmnGs66X2ZxpENdPHrgUwLnxC7hku113mfSgCn0va6VjARiEUCBGnejve7C8RQ9S2YNx4NTgWrmGfyg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8436
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Tuesday, October 12, 2021 9:18 PM
> To: Richard Zhu <hongxing.zhu@nxp.com>
> Cc: shawnguo@kernel.org; linux-phy@lists.infradead.org;
> galak@kernel.crashing.org; linux-arm-kernel@lists.infradead.org;
> devicetree@vger.kernel.org; kernel@pengutronix.de;
> linux-kernel@vger.kernel.org; tharvey@gateworks.com; kishon@ti.com;
> dl-linux-imx <linux-imx@nxp.com>; l.stach@pengutronix.de;
> vkoul@kernel.org
> Subject: Re: [PATCH v3 2/9] dt-bindings: phy: add imx8 pcie phy driver su=
pport
>=20
> On Tue, 12 Oct 2021 16:41:11 +0800, Richard Zhu wrote:
> > Add dt-binding for the standalone i.MX8 PCIe PHY driver.
> >
> > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com>
> > ---
> >  .../bindings/phy/fsl,imx8-pcie-phy.yaml       | 79
> +++++++++++++++++++
> >  1 file changed, 79 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/phy/fsl,imx8-pcie-phy.yaml
> >
>=20
> My bot found errors running 'make DT_CHECKER_FLAGS=3D-m
> dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>=20
> yamllint warnings/errors:
>=20
> dtschema/dtc warnings/errors:
> Error:
> Documentation/devicetree/bindings/phy/fsl,imx8-pcie-phy.example.dts:30.3
> 2-33 syntax error FATAL ERROR: Unable to parse input tree
> make[1]: *** [scripts/Makefile.lib:385:
> Documentation/devicetree/bindings/phy/fsl,imx8-pcie-phy.example.dt.yaml]
> Error 1
> make[1]: *** Waiting for unfinished jobs....
> make: *** [Makefile:1441: dt_binding_check] Error 2
>=20
[Richard Zhu] I forget to merge to add the "#include <dt-bindings/reset/imx=
8mq-reset.h>" after add the reset property.
Sorry about that, would add it later.


Best Regards
Richard Zhu
> doc reference errors (make refcheckdocs):
>=20
> See
> https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fpatch
> work.ozlabs.org%2Fpatch%2F1539660&amp;data=3D04%7C01%7Chongxing.zhu
> %40nxp.com%7C64619a3cd64446cf204008d98d82ca07%7C686ea1d3bc2b4c
> 6fa92cd99c5c301635%7C0%7C0%7C637696415146711570%7CUnknown%7
> CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwi
> LCJXVCI6Mn0%3D%7C1000&amp;sdata=3Djt2nbxfbKTBm1izvsYvsHEkMfSHqa6t
> 24B1sJICAtXE%3D&amp;reserved=3D0
>=20
> This check can fail if there are any dependencies. The base for a patch s=
eries
> is generally the most recent rc1.
>=20
> If you already ran 'make dt_binding_check' and didn't see the above error=
(s),
> then make sure 'yamllint' is installed and dt-schema is up to
> date:
>=20
> pip3 install dtschema --upgrade
>=20
> Please check and re-submit.