Commit f9848cfa authored by David Hunter's avatar David Hunter Committed by Stephen Boyd
Browse files

da8xx-cfgchip.c: replace of_node_put with __free improves cleanup 



The use of the __free function allows the cleanup to be based on scope
instead of on another function called later. This makes the cleanup
automatic and less susceptible to errors later.

This code was compiled without errors or warnings.

Signed-off-by: default avatarDavid Hunter <david.hunter.linux@gmail.com>
Link: https://lore.kernel.org/r/20240720152447.311442-1-david.hunter.linux@gmail.com


Reviewed-by: default avatarDavid Lechner <david@lechnology.com>
Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent 8400291e

drivers/clk/davinci/da8xx-cfgchip.c

+1 −3
Original line number Diff line number Diff line
@@ -749,11 +749,9 @@ static int da8xx_cfgchip_probe(struct platform_device *pdev) 


	clk_init = device_get_match_data(dev);

	if (clk_init) {

		struct device_node *parent;

		struct device_node *parent __free(device_node) = of_get_parent(dev->of_node);



		parent = of_get_parent(dev->of_node);

		regmap = syscon_node_to_regmap(parent);

		of_node_put(parent);

	} else if (pdev->id_entry && pdata) {

		clk_init = (void *)pdev->id_entry->driver_data;

		regmap = pdata->cfgchip;