Commit 39eaf521 authored by Javier Carrasco's avatar Javier Carrasco Committed by Viresh Kumar
Browse files

cpufreq: dt-platdev: eliminate uses of of_node_put()



Make use of the __free() cleanup handler to automatically free nodes
when they get out of scope.

Signed-off-by: default avatarJavier Carrasco <javier.carrasco.cruz@gmail.com>
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
parent 68090fda
Loading
Loading
Loading
Loading
+2 −5
Original line number Diff line number Diff line
@@ -198,19 +198,18 @@ static const struct of_device_id blocklist[] __initconst = {

static bool __init cpu0_node_has_opp_v2_prop(void)
{
	struct device_node *np = of_cpu_device_node_get(0);
	struct device_node *np __free(device_node) = of_cpu_device_node_get(0);
	bool ret = false;

	if (of_property_present(np, "operating-points-v2"))
		ret = true;

	of_node_put(np);
	return ret;
}

static int __init cpufreq_dt_platdev_init(void)
{
	struct device_node *np = of_find_node_by_path("/");
	struct device_node *np __free(device_node) = of_find_node_by_path("/");
	const struct of_device_id *match;
	const void *data = NULL;

@@ -226,11 +225,9 @@ static int __init cpufreq_dt_platdev_init(void)
	if (cpu0_node_has_opp_v2_prop() && !of_match_node(blocklist, np))
		goto create_pdev;

	of_node_put(np);
	return -ENODEV;

create_pdev:
	of_node_put(np);
	return PTR_ERR_OR_ZERO(platform_device_register_data(NULL, "cpufreq-dt",
			       -1, data,
			       sizeof(struct cpufreq_dt_platform_data)));