Thread (22 messages) read the whole thread 22 messages, 3 authors, 2025-10-10

Re: [PATCH 03/10] pmdomain: samsung: use to devm_kstrdup_const() to simplify error handling

From: André Draszik <andre.draszik@linaro.org>
Date: 2025-10-09 13:31:22
Also in: linux-devicetree, linux-pm, linux-samsung-soc, lkml

On Thu, 2025-10-09 at 09:13 +0900, Krzysztof Kozlowski wrote:
On 07/10/2025 01:43, André Draszik wrote:
quoted
Convert to using devm_kstrdup_const() so as to simplify cleanup during
error handling.

This is either a fix (then describe the fixed issue and add Fixed tag)
or you change the logic, not only simplify.

Previously on of_genpd_add_provider_simple() the memory was not
kfree_const. Now it will be.
Indeed it's a fix after all - While the driver doesn't allow unbind,
I added this patch due to the followup patches that add potential error
returns during probe, but somehow missed an already-existing error
return after this call.

Thanks Krzysztof!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help