On 12/19/24 15:50, Venkatesh Yadav Abbarapu wrote:
As the return type is "int" for find_ram_top() function and
returning the "base" which is of phys_addr_t is breaking when the
"base" address is 64-bit. So fix this by updating the return type as
phys_addr_t.
Signed-off-by: Venkatesh Yadav Abbarapu <[email protected]>
---
common/memtop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/memtop.c b/common/memtop.c
index 841d89e0799..114ba64067d 100644
--- a/common/memtop.c
+++ b/common/memtop.c
@@ -121,7 +121,7 @@ static long region_overlap_check(struct mem_region
*mem_rgn, phys_addr_t base,
return (i < mem_rgn->count) ? i : -1;
}
-static int find_ram_top(struct mem_region *free_mem,
+static phys_addr_t find_ram_top(struct mem_region *free_mem,
struct mem_region *reserved_mem, phys_size_t size)
You should actually align this line too to match (.
With that fixed feel free to add
Reviewed-by: Michal Simek <[email protected]>
and I like to get this to 2025.01 version.
Thanks,
Michal
