在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 -- - Jiaxun
- [PATCH 00/16] LoongArch initial support Jiaxun Yang
- [PATCH 01/16] lib: fdtdec: Handle multiple memory... Jiaxun Yang
- Re: [PATCH 01/16] lib: fdtdec: Handle multipl... Heinrich Schuchardt
- Re: [PATCH 01/16] lib: fdtdec: Handle mul... Jiaxun Yang
- Re: [PATCH 01/16] lib: fdtdec: Handle... Simon Glass
- Re: [PATCH 01/16] lib: fdtdec: H... Heinrich Schuchardt
- [PATCH 02/16] linux/io.h: Use map_physmem to impl... Jiaxun Yang
- [PATCH 03/16] image: Take entry point as an outpu... Jiaxun Yang
- Re: [PATCH 03/16] image: Take entry point as ... Heinrich Schuchardt
- Re: [PATCH 03/16] image: Take entry point... Jiaxun Yang
- Re: [PATCH 03/16] image: Take entry p... Tom Rini
- Re: [PATCH 03/16] image: Take en... Jiaxun Yang
- Re: [PATCH 03/16] image: Tak... Tom Rini
- [PATCH 04/16] elf.h Define LoongArch bits Jiaxun Yang