[PATCH 01/16] lib: fdtdec: Handle multiple memory nodes
Simon Glass
sjg at chromium.org
Tue Jun 11 20:51:55 CEST 2024
Hi Jiaxun,
On Tue, 11 Jun 2024 at 07:48, Jiaxun Yang <jiaxun.yang at flygoat.com> wrote:
>
>
>
> 在2024年6月11日六月 下午1:26,Heinrich Schuchardt写道:
> Hi Heinrich,
>
> Thanks for your comments, will fix most of them in next reversion.
>
> [...]
> >> - gd->bd->bi_dram[bank].start = (phys_addr_t)res.start;
> >
> > The conversion is superfluous.
>
> This is necessary as phys_addr_t can be unsigned long or unsigned long long.
>
> >
> [...]
> >> + __maybe_unused const char *final_name;
> >> + __maybe_unused int final_reg;
> >
> > '__maybe_unused' is superfluous.
>
> This is necessary as when LOG_DEBUG is disabled these two variables won't
> be referenced. Compilers would complain about it.
>
> >
> >>
> >> gd->ram_base = (unsigned long)~0;
> >
> > gd->ram_base = ULONG_MAX;
> >
> >>
> [...]
>
> Thanks
> --
Can you please add a test for this function? Perhaps test/dm/fdtdec.c
might be a good place?
Regards,
Simon
> - Jiaxun
More information about the U-Boot
mailing list