[PATCH 1/2] arm: mvebu: armada-370-xp.dtsi: add u-boot,dm-pre-reloc to spi0
ezra at easyb.ch
Sat May 9 10:10:37 CEST 2020
On 5 May 2020, at 10:58, Stefan Roese <sr at denx.de> wrote:
> Hi Ezra,
> On 05.05.20 09:10, Ezra Buehler wrote:
>> On 5 May 2020, at 08:45, Stefan Roese <sr at denx.de> wrote:
>>> U-Boot specific DT properties, like "u-boot,dm-pre-reloc" should be
>>> added in U-Boot specific DT dtsi files
>> OK, that makes sense. However, in your commit 1718a9f3b7 (arm: mvebu:
>> armada-370-xp.dtsi: Add "u-boot, dm-pre-reloc" to "internal-regs") you
>> I'm not adding this property in an *u-boot.dtsi file, since there is
>> none matching the generic rules for all files including this dtsi
>> file. So to not miss any of the boards using this dtsi file, I'm
>> adding it to this file directly, which makes the Linux merge a less
>> easy unforunately.
>> So, do you want me to try this now?
> Hmmm, I can't remember the exact details. Thanks for the reference.
> But perhaps there is a matching *-u-boot.dtsi rule / pattern that can
> be used to include such a newly introduced file. I would very much
> welcome it, when you would investigate here.
From looking at other *.dtsi files I understand one would have to
include the armada-370-xp-u-boot.dtsi in armada-370-xp.dtsi in order to
move the U-Boot-specific properties out. I guess one should then do this
for other boards too. I am not sure if I am confortable with touching
all these boards. Maybe one should do this the next time when syncing
the DTS files with Linux.
If it is OK for you, for this patch, I would simply introduce a
armada-xp-synology-ds414-u-boot.dtsi as Chris suggested.
More information about the U-Boot