Re: [PATCH] xen/arm: Skip memory nodes if not enabled

[Leo Yan <leo.yan@xxxxxxxxxx>]

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