diff options
| author | Sughosh Ganu <sughosh.ganu@linaro.org> | 2025-03-03 19:02:29 +0530 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2025-03-17 19:39:27 -0600 |
| commit | f5f0a0287134223c16ce64303df60c3708684e6a (patch) | |
| tree | 617d34d9f39d7ff1fe317ba63c484c397b7d9037 /lib | |
| parent | 6e4df5886d27cff043561c8087f373e26cfe9f34 (diff) | |
lmb: remove superfluous address overlap check from lmb_add_region_flags()
U-Boot allows re-use of already reserved memory through the
lmb_reserve() and lmb_alloc_addr() API's. This memory re-use is
allowed only when the flag of the existing reserved region and that of
the requested region is LMB_NONE. A check was put in the
lmb_add_region_flags() in commit 8b8b35a4f5e to handle the scenario
where an already reserved region was re-requested with region flag
other than LMB_NONE -- the function then returns -EEXIST in such a
scenario.
The lmb_reserve() function now does a check for a reservation request
with existing reserved regions, and returns -EEXIST in case of an
overlap but when the flag check fails. Remove this now redundant check
from lmb_add_region_flags().
Signed-off-by: Sughosh Ganu <sughosh.ganu@linaro.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/lmb.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/lib/lmb.c b/lib/lmb.c index 53af96fa2a9..9af942c6b42 100644 --- a/lib/lmb.c +++ b/lib/lmb.c @@ -195,9 +195,6 @@ static long lmb_add_region_flags(struct alist *lmb_rgn_lst, phys_addr_t base, coalesced++; break; } else if (lmb_addrs_overlap(base, size, rgnbase, rgnsize)) { - if (flags != LMB_NONE) - return -EEXIST; - ret = lmb_resize_regions(lmb_rgn_lst, i, base, size); if (ret < 0) return -1; |