Re: [PATCH] clocksource/drivers/timer-ti-dm: fix child node refcount handling

From: Daniel Lezcano
Date: Mon Oct 28 2024 - 09:50:05 EST


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



--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog