Thread (1 message) 1 message, 1 author, 2015-03-21
DORMANTno replies REVIEWED: 1 (0M)

1 review trailer.

[PATCH V2] PM / Domains: Sync runtime PM status with genpd after probe

From: khilman@kernel.org (Kevin Hilman)
Date: 2015-03-21 00:14:34
Also in: linux-pm

Ulf Hansson [off-list ref] writes:
From: Russell King <redacted>

Buses which currently supports attaching devices to their PM domains,
will invoke the dev_pm_domain_attach() API from their ->probe()
callbacks. During the attach procedure, genpd power up the PM domain.

In those scenarios where the bus/driver don't need to access its device
during probe, it may leave it in runtime PM suspended state since
that's also the default state. In that way, no notifications through
the runtime PM callbacks will reach the PM domain during probe.

For genpd, the consequence from the above scenario means the PM domain
will remain powered. Therefore, implement the struct dev_pm_domain's
->sync() callback, which is invoked from driver core after the
bus/driver has probed the device. It allows genpd to power off the PM
domain if it's unused.

Signed-off-by: Russell King <redacted>
[Ulf: Updated patch according to updates in driver core]
Signed-off-by: Ulf Hansson <redacted>
---

Changes in v2:
	- I picked up Russell's earlier patch [1] and updated it to fit the
	changes [2] proposed to be done to the driver core.

	[1]: [PATCH 3/9] pm: domains: sync runtime PM status with PM domains
	after probe

	[2]: [PATCH v3] driver core / PM: Add PM domain callbacks for device
	setup/cleanup
	http://www.spinics.net/lists/arm-kernel/msg407394.html
Ah, I hadn't seen this when I reviewed Rafael's v3 and suggested
Russell's patch be reworked.  This is just what I had in mind.

Acked-by: Kevin Hilman <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help