linux-kernel - Re: [PATCH v3 0/2] Add Bananapi R3 Mini

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <171534910744.4114753.13367969845540831259.robh@kernel.org>
Date: Fri, 10 May 2024 08:53:06 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Frank Wunderlich <linux@...web.de>
Cc: devicetree@...r.kernel.org, linux-clk@...r.kernel.org, 
 Stephen Boyd <sboyd@...nel.org>, linux-leds@...r.kernel.org, 
 linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org, 
 Frank Wunderlich <frank-w@...lic-files.de>, 
 Eric Woudstra <ericwouds@...il.com>, Tianling Shen <cnsztl@...ortalwrt.org>, 
 Jakub Kicinski <kuba@...nel.org>, linux-arm-kernel@...ts.infradead.org, 
 Pavel Machek <pavel@....cz>, Conor Dooley <conor+dt@...nel.org>, 
 "David S. Miller" <davem@...emloft.net>, 
 Michael Turquette <mturquette@...libre.com>, Lee Jones <lee@...nel.org>, 
 Eric Dumazet <edumazet@...gle.com>, 
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, 
 Matthias Brugger <matthias.bgg@...il.com>, netdev@...r.kernel.org, 
 Paolo Abeni <pabeni@...hat.com>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
Subject: Re: [PATCH v3 0/2] Add Bananapi R3 Mini

On Fri, 10 May 2024 11:57:05 +0200, Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@...lic-files.de>
> 
> Add mt7986 based BananaPi R3 Mini SBC.
> 
> changes in v3:
> - fixed unicde parenthesis in commit description of dts patch
> - dropped "dts:" from title of binding patch
> - added AB to binding and RB to dts patch
> 
> changes in v2:
> - dropped patches for unrealated binding fixes which are already fixed in next.
> - add missing node for nand
> - add some information about the board in description
> 
> change dts based on review from angelo+krzysztof
> 
>  - drop fan status
>  - rename phy14 to phy0 and phy15 to phy1
>  - drop default-trigger from phys and so also the binding-patch
>  - use regulator names based on regexp regulator-[0-9]+v[0-9]+
>  - add comment for pwm
> 
> Frank Wunderlich (2):
>   dt-bindings: arm64: mediatek: add BananaPi R3 Mini
>   arm64: dts: mediatek: Add  mt7986 based Bananapi R3 Mini
> 
>  .../devicetree/bindings/arm/mediatek.yaml     |   1 +
>  arch/arm64/boot/dts/mediatek/Makefile         |   1 +
>  .../mediatek/mt7986a-bananapi-bpi-r3-mini.dts | 493 ++++++++++++++++++
>  3 files changed, 495 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-mini.dts
> 
> --
> 2.34.1
> 
> 
> 

My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade

New warnings running 'make CHECK_DTBS=y mediatek/mt7986a-bananapi-bpi-r3-mini.dtb' for 20240510095707.6895-1-linux@...web.de:

arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-mini.dtb: crypto@...20000: interrupts: [[0, 116, 4], [0, 117, 4], [0, 118, 4], [0, 119, 4]] is too short
	from schema $id: http://devicetree.org/schemas/crypto/inside-secure,safexcel.yaml#
arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-mini.dtb: crypto@...20000: interrupt-names: ['ring0', 'ring1', 'ring2', 'ring3'] is too short
	from schema $id: http://devicetree.org/schemas/crypto/inside-secure,safexcel.yaml#