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