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] [day] [month] [year] [list]

Message-ID: <87edr9bxiv.fsf@BL-laptop>
Date:   Wed, 01 Feb 2023 09:27:52 +0100
From:   Gregory CLEMENT <gregory.clement@...tlin.com>
To:     Andrew Lunn <andrew@...n.ch>,
        Pali Rohár <pali@...nel.org>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

Hello,

> On Tue, Aug 09, 2022 at 03:12:52PM +0200, Pali Rohár wrote:
>> + Andrew, Gregory, Sebastian
>> 
>> please comment on this
>
> Krzysztof said:
>
>> 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 was not involved in the armada37xx enablement. I have no idea why
> marvell,armada3710 fallback was chosen.
>
> Gregory might know, he is the author of this file.
>
> It does however seem reasonable. And a grep does suggest this
> compatible is purely cosmetic, it is not matched on anywhere.

This compatible was for the single core version of the Armada 37xx.
I think the idea behind using it as fallback was the fact that if it
works for aramda 3720 then it works for armada 3710.

Gregory

>
> 	   Andrew

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com