From: Chuanhong Guo <gch981213@gmail.com>
To: Rob Herring <robh@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>,
linux-spi@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Vignesh Raghavendra <vigneshr@ti.com>,
Richard Weinberger <richard@nod.at>,
Pratyush Yadav <p.yadav@ti.com>,
linux-arm-kernel@lists.infradead.org,
Cai Huoqing <cai.huoqing@linux.dev>,
Roger Quadros <rogerq@kernel.org>, Yu Kuai <yukuai3@huawei.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-mediatek@lists.infradead.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
devicetree@vger.kernel.org, Mark Brown <broonie@kernel.org>,
linux-kernel@vger.kernel.org,
Paul Cercueil <paul@crapouillou.net>,
Florian Fainelli <f.fainelli@gmail.com>,
Colin Ian King <colin.king@intel.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-mtd@lists.infradead.org
Subject: Re: [PATCH v3 4/5] dt-bindings: spi: add binding doc for spi-mtk-snfi
Date: Tue, 5 Apr 2022 10:55:51 +0800 [thread overview]
Message-ID: <CAJsYDVLaXAoL=TcPun6rckcA_cdUS-zFy_7M6uCpfzX+jbQEag@mail.gmail.com> (raw)
In-Reply-To: <1649088538.050456.1436949.nullmailer@robh.at.kernel.org>
Hi Rob!
On Tue, Apr 5, 2022 at 12:09 AM Rob Herring <robh@kernel.org> wrote:
> [...]
> 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:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml: spi@1100d000: 'ecc-engine' is a required property
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
> Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml:0:0: /example-0/soc/spi@1100d000/flash@0: failed to match any schema with compatible: ['spi-nand']
I ran the tests myself and it's only complaining about the ecc-engine name:
/home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dtb:
spi@1100d000: 'ecc-engine' is a required property
From schema: /home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
It says nothing about the spi-nand part.
I'd like to keep the flash@0 node in the example to demonstrate the
nand-ecc-engine usage. What should I do?
--
Regards,
Chuanhong Guo
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
WARNING: multiple messages have this Message-ID (diff)
From: Chuanhong Guo <gch981213@gmail.com>
To: Rob Herring <robh@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>,
linux-spi@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Vignesh Raghavendra <vigneshr@ti.com>,
Richard Weinberger <richard@nod.at>,
Pratyush Yadav <p.yadav@ti.com>,
linux-arm-kernel@lists.infradead.org,
Cai Huoqing <cai.huoqing@linux.dev>,
Roger Quadros <rogerq@kernel.org>, Yu Kuai <yukuai3@huawei.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-mediatek@lists.infradead.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
devicetree@vger.kernel.org, Mark Brown <broonie@kernel.org>,
linux-kernel@vger.kernel.org,
Paul Cercueil <paul@crapouillou.net>,
Florian Fainelli <f.fainelli@gmail.com>,
Colin Ian King <colin.king@intel.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-mtd@lists.infradead.org
Subject: Re: [PATCH v3 4/5] dt-bindings: spi: add binding doc for spi-mtk-snfi
Date: Tue, 5 Apr 2022 10:55:51 +0800 [thread overview]
Message-ID: <CAJsYDVLaXAoL=TcPun6rckcA_cdUS-zFy_7M6uCpfzX+jbQEag@mail.gmail.com> (raw)
In-Reply-To: <1649088538.050456.1436949.nullmailer@robh.at.kernel.org>
Hi Rob!
On Tue, Apr 5, 2022 at 12:09 AM Rob Herring <robh@kernel.org> wrote:
> [...]
> 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:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml: spi@1100d000: 'ecc-engine' is a required property
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
> Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml:0:0: /example-0/soc/spi@1100d000/flash@0: failed to match any schema with compatible: ['spi-nand']
I ran the tests myself and it's only complaining about the ecc-engine name:
/home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dtb:
spi@1100d000: 'ecc-engine' is a required property
From schema: /home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
It says nothing about the spi-nand part.
I'd like to keep the flash@0 node in the example to demonstrate the
nand-ecc-engine usage. What should I do?
--
Regards,
Chuanhong Guo
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
WARNING: multiple messages have this Message-ID (diff)
From: Chuanhong Guo <gch981213@gmail.com>
To: Rob Herring <robh@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>,
linux-spi@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Vignesh Raghavendra <vigneshr@ti.com>,
Richard Weinberger <richard@nod.at>,
Pratyush Yadav <p.yadav@ti.com>,
linux-arm-kernel@lists.infradead.org,
Cai Huoqing <cai.huoqing@linux.dev>,
Roger Quadros <rogerq@kernel.org>, Yu Kuai <yukuai3@huawei.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-mediatek@lists.infradead.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
devicetree@vger.kernel.org, Mark Brown <broonie@kernel.org>,
linux-kernel@vger.kernel.org,
Paul Cercueil <paul@crapouillou.net>,
Florian Fainelli <f.fainelli@gmail.com>,
Colin Ian King <colin.king@intel.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-mtd@lists.infradead.org
Subject: Re: [PATCH v3 4/5] dt-bindings: spi: add binding doc for spi-mtk-snfi
Date: Tue, 5 Apr 2022 10:55:51 +0800 [thread overview]
Message-ID: <CAJsYDVLaXAoL=TcPun6rckcA_cdUS-zFy_7M6uCpfzX+jbQEag@mail.gmail.com> (raw)
In-Reply-To: <1649088538.050456.1436949.nullmailer@robh.at.kernel.org>
Hi Rob!
On Tue, Apr 5, 2022 at 12:09 AM Rob Herring <robh@kernel.org> wrote:
> [...]
> 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:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml: spi@1100d000: 'ecc-engine' is a required property
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
> Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml:0:0: /example-0/soc/spi@1100d000/flash@0: failed to match any schema with compatible: ['spi-nand']
I ran the tests myself and it's only complaining about the ecc-engine name:
/home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dtb:
spi@1100d000: 'ecc-engine' is a required property
From schema: /home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
It says nothing about the spi-nand part.
I'd like to keep the flash@0 node in the example to demonstrate the
nand-ecc-engine usage. What should I do?
--
Regards,
Chuanhong Guo
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Chuanhong Guo <gch981213@gmail.com>
To: Rob Herring <robh@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>,
linux-spi@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Vignesh Raghavendra <vigneshr@ti.com>,
Richard Weinberger <richard@nod.at>,
Pratyush Yadav <p.yadav@ti.com>,
linux-arm-kernel@lists.infradead.org,
Cai Huoqing <cai.huoqing@linux.dev>,
Roger Quadros <rogerq@kernel.org>, Yu Kuai <yukuai3@huawei.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-mediatek@lists.infradead.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
devicetree@vger.kernel.org, Mark Brown <broonie@kernel.org>,
linux-kernel@vger.kernel.org,
Paul Cercueil <paul@crapouillou.net>,
Florian Fainelli <f.fainelli@gmail.com>,
Colin Ian King <colin.king@intel.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-mtd@lists.infradead.org
Subject: Re: [PATCH v3 4/5] dt-bindings: spi: add binding doc for spi-mtk-snfi
Date: Tue, 5 Apr 2022 10:55:51 +0800 [thread overview]
Message-ID: <CAJsYDVLaXAoL=TcPun6rckcA_cdUS-zFy_7M6uCpfzX+jbQEag@mail.gmail.com> (raw)
In-Reply-To: <1649088538.050456.1436949.nullmailer@robh.at.kernel.org>
Hi Rob!
On Tue, Apr 5, 2022 at 12:09 AM Rob Herring <robh@kernel.org> wrote:
> [...]
> 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:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml: spi@1100d000: 'ecc-engine' is a required property
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
> Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dt.yaml:0:0: /example-0/soc/spi@1100d000/flash@0: failed to match any schema with compatible: ['spi-nand']
I ran the tests myself and it's only complaining about the ecc-engine name:
/home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.example.dtb:
spi@1100d000: 'ecc-engine' is a required property
From schema: /home/user/src/kernels/linux/Documentation/devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml
It says nothing about the spi-nand part.
I'd like to keep the flash@0 node in the example to demonstrate the
nand-ecc-engine usage. What should I do?
--
Regards,
Chuanhong Guo
next prev parent reply other threads:[~2022-04-05 2:56 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-04 13:18 [PATCH v3 0/5] spi: add support for Mediatek SPI-NAND controller Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` [PATCH v3 1/5] mtd: nand: make mtk_ecc.c a separated module Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` [PATCH v3 2/5] spi: add driver for MTK SPI NAND Flash Interface Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` [PATCH v3 3/5] mtd: nand: mtk-ecc: also parse nand-ecc-engine if available Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` [PATCH v3 4/5] dt-bindings: spi: add binding doc for spi-mtk-snfi Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:52 ` Krzysztof Kozlowski
2022-04-04 13:52 ` Krzysztof Kozlowski
2022-04-04 13:52 ` Krzysztof Kozlowski
2022-04-04 13:52 ` Krzysztof Kozlowski
2022-04-04 14:04 ` Mark Brown
2022-04-04 14:04 ` Mark Brown
2022-04-04 14:04 ` Mark Brown
2022-04-04 14:04 ` Mark Brown
2022-04-04 14:15 ` Chuanhong Guo
2022-04-04 14:15 ` Chuanhong Guo
2022-04-04 14:15 ` Chuanhong Guo
2022-04-04 14:15 ` Chuanhong Guo
2022-04-04 16:08 ` Rob Herring
2022-04-04 16:08 ` Rob Herring
2022-04-04 16:08 ` Rob Herring
2022-04-04 16:08 ` Rob Herring
2022-04-05 2:55 ` Chuanhong Guo [this message]
2022-04-05 2:55 ` Chuanhong Guo
2022-04-05 2:55 ` Chuanhong Guo
2022-04-05 2:55 ` Chuanhong Guo
2022-04-05 7:20 ` Miquel Raynal
2022-04-05 7:20 ` Miquel Raynal
2022-04-05 7:20 ` Miquel Raynal
2022-04-05 7:20 ` Miquel Raynal
2022-04-05 14:39 ` Chuanhong Guo
2022-04-05 14:39 ` Chuanhong Guo
2022-04-05 14:39 ` Chuanhong Guo
2022-04-05 14:39 ` Chuanhong Guo
2022-04-05 15:52 ` Rob Herring
2022-04-05 15:52 ` Rob Herring
2022-04-05 15:52 ` Rob Herring
2022-04-05 15:52 ` Rob Herring
2022-04-05 16:08 ` Chuanhong Guo
2022-04-05 16:08 ` Chuanhong Guo
2022-04-05 16:08 ` Chuanhong Guo
2022-04-05 16:08 ` Chuanhong Guo
2022-04-04 13:18 ` [PATCH v3 5/5] arm64: dts: mediatek: add mtk-snfi for mt7622 Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
2022-04-04 13:18 ` Chuanhong Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAJsYDVLaXAoL=TcPun6rckcA_cdUS-zFy_7M6uCpfzX+jbQEag@mail.gmail.com' \
--to=gch981213@gmail.com \
--cc=broonie@kernel.org \
--cc=cai.huoqing@linux.dev \
--cc=colin.king@intel.com \
--cc=devicetree@vger.kernel.org \
--cc=f.fainelli@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-spi@vger.kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=miquel.raynal@bootlin.com \
--cc=p.yadav@ti.com \
--cc=paul@crapouillou.net \
--cc=richard@nod.at \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=rogerq@kernel.org \
--cc=tsbogend@alpha.franken.de \
--cc=vigneshr@ti.com \
--cc=wsa+renesas@sang-engineering.com \
--cc=yukuai3@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.