Commit a73776ca authored by Ulf Hansson's avatar Ulf Hansson
Browse files

pmdomain: core: Use of_fwnode_handle() 



Let's avoid accessing the np->fwnode directly and use the common helper
of_fwnode_handle() instead.

Suggested-by: default avatarSaravana Kannan <saravanak@google.com>
Reviewed-by: default avatarDhruva Gole <d-gole@ti.com>
Tested-by: Hiago De Franco <hiago.franco@toradex.com> # Colibri iMX8X
Tested-by: default avatarTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Link: https://lore.kernel.org/r/20250523134025.75130-2-ulf.hansson@linaro.org
parent 152d59f1

drivers/pmdomain/core.c

+5 −5
Original line number Diff line number Diff line
@@ -2557,7 +2557,7 @@ static int genpd_add_provider(struct device_node *np, genpd_xlate_t xlate, 
	cp->node = of_node_get(np);

	cp->data = data;

	cp->xlate = xlate;

	fwnode_dev_initialized(&np->fwnode, true);

	fwnode_dev_initialized(of_fwnode_handle(np), true);



	mutex_lock(&of_genpd_mutex);

	list_add(&cp->link, &of_genpd_providers);
@@ -2727,7 +2727,7 @@ void of_genpd_del_provider(struct device_node *np) 
			 * so that the PM domain can be safely removed.

			 */

			list_for_each_entry(gpd, &gpd_list, gpd_list_node) {

				if (gpd->provider == &np->fwnode) {

				if (gpd->provider == of_fwnode_handle(np)) {

					gpd->has_provider = false;



					if (gpd->opp_table) {
@@ -2737,7 +2737,7 @@ void of_genpd_del_provider(struct device_node *np) 
				}

			}



			fwnode_dev_initialized(&cp->node->fwnode, false);

			fwnode_dev_initialized(of_fwnode_handle(cp->node), false);

			list_del(&cp->link);

			of_node_put(cp->node);

			kfree(cp);
@@ -2916,7 +2916,7 @@ struct generic_pm_domain *of_genpd_remove_last(struct device_node *np) 


	mutex_lock(&gpd_list_lock);

	list_for_each_entry_safe(gpd, tmp, &gpd_list, gpd_list_node) {

		if (gpd->provider == &np->fwnode) {

		if (gpd->provider == of_fwnode_handle(np)) {

			ret = genpd_remove(gpd);

			genpd = ret ? ERR_PTR(ret) : gpd;

			break;
@@ -3269,7 +3269,7 @@ static int genpd_parse_state(struct genpd_power_state *genpd_state, 


	genpd_state->power_on_latency_ns = 1000LL * exit_latency;

	genpd_state->power_off_latency_ns = 1000LL * entry_latency;

	genpd_state->fwnode = &state_node->fwnode;

	genpd_state->fwnode = of_fwnode_handle(state_node);



	return 0;

}