linux-kernel - Re: [PATCH v2 0/4] Cleanup Spreadtum/Unisoc ARM64 DT

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: <172286967300.2710789.8114229281680847298.robh@kernel.org>
Date: Mon, 05 Aug 2024 09:00:20 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Stanislav Jakubek <stano.jakubek@...il.com>
Cc: Baolin Wang <baolin.wang7@...il.com>, Orson Zhai <orsonzhai@...il.com>, 
 devicetree@...r.kernel.org, Krzysztof Kozlowski <krzk+dt@...nel.org>, 
 linux-kernel@...r.kernel.org, Conor Dooley <conor+dt@...nel.org>, 
 linux-arm-kernel@...ts.infradead.org, 
 Baolin Wang <baolin.wang@...ux.alibaba.com>, 
 Chunyan Zhang <zhang.lyra@...il.com>
Subject: Re: [PATCH v2 0/4] Cleanup Spreadtum/Unisoc ARM64 DT


On Mon, 05 Aug 2024 09:23:25 +0200, Stanislav Jakubek wrote:
> This series contains various minor cleanups for Spreadtum/Unisoc ARM64
> DeviceTrees. Also contains the 2nd patch from the SC2731 MFD cleanup series
> to reduce unnecessary noise and make these easier to apply.
> 
> Changes in V2:
>   - add 2nd patch from the SC2731 MFD cleanup series (1st patch was dropped
>     due to not being correct, dt-bindings will have to get fixed up instead)
>   - add Baolin's R-b
>   - rebase on next-20240805
> 
> Link to original V1: https://lore.kernel.org/lkml/cover.1720112081.git.stano.jakubek@gmail.com/
> Link to SC2731 MFD cleanup V1: https://lore.kernel.org/lkml/cover.1720957783.git.stano.jakubek@gmail.com/
> 
> Stanislav Jakubek (4):
>   arm64: dts: sprd: rename SDHCI nodes to mmc
>   arm64: dts: sprd: reorder clock-names after clocks
>   arm64: dts: sprd: move/add SPDX license to top of the file
>   arm64: dts: sprd: sc2731: rename fuel gauge node to be generic
> 
>  arch/arm64/boot/dts/sprd/sc2731.dtsi          |  5 +--
>  arch/arm64/boot/dts/sprd/sc9836-openphone.dts |  3 +-
>  arch/arm64/boot/dts/sprd/sc9836.dtsi          |  3 +-
>  arch/arm64/boot/dts/sprd/sc9860.dtsi          |  3 +-
>  arch/arm64/boot/dts/sprd/sc9863a.dtsi         |  8 ++--
>  arch/arm64/boot/dts/sprd/sharkl64.dtsi        |  3 +-
>  arch/arm64/boot/dts/sprd/sp9860g-1h10.dts     |  3 +-
>  arch/arm64/boot/dts/sprd/ums512.dtsi          |  4 +-
>  arch/arm64/boot/dts/sprd/whale2.dtsi          | 43 ++++++++++---------
>  9 files changed, 36 insertions(+), 39 deletions(-)
> 
> --
> 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 sprd/sc9836-openphone.dtb sprd/sp9860g-1h10.dtb' for cover.1722842066.git.stano.jakubek@...il.com:

arch/arm64/boot/dts/sprd/sp9860g-1h10.dtb: fuel-gauge@a00: 'battery-detect-gpios' is a required property
	from schema $id: http://devicetree.org/schemas/power/supply/sc27xx-fg.yaml#
arch/arm64/boot/dts/sprd/sp9860g-1h10.dtb: fuel-gauge@a00: 'sprd,calib-resistance-micro-ohms' is a required property
	from schema $id: http://devicetree.org/schemas/power/supply/sc27xx-fg.yaml#
arch/arm64/boot/dts/sprd/sp9860g-1h10.dtb: fuel-gauge@a00: 'bat-detect-gpio', 'interrupt-parent', 'interrupts' do not match any of the regexes: 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/power/supply/sc27xx-fg.yaml#