On Wed, Dec 11, 2024 at 4:55 AM Ilias Apalodimas <[email protected]> wrote: > > There's no point defining a function that's called only once just to > avoid passing the flags. Remove the wrapper and just call > lmb_add_region_flags(). > > Acked-by: Sughosh Ganu <[email protected]> > Reviewed-by: Heinrich Schuchardt <[email protected]> > Signed-off-by: Ilias Apalodimas <[email protected]> > ---
Reviewed-by: Sam Protsenko <[email protected]> > lib/lmb.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/lib/lmb.c b/lib/lmb.c > index da960e422ada..659581f13f20 100644 > --- a/lib/lmb.c > +++ b/lib/lmb.c > @@ -632,19 +632,13 @@ void lmb_add_memory(void) > } > } > > -static long lmb_add_region(struct alist *lmb_rgn_lst, phys_addr_t base, > - phys_size_t size) > -{ > - return lmb_add_region_flags(lmb_rgn_lst, base, size, LMB_NONE); > -} > - > /* This routine may be called with relocation disabled. */ > long lmb_add(phys_addr_t base, phys_size_t size) > { > long ret; > struct alist *lmb_rgn_lst = &lmb.available_mem; > > - ret = lmb_add_region(lmb_rgn_lst, base, size); > + ret = lmb_add_region_flags(lmb_rgn_lst, base, size, LMB_NONE); > if (ret) > return ret; > > -- > 2.45.2 >
