Commit 49a27e27 authored by Ulf Hansson's avatar Ulf Hansson Committed by Rafael J. Wysocki
Browse files

PM / Domains: Add generic data pointer to struct genpd_power_state 



Add a data pointer to the genpd_power_state struct, to allow a genpd
backend driver to store per-state specific data. To introduce the
pointer, change the way genpd deals with freeing of the corresponding
allocated data.

More precisely, clarify the responsibility of whom that shall free the
data, by adding a ->free_states() callback to the generic_pm_domain
structure. The one allocating the data will be expected to set the
callback, to allow genpd to invoke it from genpd_remove().

Co-developed-by: default avatarLina Iyer <lina.iyer@linaro.org>
Acked-by: default avatarDaniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
[ rjw: Subject & changelog ]
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent c324f43a

drivers/base/power/domain.c

+10 −2
Original line number Diff line number Diff line
@@ -1686,6 +1686,12 @@ out: 
}

EXPORT_SYMBOL_GPL(pm_genpd_remove_subdomain);



static void genpd_free_default_power_state(struct genpd_power_state *states,

					   unsigned int state_count)

{

	kfree(states);

}



static int genpd_set_default_power_state(struct generic_pm_domain *genpd)

{

	struct genpd_power_state *state;
@@ -1696,7 +1702,7 @@ static int genpd_set_default_power_state(struct generic_pm_domain *genpd) 


	genpd->states = state;

	genpd->state_count = 1;

	genpd->free = state;

	genpd->free_states = genpd_free_default_power_state;



	return 0;

}
@@ -1812,7 +1818,9 @@ static int genpd_remove(struct generic_pm_domain *genpd) 
	list_del(&genpd->gpd_list_node);

	genpd_unlock(genpd);

	cancel_work_sync(&genpd->power_off_work);

	kfree(genpd->free);

	if (genpd->free_states)

		genpd->free_states(genpd->states, genpd->state_count);



	pr_debug("%s: removed %s\n", __func__, genpd->name);



	return 0;

include/linux/pm_domain.h

+3 −1
Original line number Diff line number Diff line
@@ -69,6 +69,7 @@ struct genpd_power_state { 
	s64 residency_ns;

	struct fwnode_handle *fwnode;

	ktime_t idle_time;

	void *data;

};



struct genpd_lock_ops;
@@ -110,9 +111,10 @@ struct generic_pm_domain { 
			   struct device *dev);

	unsigned int flags;		/* Bit field of configs for genpd */

	struct genpd_power_state *states;

	void (*free_states)(struct genpd_power_state *states,

			    unsigned int state_count);

	unsigned int state_count; /* number of states */

	unsigned int state_idx; /* state that genpd will go to when off */

	void *free; /* Free the state that was allocated for default */

	ktime_t on_time;

	ktime_t accounting_time;

	const struct genpd_lock_ops *lock_ops;

CRA Git | Maintained and supported by SUSTech CRA and CCSE