]> git.ipfire.org Git - thirdparty/u-boot.git/commit - lib/lmb.c
lmb: Fix LMB_MEMORY_REGIONS flag usage
authorPatrick Delaunay <patrick.delaunay@foss.st.com>
Wed, 22 Mar 2023 18:12:25 +0000 (19:12 +0100)
committerTom Rini <trini@konsulko.com>
Thu, 6 Apr 2023 19:08:46 +0000 (15:08 -0400)
commit94c8da21216ac077842c413a4339e71285bcf94d
tree9667b7d73b5af6fc6f975d05a1f519a2744fe8c0
parentb0b77fdf3d7d2c1a5e48c3971a677f14e372c164
lmb: Fix LMB_MEMORY_REGIONS flag usage

Remove test on CONFIG_LMB_MEMORY_REGIONS introduced by commit
7c1860fce4e3 ("lmb: Fix lmb property's defination under struct lmb").

This code in lmb_init() is strange, because if CONFIG_LMB_USE_MAX_REGIONS
and CONFIG_LMB_MEMORY_REGIONS are not defined, the implicit #else is empty
and the required initialization is not done:
lmb->memory.max = ?
lmb->reserved.max = ?

But this setting is not possible:
- CONFIG_LMB_USE_MAX_REGIONS not defined
- CONFIG_LMB_MEMORY_REGIONS not defined
because CONFIG_LMB_MEMORY_REGIONS and CONFIG_LMB_RESERVED_REGIONS are
defined as soon as the CONFIG_LMB_USE_MAX_REGIONS is not defined.

This patch removes this impossible case #elif and I add some
explanation in lmb.h to explain why in the struct lmb {} the lmb
property is defined if CONFIG_LMB_MEMORY_REGIONS is NOT defined.

This patch also removes CONFIG_LMB_XXX dependency on CONFIG_LMB as these
defines are used in API file lmb.h and not only in library file.

Fixes: 5e2548c1d6e03 ("lmb: Fix LMB_MEMORY_REGIONS flag usage")
Reported-by: Mark Millard <marklmi@yahoo.com>
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Acked-by: Michal Simek <michal.simek@amd.com>
include/lmb.h
lib/Kconfig
lib/lmb.c