Hi Michal, Julien, On Wed, Sep 27, 2023 at 02:49:23PM +0200, Michal Orzel wrote:
[...] > Either way is fine. The advantage of placing the check in boot_fdt_info() is > that we can have a single check instead of duplicated and we do the check > before > the "first" use which happens to be the banks sorting. The advantage of > setup_mm() > is that it is the one dealing with memory banks and is called after > early_print_info() > so user can see some additional info. > > @Leo, will you send a patch for that? Don't feel obliged as it is not > strictly related > to your patch in which case I can handle it. @Michal, since you have much better sense than me for adding check for memory banks, I'd like to leave it to you. I just refined the patch v2 according to your comments and sent out for review. Thanks, Leo
