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