On 28/10/2024 13:26, Javier Carrasco wrote:
[ ... ]
Hi Daniel, thanks for your feedback.
Actually, if we are going to refactor the code, we would not need the extra variable or even the call to of_node_put(), since we could use the __free() macro. That would be a second patch after the fix, which could stay as it is without refactoring, because it is only to backport the missing calls to of_node_put().
I can send a v2 with the extra patch leaving this one as it is, or if really desired, with the available variable.
V2 with __free is fine