[PATCH 1/2] arm: mvebu: armada-370-xp.dtsi: add u-boot,dm-pre-reloc to spi0

[Stefan Roese]

Hi Ezra,

On 09.05.20 10:10, Ezra Buehler wrote:
> 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
>>> wrote:
>>>    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.
>>
>> Thanks,
>> Stefan
> 
>  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.

Yes, please continue this way.

Thanks,
Stefan