From mboxrd@z Thu Jan  1 00:00:00 1970
From: Chris Packham <judge.packham@gmail.com>
Date: Mon, 13 May 2019 08:32:42 +1200
Subject: [U-Boot] [PATCH] arm: mvebu: armada-370-xp.dtsi: Add "u-boot,
 dm-pre-reloc" to "internal-regs"
In-Reply-To: <747816ff-8984-dbad-7da7-9d9d95931510@denx.de>
References: <20190510113405.17743-1-sr@denx.de>
 <CAFOYHZAa7fmwvqqhJGt3=aNPx0GP+VhEhept1QWZ6=HvNLbHQw@mail.gmail.com>
 <747816ff-8984-dbad-7da7-9d9d95931510@denx.de>
Message-ID: <CAFOYHZCZ4RMhWuKDLTze68EdgHx-Upio5ehq0SUF3596DxLj3g@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: u-boot@lists.denx.de

On Sun, May 12, 2019 at 10:09 PM Stefan Roese <sr@denx.de> wrote:
>
> On 11.05.19 02:09, Chris Packham wrote:
> >
> >
> > On Fri, 10 May 2019, 11:34 PM Stefan Roese, <sr at denx.de <mailto:sr@d=
enx.de>> wrote:
> >
> >     Without this U-Boot specific property, booting on Armada XP theador=
able
> >     fails in SPL. All nodes in the "internal-regs" (simple-bus) DT node=
 are
> >     not scanned, so the UART node is missing (and others).
> >
> >     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.
> >
> >     Signed-off-by: Stefan Roese <sr at denx.de <mailto:sr@denx.de>>
> >     Cc: Chris Packham <judge.packham at gmail.com <mailto:judge.packham=
@gmail.com>>
> >     Cc: Marek Beh=C3=BAn <marek.behun at nic.cz <mailto:marek.behun@nic=
.cz>>
> >
> >
> > Thanks. Is this a regression from my last sync? The only Armada XP
> > like platform I had access to was msys and that doesn't use SPL (yet).
>
> I'm not 100% sure if its only a regression caused by your DT sync or
> also by this patch:
>
> c7a88dae "dm: remove pre reloc properties in SPL and TPL device tree"
>
> I stopped debugging at some point, once I had the node enabled in the
> SPL DT again.
>
> > Reviewed-by: Chris Packham <judge.packham at gmail.com <mailto:judge.pa=
ckham@gmail.com>>

^^^ Gah, this is from the gmail app on my tablet it seems intent on
linkifying things now. This has been known to make the patchwork
client spew html into the patch when you download it.