Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 560928b2 authored by Viresh Kumar's avatar Viresh Kumar
Browse files

PM / Domains: Rename genpd virtual devices as virt_dev 



There are several struct device instances that genpd core handles. The
most common one is the consumer device structure, which is named
(correctly) as "dev" within genpd core. The second one is the genpd's
device structure, referenced as genpd->dev. The third one is the virtual
device structures created by the genpd core to represent the consumer
device for multiple power domain case, currently named as genpd_dev. The
naming of these virtual devices isn't very clear or readable and it
looks more like the genpd->dev.

Rename the virtual device instances within the genpd core as "virt_dev".

Reviewed-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
parent 65102238

drivers/base/power/domain.c

+13 −13
Original line number Original line Diff line number Diff line
@@ -2338,7 +2338,7 @@ EXPORT_SYMBOL_GPL(genpd_dev_pm_attach); 
struct device *genpd_dev_pm_attach_by_id(struct device *dev,

struct device *genpd_dev_pm_attach_by_id(struct device *dev,

					 unsigned int index)

					 unsigned int index)

{

{

	struct device *genpd_dev;

	struct device *virt_dev;

	int num_domains;

	int num_domains;

	int ret;

	int ret;
@@ -2352,31 +2352,31 @@ struct device *genpd_dev_pm_attach_by_id(struct device *dev, 
		return NULL;

		return NULL;





	/* Allocate and register device on the genpd bus. */

	/* Allocate and register device on the genpd bus. */

	genpd_dev = kzalloc(sizeof(*genpd_dev), GFP_KERNEL);

	virt_dev = kzalloc(sizeof(*virt_dev), GFP_KERNEL);

	if (!genpd_dev)

	if (!virt_dev)

		return ERR_PTR(-ENOMEM);

		return ERR_PTR(-ENOMEM);





	dev_set_name(genpd_dev, "genpd:%u:%s", index, dev_name(dev));

	dev_set_name(virt_dev, "genpd:%u:%s", index, dev_name(dev));

	genpd_dev->bus = &genpd_bus_type;

	virt_dev->bus = &genpd_bus_type;

	genpd_dev->release = genpd_release_dev;

	virt_dev->release = genpd_release_dev;





	ret = device_register(genpd_dev);

	ret = device_register(virt_dev);

	if (ret) {

	if (ret) {

		kfree(genpd_dev);

		kfree(virt_dev);

		return ERR_PTR(ret);

		return ERR_PTR(ret);

	}

	}





	/* Try to attach the device to the PM domain at the specified index. */

	/* Try to attach the device to the PM domain at the specified index. */

	ret = __genpd_dev_pm_attach(genpd_dev, dev->of_node, index, false);

	ret = __genpd_dev_pm_attach(virt_dev, dev->of_node, index, false);

	if (ret < 1) {

	if (ret < 1) {

		device_unregister(genpd_dev);

		device_unregister(virt_dev);

		return ret ? ERR_PTR(ret) : NULL;

		return ret ? ERR_PTR(ret) : NULL;

	}

	}





	pm_runtime_enable(genpd_dev);

	pm_runtime_enable(virt_dev);

	genpd_queue_power_off_work(dev_to_genpd(genpd_dev));

	genpd_queue_power_off_work(dev_to_genpd(virt_dev));





	return genpd_dev;

	return virt_dev;

}

}

EXPORT_SYMBOL_GPL(genpd_dev_pm_attach_by_id);

EXPORT_SYMBOL_GPL(genpd_dev_pm_attach_by_id);