Re: [PATCH 01/16] lib: fdtdec: Handle multiple memory nodes

在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