From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bin Meng <bmeng.cn@gmail.com>
Date: Sat, 30 Jan 2021 18:13:52 +0800
Subject: [PATCH 2/7] global_data.h: Change ram_top type to phys_addr_t
In-Reply-To: <CAPnjgZ2pVaacE6LzHXKKk4U2pWrzdjoP65WQr+oups=A4J3ASQ@mail.gmail.com>
References: <20210121150015.25558-1-bmeng.cn@gmail.com>
 <20210121150015.25558-3-bmeng.cn@gmail.com>
 <CAPnjgZ3ghiXcv4EE3Ru+g7-DbWZdc1pe0fO=vPtxpzuC66wvpw@mail.gmail.com>
 <CAPnjgZ2pVaacE6LzHXKKk4U2pWrzdjoP65WQr+oups=A4J3ASQ@mail.gmail.com>
Message-ID: <CAEUhbmVAmu_AmH=254_+Crrpbg6ffXUTbuPot1F=Y6hx7Q0h+g@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

On Sat, Jan 30, 2021 at 1:48 AM Simon Glass <sjg@chromium.org> wrote:
>
> Hi Bin,
>
> On Sat, 23 Jan 2021 at 19:03, Simon Glass <sjg@chromium.org> wrote:
> >
> > On Thu, 21 Jan 2021 at 08:00, Bin Meng <bmeng.cn@gmail.com> wrote:
> > >
> > > From: Bin Meng <bin.meng@windriver.com>
> > >
> > > It's possible to have ram_top above 4 GiB in a 32-bit system, hence
> > > we need to declare ram_top as `phys_addr_t`.
> > >
> > > Signed-off-by: Bin Meng <bin.meng@windriver.com>
> > > ---
> > >
> > >  include/asm-generic/global_data.h | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Reviewed-by: Simon Glass <sjg@chromium.org>
>
> This patch causes build warnings...can you please take a look?
>
> https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/213226

That's strange. I did run Azure before I posted this series, and Azure
did not report any failure.

See https://dev.azure.com/bmeng/GitHub/_build/results?buildId=307&view=results

Did you apply all patches in this series?

Regards,
Bin