Hi,

Please see my comment on the patch.

Regards, Simon

On Fri, 19 Nov 2021 at 11:11, Andy Shevchenko andy.shevchenko@gmail.com wrote:

Hi!

It seems that lib/physmem.c is either not used or used only by sandbox (not sure). Do we need CONFIG_PHYSMEM at all?

Note x86 has its own implementation that is always present for all platforms.

-- With Best Regards, Andy Shevchenko