[PATCH v5 09/10] tools: spkgimage: add Renesas SPKG format
Ralph Siemsen
ralph.siemsen at linaro.org
Fri May 12 22:14:29 CEST 2023
On Tue, May 09, 2023 at 11:42:30AM -0400, Ralph Siemsen wrote:
>On Tue, May 09, 2023 at 04:52:45PM +0200, Marek Vasut wrote:
>>
>>Do we have some sort of global (?) state structure which exists
>>during the whole work cycle of the tool ? If so, add a link list
>>into there.
>
>There is struct image_tool_params which is passed to the callbacks and
>holds most of the state. And in fact it is a global, but declared
>static in mkimage.c, without any accessor function.
>
>If we really want to worry about the lifecycle of these dynamic
>allocations, then we'd probably need to add some kind of "cleanup"
>method to the API, and call it right before exiting from main().
>
>I can do an example implementation, but there are 9 other image
>formats already in the tree, which also do dynamic allocation. I don't
>want to start touching each one of those, or I'll never get this RZ/N1
>in...
I'll keep the current logic (which leaks) and send a separate RFC patch
to discuss a strategy for fixing this in all the tools/*image.c drivers.
Ralph
More information about the U-Boot
mailing list