Hi Marek,
On 10/09/2015 10:49 PM, Marek Vasut wrote:
+ *handle = (unsigned long)memalign(ARCH_DMA_MINALIGN, len);
This still modifies the handle in both cases (failure and success).
We really want to modify external variables in case of failure, no?
The *handle return a DMA address, and the function itself return a
virtual address. Both of them should be set to NULL in case of failure.
It might be wrong to keep DMA address looks alive but actually dead.
Best regards,
Thomas Chou
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
