[PATCH 2/7] global_data.h: Change ram_top type to phys_addr_t

Bin Meng bmeng.cn at gmail.com
Sat Jan 30 11:13:52 CET 2021


Hi Simon,

On Sat, Jan 30, 2021 at 1:48 AM Simon Glass <sjg at chromium.org> wrote:
>
> Hi Bin,
>
> On Sat, 23 Jan 2021 at 19:03, Simon Glass <sjg at chromium.org> wrote:
> >
> > On Thu, 21 Jan 2021 at 08:00, Bin Meng <bmeng.cn at gmail.com> wrote:
> > >
> > > From: Bin Meng <bin.meng at 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 at windriver.com>
> > > ---
> > >
> > >  include/asm-generic/global_data.h | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Reviewed-by: Simon Glass <sjg at 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


More information about the U-Boot mailing list