linux-kernel - Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

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]

Date:   Tue, 9 Aug 2022 15:13:57 +0200
From:   Pali Rohár <pali@...nel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform
 bindings

On Tuesday 09 August 2022 08:58:50 Krzysztof Kozlowski wrote:
> On 08/08/2022 23:23, Pali Rohár wrote:
> > PING?
> > 
> > On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> >> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> >> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> >> indicate SoC in example.
> >>
> >> Signed-off-by: Pali Rohár <pali@...nel.org>
> >> ---
> >>  .../devicetree/bindings/arm/marvell/armada-37xx.txt        | 7 ++++++-
> >>  1 file changed, 6 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> index f6d6642d81c0..d2ca008de266 100644
> >> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> >>  Boards using a SoC of the Marvell Armada 37xx family must carry the
> >>  following root node property:
> >>  
> >> + - compatible: must contain "marvell,armada3700"
> >> +
> >> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> >> +following property before the previous one:
> >> +
> 
> The change is an ABI break, which looks reasonable, but still platform
> maintainer should comment on it. Especially on the aspect why the
> marvell,armada3710 fallback was chosen at the first place.

I do not think this is ABI break but rather incorrect documentation and
bug in some board dts files.

> Unfortunately none of Marvel folks were CCed here, so you will need to
> resend it. Otherwise no one will apply it, regardless of our ack.

Done.