[PATCH] memory: fix dangling if in for_each_reserved_region

Ahmad Fatoum a.fatoum at pengutronix.de
Thu Dec 11 12:42:16 PST 2025


This macro will cause issues if followed by an if else statement.
Invert the condition to fix this.

Signed-off-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
---
 include/memory.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/memory.h b/include/memory.h
index 65614b97ec8b..6a6f0e2eb7d3 100644
--- a/include/memory.h
+++ b/include/memory.h
@@ -28,7 +28,7 @@ int barebox_add_memory_bank(const char *name, resource_size_t start,
 #define for_each_memory_bank(mem)	list_for_each_entry(mem, &memory_banks, list)
 #define for_each_reserved_region(mem, rsv) \
 	list_for_each_entry(rsv, &(mem)->res->children, sibling) \
-		if (is_reserved_resource(rsv))
+		if (!is_reserved_resource(rsv)) {} else
 
 #define for_each_memory_bank_region(bank, region) \
 	for_each_resource_region((bank)->res, region)
-- 
2.47.3




More information about the barebox mailing list