[PATCH v2 2/7] common: Tidy up how malloc() is inited

Tom Rini trini at konsulko.com
Sat Oct 12 03:33:11 CEST 2024


On Fri, Oct 11, 2024 at 03:40:26PM -0600, Simon Glass wrote:

> The call to malloc() is a bit strange. The naming of the arguments
> suggests that an address is passed, but in fact it is a pointer, at
> least in the board_init_r() function and SPL equivalent.
> 
> Update it to work as described. Add a function comment as well.
> 
> Note that this does adjustment does not extend into the malloc()
> implementation itself, apart from changing mem_malloc_init(), since
> there are lots of casts and pointers and integers are used
> interchangeably.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>

Reviewed-by: Tom Rini <trini at konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241011/8d1dc50c/attachment.sig>


More information about the U-Boot mailing list