On Sun, Dec 08, 2024 at 12:52:04PM +0200, Ilias Apalodimas wrote:

> lmb_reserve() is just calling lmb_reserve_flags() with LMB_NONE.
> There's not much we gain from this abstraction, so let's remove the
> former and make the code a bit easier to follow.
> 
> The code size increase is minimal - e.g for sandbox which includes all
> of the LMB tests
> 
> add/remove: 0/1 grow/shrink: 12/0 up/down: 46/-4 (42)
> Function                                     old     new   delta
> lib_test_lmb_overlapping_reserve            1018    1030     +12
> version_string                                70      76      +6
> test_get_unreserved_size                    1032    1038      +6
> test_alloc_addr                             2933    2939      +6
> test_multi_alloc.constprop                  3034    3036      +2
> test_bigblock                                911     913      +2
> load_serial                                  946     948      +2
> lib_test_lmb_flags                          2101    2103      +2
> do_bootz                                     526     528      +2
> do_bootm_linux                              2067    2069      +2
> bootm_run_states                            5275    5277      +2
> boot_relocate_fdt                            599     601      +2
> lmb_reserve                                    4       -      -4
> Total: Before=2492742, After=2492784, chg +0.00%
> 
> Signed-off-by: Ilias Apalodimas <[email protected]>

Reviewed-by: Tom Rini <[email protected]>

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to