[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ee7e43d86102455aa72c38d02b747284@sphcmbx02.sunplus.com.tw>
Date: Fri, 12 Nov 2021 02:57:58 +0000
From: Wells Lu 呂芳騰 <wells.lu@...plus.com>
To: Rob Herring <robh@...nel.org>, Wells Lu <wellslutw@...il.com>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Vincent Shih 施錕鴻 <vincent.shih@...plus.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"kuba@...nel.org" <kuba@...nel.org>,
"robh+dt@...nel.org" <robh+dt@...nel.org>
Subject: RE: [PATCH v2 1/2] devicetree: bindings: net: Add bindings doc for
Sunplus SP7021.
Hi,
> On Thu, 11 Nov 2021 17:04:20 +0800, Wells Lu wrote:
> > Add bindings documentation for Sunplus SP7021.
> >
> > Signed-off-by: Wells Lu <wells.lu@...plus.com>
> > ---
> > Changes in V2
> > - Added mdio and phy sub-nodes.
> >
> > .../bindings/net/sunplus,sp7021-emac.yaml | 152 +++++++++++++++++++++
> > MAINTAINERS | 7 +
> > 2 files changed, 159 insertions(+)
> > create mode 100644
> > Documentation/devicetree/bindings/net/sunplus,sp7021-emac.yaml
> >
>
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>
> yamllint warnings/errors:
> ./Documentation/devicetree/bindings/net/sunplus,sp7021-emac.yaml:94:12: [warning] wrong
> indentation: expected 10 but found 11 (indentation)
>
> dtschema/dtc warnings/errors:
>
> doc reference errors (make refcheckdocs):
>
> See https://patchwork.ozlabs.org/patch/1553831
>
> This check can fail if there are any dependencies. The base for a patch series is generally the most
> recent rc1.
>
> 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:
>
> pip3 install dtschema --upgrade
>
> Please check and re-submit.
Thank you very much for review and tests.
I'll fix wrong indentation next patch.
Powered by blists - more mailing lists