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 <sughosh.g...@linaro.org>
Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>
Reviewed-by: Sam Protsenko <semen.protse...@linaro.org>
Tested-by: Sam Protsenko <semen.protse...@linaro.org>
Signed-off-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>
---
 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

Reply via email to