diff options
| author | Pranav Tilak <[email protected]> | 2026-05-07 17:03:59 +0530 |
|---|---|---|
| committer | Michal Simek <[email protected]> | 2026-05-25 15:14:05 +0200 |
| commit | 9793931f36d585878305ff0bf907b32138673b95 (patch) | |
| tree | 4c89ead32f66492e86cd37cf4e4a2bcb84151e4d /doc/develop/devicetree | |
| parent | 987c93fc68a641cc735c9828872511a947e54191 (diff) | |
fpga: versalpl: Fix unaligned buffer handling
When fpga load is called with a misaligned buffer address, the
versal_align_dma_buffer() function shifts the pointer forward to the
next aligned boundary and uses memcpy() to copy the data. Since the
destination is ahead of the source and the regions overlap, memcpy()
produces undefined behavior; in practice U-Boot's generic memcpy()
copies forward, repeating the first ARCH_DMA_MINALIGN-aligned chunk
throughout the buffer.
Replace memcpy() with memmove() which correctly handles overlapping
regions by copying backwards when the destination is ahead of the
source.
Fixes: 26e054c943a7 ("arm64: versal: fpga: Add PL bit stream load support")
Signed-off-by: Pranav Tilak <[email protected]>
Signed-off-by: Michal Simek <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'doc/develop/devicetree')
0 files changed, 0 insertions, 0 deletions
