Hi Marek,
On 9/7/2026 5:43 PM, Marek Vasut wrote:
On 9/7/26 1:56 PM, Balaji Selvanathan wrote:
_lmb_alloc_addr() returns -EINVAL when a reservation overlaps an
available memory bank but does not fully fit inside it, and -EFAULT
when it does not overlap any bank at all.
So for the partial-overlap case, it returns -EINVAL and
boot_fdt_handle_region() in boot/image-fdt.c prints "ERROR: reserving
fdt memory region failed" during boot which makes it sound like a
serious error.
The commit message does not explain why do you think the overlap is
not a serious error, and why it could be ignored.
Understood. Have reworded the commit description to better explain. Pls
find v2 here:
https://lore.kernel.org/u-boot/20260912-lmb-alloc-addr-efault-v2-1-0a796adfd...@oss.qualcomm.com/
Thanks,
Balaji
