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:   ChiaWei Wang <chiawei_wang@aspeedtech.com>
To:     Rob Herring <robh@kernel.org>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
        "linux-aspeed@lists.ozlabs.org" <linux-aspeed@lists.ozlabs.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "joel@jms.id.au" <joel@jms.id.au>,
        "andrew@aj.id.au" <andrew@aj.id.au>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: RE: [PATCH v3 1/4] dt-bindings: aspeed: Add eSPI controller
Thread-Topic: [PATCH v3 1/4] dt-bindings: aspeed: Add eSPI controller
Thread-Index: AQHXmkIdttmpXm7UpE2tjp2jWlnrTKuFx06AgADlQGA=
Date:   Fri, 27 Aug 2021 03:08:19 +0000
Message-ID: <HK0PR06MB3779F7F75D24326E6F0DB23591C89@HK0PR06MB3779.apcprd06.prod.outlook.com>
References: <20210826061623.6352-1-chiawei_wang@aspeedtech.com>
 <20210826061623.6352-2-chiawei_wang@aspeedtech.com>
 <1629984361.334223.1033382.nullmailer@robh.at.kernel.org>
In-Reply-To: <1629984361.334223.1033382.nullmailer@robh.at.kernel.org>
Accept-Language: zh-TW, en-US
Content-Language: zh-TW
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?tspzspcT+keZkB/C3wd+xiUhDpVASTXz05/MWKkCvLAIqZ7UroxBXfSOE1eB?=
 =?us-ascii?Q?3ouNqsQdGiRRNc5WPU/aW0cvnXeGiMvtQw/FGrtd5bcP0A9IR+PV9yo+ql0Q?=
 =?us-ascii?Q?AyYWtzTH2kjYR1HsbETc8m1swXFpIp1Mv3Kaxiftw9F3N/qKvGe6cvWdAwsd?=
 =?us-ascii?Q?6mUq20VKkeKp8ApR+Y8LhMPEfTkRWCDabLopYnJQhcEW79cSqQZnHCSRGW4z?=
 =?us-ascii?Q?3J7vqBMFE8b5c1Puq1Jjw3uOiFLkkpX1XIIttKuDcF22jmjFaXT/tFPyej2/?=
 =?us-ascii?Q?GZESvZRdUHFao58l9/Pn5Fr7totZOmcz3ZSvK/NkOyaB8Hwx40jNAdXVFw4t?=
 =?us-ascii?Q?TVAepcpFM/7wJYImBYib9ZQAIorujiKMAsmDrAZXAWIzUqfzEiQCB7SihCE6?=
 =?us-ascii?Q?y430bQZ4ZxYs6DisUzFh2j1D/XIjgyGK6bP+fAPyUamsXKiNT+eWB7TCip9F?=
 =?us-ascii?Q?ztjpf8Vb1vbXeGAQ61SY/t2MF2StbQ8rIzzTqs6GW4hiXqy0vLg8Yz9nOk1m?=
 =?us-ascii?Q?gb8Coen6eVvuMNDjtCJSFbmZ057sR6/kuOqOMBi2umiMZqyqOqiBBPqPIDfl?=
 =?us-ascii?Q?bo9p+qZgutWodYILRdTf2Il44TGzBTu926NiKxua+6AJrDAdUCQytjpUWujG?=
 =?us-ascii?Q?ch/ym0aCVCqyvbaNOPwjTHwGf4qMWD6ipCwWeQAx67/D5a09QqvAPJUR83V1?=
 =?us-ascii?Q?W3uRNnjpOTFQnBGWJjaYshSW9ZNesk6Jf+PoCywfQMJWgBfBfo7p9QAXFKb0?=
 =?us-ascii?Q?mj2L1zlzr3WAO/NBexWVkxujeAhfPO+Me7RzWMkOPPsUKiRdCnerZomCWqNB?=
 =?us-ascii?Q?bm1izL3O9kSokm6yGt7kxCYby6E0JEP7tlwIcLyC3oaoQDUUeqAfCEdg0vc2?=
 =?us-ascii?Q?mFl/Uf2247hMtqds0ob84MBcPGj3MfkJIpsA2SY7CgrUOulLD0aqNS9xcKkH?=
 =?us-ascii?Q?H1oKmamdIGjgT1B3i/x0+TJcEK6WXIec4hYBMJuY5hZpj/SQLXUq1bd5Su/u?=
 =?us-ascii?Q?xf575x06hXUw+glRe+Rg1NDIbwrDyekxNWS6Xb1322cleF0gnbkbf6W7nqLc?=
 =?us-ascii?Q?TqOL9b7Tl4TQcAEELptIv+CpNOjJHhUDvqvH9NzWcBMLZtcwDiWE2CsQ40n6?=
 =?us-ascii?Q?sNWZJkzl/mroO4aLpFq5E7JNGobzdMdIIjCvR3w66+FgTSwcdumUhnmjP1M9?=
 =?us-ascii?Q?YvDR6FzrPhOQUm4old4kubMMz/d9gKo0FbqKNmgw30D8PgJA1RhQ0cCi8pEo?=
 =?us-ascii?Q?M+R/0BVETVC/SWht1R2a4cCcAC9FIBERZ59ywBY1gISSZbeF6WBoLKxMgbff?=
 =?us-ascii?Q?Ovhc4zh/Li9Tc0TUFaIJHXBv?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: aspeedtech.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: HK0PR06MB3779.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 99353c3b-5609-494b-c194-08d96907ec09
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Aug 2021 03:08:19.4955
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 43d4aa98-e35b-4575-8939-080e90d5a249
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: XxLgH44MPXHles6L0mPme1CFVYHYRKQ8DZlftl/4LZhqFhd8BgWxDl6ltXzO+e4P3WK/spTXm71zOsTuoHqPGkVKPMy2aPFI78ol8DUBkBw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HK2PR06MB3554
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Rob Herring <robh@kernel.org>
> Sent: Thursday, August 26, 2021 9:26 PM
>=20
> On Thu, 26 Aug 2021 14:16:20 +0800, Chia-Wei Wang wrote:
> > Add dt-bindings for Aspeed eSPI controller
> >
> > Signed-off-by: Chia-Wei Wang <chiawei_wang@aspeedtech.com>
> > ---
> >  .../devicetree/bindings/soc/aspeed/espi.yaml  | 157
> > ++++++++++++++++++
> >  1 file changed, 157 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/soc/aspeed/espi.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:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/soc/=
as
> peed/espi.example.dt.yaml: espi@1e6ee000: 'espi-ctrl@0', 'espi-mmbi@800'
> do not match any of the regexes: 'pinctrl-[0-9]+'
> 	From schema:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/soc/=
as
> peed/espi.yaml
>=20
> doc reference errors (make refcheckdocs):
>=20
> See https://patchwork.ozlabs.org/patch/1520968
>=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.

The warning has been fixed by using patternProperties to define the espi-ct=
rl and the espi-mmbi child nodes.
The updated version will be submitted soon. Thanks.

Chiawei