Re: [PATCH v3 3/3] dt-bindings: mediatek: convert pinctrl to yaml

From: Rob Herring
Date: Tue Aug 03 2021 - 10:55:06 EST


On Tue, Aug 3, 2021 at 7:59 AM Rob Herring <robh@xxxxxxxxxx> wrote:
>
> On Tue, 03 Aug 2021 13:13:19 +0800, Hsin-Yi Wang wrote:
> > Convert mt65xx, mt6796, mt7622, mt8183 bindings to yaml.
> >
> > Signed-off-by: Hsin-Yi Wang <hsinyi@xxxxxxxxxxxx>
> > ---
> > v2->v3:
> > fix comments in v2.
> > ---
> > .../pinctrl/mediatek,mt65xx-pinctrl.yaml | 206 ++++++++
> > .../pinctrl/mediatek,mt6797-pinctrl.yaml | 173 +++++++
> > .../pinctrl/mediatek,mt7622-pinctrl.yaml | 373 +++++++++++++
> > .../pinctrl/mediatek,mt8183-pinctrl.yaml | 228 ++++++++
> > .../bindings/pinctrl/pinctrl-mt65xx.txt | 156 ------
> > .../bindings/pinctrl/pinctrl-mt6797.txt | 83 ---
> > .../bindings/pinctrl/pinctrl-mt7622.txt | 490 ------------------
> > .../bindings/pinctrl/pinctrl-mt8183.txt | 132 -----
> > 8 files changed, 980 insertions(+), 861 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/pinctrl/mediatek,mt65xx-pinctrl.yaml
> > create mode 100644 Documentation/devicetree/bindings/pinctrl/mediatek,mt6797-pinctrl.yaml
> > create mode 100644 Documentation/devicetree/bindings/pinctrl/mediatek,mt7622-pinctrl.yaml
> > create mode 100644 Documentation/devicetree/bindings/pinctrl/mediatek,mt8183-pinctrl.yaml
> > delete mode 100644 Documentation/devicetree/bindings/pinctrl/pinctrl-mt65xx.txt
> > delete mode 100644 Documentation/devicetree/bindings/pinctrl/pinctrl-mt6797.txt
> > delete mode 100644 Documentation/devicetree/bindings/pinctrl/pinctrl-mt7622.txt
> > delete mode 100644 Documentation/devicetree/bindings/pinctrl/pinctrl-mt8183.txt
> >
>
> 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:
>
> dtschema/dtc warnings/errors:
> Documentation/devicetree/bindings/pinctrl/mediatek,mt8183-pinctrl.example.dts:21:18: fatal error: dt-bindings/pinctrl/mt8183-pinfunc.h: No such file or directory
> 21 | #include <dt-bindings/pinctrl/mt8183-pinfunc.h>
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> compilation terminated.
> make[1]: *** [scripts/Makefile.lib:380: Documentation/devicetree/bindings/pinctrl/mediatek,mt8183-pinctrl.example.dt.yaml] Error 1
> make[1]: *** Waiting for unfinished jobs....
> make: *** [Makefile:1419: dt_binding_check] Error 2
> \ndoc reference errors (make refcheckdocs):
> MAINTAINERS: Documentation/devicetree/bindings/pinctrl/pinctrl-mt65xx.txt
> MAINTAINERS: Documentation/devicetree/bindings/pinctrl/pinctrl-mt7622.txt

Not sure why the header patch is not getting applied, so ignore that.
But these MAINTAINERS references need to be fixed.

Rob