Message ID | 20240627-k1-01-basic-dt-v2-3-cc06c7555f07@gentoo.org (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | riscv: add initial support for SpacemiT K1 | expand |
On Thu, 27 Jun 2024 15:31:17 +0000, Yixun Lan wrote: > From: Yangyu Chen <cyy@cyyself.name> > > Add DT binding documentation for the SpacemiT K1 Soc[1] and the Banana > Pi BPi-F3 board[2] which used it. > > [1] https://www.spacemit.com/en/spacemit-key-stone-2/ > [2] https://docs.banana-pi.org/en/BPI-F3/BananaPi_BPI-F3 > > Signed-off-by: Yangyu Chen <cyy@cyyself.name> > Signed-off-by: Yixun Lan <dlan@gentoo.org> > --- > .../devicetree/bindings/riscv/spacemit.yaml | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/riscv/spacemit.yaml: 'maintainers' is a required property hint: Metaschema for devicetree binding documentation from schema $id: http://devicetree.org/meta-schemas/base.yaml# doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240627-k1-01-basic-dt-v2-3-cc06c7555f07@gentoo.org The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. 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 after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 27/06/2024 17:31, Yixun Lan wrote: > From: Yangyu Chen <cyy@cyyself.name> > > Add DT binding documentation for the SpacemiT K1 Soc[1] and the Banana > Pi BPi-F3 board[2] which used it. > > [1] https://www.spacemit.com/en/spacemit-key-stone-2/ > [2] https://docs.banana-pi.org/en/BPI-F3/BananaPi_BPI-F3 You got a bug reported by tool and you send the same version again, producing the same bug. In case it is not clear: *You cannot ignore* bug reports, comments, reviewer requests or any other reply to your patchset. Each one must be addressed one way or another. Best regards, Krzysztof
On 09:34 Fri 28 Jun , Krzysztof Kozlowski wrote: > On 27/06/2024 17:31, Yixun Lan wrote: > > From: Yangyu Chen <cyy@cyyself.name> > > > > Add DT binding documentation for the SpacemiT K1 Soc[1] and the Banana > > Pi BPi-F3 board[2] which used it. > > > > [1] https://www.spacemit.com/en/spacemit-key-stone-2/ > > [2] https://docs.banana-pi.org/en/BPI-F3/BananaPi_BPI-F3 > > You got a bug reported by tool and you send the same version again, > producing the same bug. > > In case it is not clear: > > *You cannot ignore* bug reports, comments, reviewer requests or any > other reply to your patchset. Each one must be addressed one way or another. > sorry, it's my fault, I rushed to send out this series. and yes, I should really go back and check all the thread.. I will fix all the issues in next version (tags, dt error..)
On Thu, Jun 27, 2024 at 03:31:17PM +0000, Yixun Lan wrote: > From: Yangyu Chen <cyy@cyyself.name> > > Add DT binding documentation for the SpacemiT K1 Soc[1] and the Banana > Pi BPi-F3 board[2] which used it. > > [1] https://www.spacemit.com/en/spacemit-key-stone-2/ > [2] https://docs.banana-pi.org/en/BPI-F3/BananaPi_BPI-F3 > Please make these link tags when you resend, like Link: https://foo [1] and don't leave blank lines between them and the signoff. > Signed-off-by: Yangyu Chen <cyy@cyyself.name> > Signed-off-by: Yixun Lan <dlan@gentoo.org> > --- > .../devicetree/bindings/riscv/spacemit.yaml | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/Documentation/devicetree/bindings/riscv/spacemit.yaml b/Documentation/devicetree/bindings/riscv/spacemit.yaml > new file mode 100644 > index 0000000000000..3b151fd02473e > --- /dev/null > +++ b/Documentation/devicetree/bindings/riscv/spacemit.yaml > @@ -0,0 +1,24 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/riscv/spacemit.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: SpacemiT SoC-based boards > + > +description: > + SpacemiT SoC-based boards Please work out who is gonna maintain these SoCs and add that here to resolve the bot's report. Thanls, Conor. > + > +properties: > + $nodename: > + const: '/' > + compatible: > + oneOf: > + - items: > + - enum: > + - bananapi,bpi-f3 > + - const: spacemit,k1 > + > +additionalProperties: true > + > +... > > -- > 2.45.2 >
Hi On 13:24 Mon 01 Jul , Conor Dooley wrote: > On Thu, Jun 27, 2024 at 03:31:17PM +0000, Yixun Lan wrote: > > From: Yangyu Chen <cyy@cyyself.name> > > > > Add DT binding documentation for the SpacemiT K1 Soc[1] and the Banana > > Pi BPi-F3 board[2] which used it. > > > > [1] https://www.spacemit.com/en/spacemit-key-stone-2/ > > [2] https://docs.banana-pi.org/en/BPI-F3/BananaPi_BPI-F3 > > > > Please make these link tags when you resend, like > > Link: https://foo [1] > > and don't leave blank lines between them and the signoff. > sure, will fix in v3 > > Signed-off-by: Yangyu Chen <cyy@cyyself.name> > > Signed-off-by: Yixun Lan <dlan@gentoo.org> > > --- > > .../devicetree/bindings/riscv/spacemit.yaml | 24 ++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/riscv/spacemit.yaml b/Documentation/devicetree/bindings/riscv/spacemit.yaml > > new file mode 100644 > > index 0000000000000..3b151fd02473e > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/riscv/spacemit.yaml > > @@ -0,0 +1,24 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/riscv/spacemit.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: SpacemiT SoC-based boards > > + > > +description: > > + SpacemiT SoC-based boards > > Please work out who is gonna maintain these SoCs and add that here to > resolve the bot's report. > sure, talked to Yangyu, will also add him as maintainer this should fix bot's complaint.. btw, thanks for all your other comments in the whole thread, will fix them all > Thanls, > Conor. > > > + > > +properties: > > + $nodename: > > + const: '/' > > + compatible: > > + oneOf: > > + - items: > > + - enum: > > + - bananapi,bpi-f3 > > + - const: spacemit,k1 > > + > > +additionalProperties: true > > + > > +... > > > > -- > > 2.45.2 > >
diff --git a/Documentation/devicetree/bindings/riscv/spacemit.yaml b/Documentation/devicetree/bindings/riscv/spacemit.yaml new file mode 100644 index 0000000000000..3b151fd02473e --- /dev/null +++ b/Documentation/devicetree/bindings/riscv/spacemit.yaml @@ -0,0 +1,24 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/riscv/spacemit.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: SpacemiT SoC-based boards + +description: + SpacemiT SoC-based boards + +properties: + $nodename: + const: '/' + compatible: + oneOf: + - items: + - enum: + - bananapi,bpi-f3 + - const: spacemit,k1 + +additionalProperties: true + +...