[RFC PATCH v2 09/48] lmb: remove call to lmb_init()

[Simon Glass]

Hi Sughosh,

On Mon, 15 Jul 2024 at 10:31, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> hi Simon,
>
> On Sat, 13 Jul 2024 at 20:45, Simon Glass <sjg at chromium.org> wrote:
> >
> > kHi Sughosh,
> >
> > On Thu, 4 Jul 2024 at 08:36, Sughosh Ganu <sughosh.ganu at 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.ganu at 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