From:   Gregory CLEMENT <gregory.clement@bootlin.com>
To:     Andrew Lunn <andrew@lunn.ch>,
        Pali =?utf-8?Q?Roh=C3=A1r?= <pali@kernel.org>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings
In-Reply-To: <YvJ2HKqMTTyWiNth@lunn.ch>
References: <20220713200123.22612-1-pali@kernel.org>
 <20220809131252.z3mbmiz4duxwlauw@pali> <YvJ2HKqMTTyWiNth@lunn.ch>
Date:   Wed, 01 Feb 2023 09:27:52 +0100
Message-ID: <87edr9bxiv.fsf@BL-laptop>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hello,

> On Tue, Aug 09, 2022 at 03:12:52PM +0200, Pali Roh=C3=A1r wrote:
>> + Andrew, Gregory, Sebastian
>>=20
>> 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

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