Re: [PATCH v2] pstore/ram: Replace of_node_put with __free() for automatic cleanup

2024-06-09 Thread Abhinav Jain
On Wed, Jun 05, 2024 at 09:49:44PM, Kees Cook wrote: > The history and version links should be below the "---" line. I have made the change. > > But more importantly, please base your patch on the upstream tree, > rather than on your v1 patch. :) (i.e. squash your v1 and v2 together). I have squ

Re: [PATCH v2] pstore/ram: Replace of_node_put with __free() for automatic cleanup

2024-06-06 Thread Kees Cook
On Wed, Jun 05, 2024 at 09:49:44PM +, Abhinav Jain wrote: > Add __free(device_node) to the parent_node struct declaration > Add changes to incorporate the review comments from v1 > > Signed-off-by: Abhinav Jain > > PATCH v1 link: > https://lore.kernel.org/all/20240415161409.8375-1-jain.abhin

[PATCH v2] pstore/ram: Replace of_node_put with __free() for automatic cleanup

2024-06-05 Thread Abhinav Jain
Add __free(device_node) to the parent_node struct declaration Add changes to incorporate the review comments from v1 Signed-off-by: Abhinav Jain PATCH v1 link: https://lore.kernel.org/all/20240415161409.8375-1-jain.abhinav...@gmail.com/ Changes since v1: - Moved the variable definition back to