Hi Sughosh, On Mon, 15 Jul 2024 at 10:31, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > hi Simon, > > On Sat, 13 Jul 2024 at 20:45, Simon Glass <s...@chromium.org> wrote: > > > > kHi Sughosh, > > > > On Thu, 4 Jul 2024 at 08:36, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > > > > > The LMB module will be changed to have persistent and global memory > > > maps of available and used memory. With this change, there won't be > > > any need to explicitly initialise the LMB memory maps. Remove the call > > > to the lmb_init() function. > > > > > > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org> > > > --- > > > Changes since V1: New patch > > > > > > arch/arm/mach-stm32mp/dram_init.c | 1 - > > > board/xilinx/common/board.c | 1 - > > > drivers/iommu/apple_dart.c | 1 - > > > drivers/iommu/sandbox_iommu.c | 1 - > > > include/lmb.h | 1 - > > > lib/lmb.c | 18 ------------------ > > > test/lib/lmb.c | 18 ------------------ > > > 7 files changed, 41 deletions(-) > > > > I can't actually find the call to lmb_init(). Where is it? > > Sorry, I do not understand this question. This patch removes the > lmb_init() definition, as well as calls to the function from the rest > of the files.
I see later you add lmb_mem_regions_init() so that there are two init functions. We should really only have one. Regards, Simon
