On Monday, September 12, 2016 03:02:13 PM Ulf Hansson wrote:
On 12 September 2016 at 13:01, Jon Hunter [off-list ref] wrote:
quoted
In order to safely remove PM domains there are a few changes that need to be
made to ensure that no one is holding an external reference to a PM domain
after it has been removed. One solution, implemented here, solves this by
eliminating external references to PM domain.
Changes from V1:
- Added test when removing a PM domain to ensure the PM domain provider
has been removed.
- Re-organised patches 8-10 so that patch #8 adds the code to verify if
the provider is present, patch #9 adds basic support for removing PM
domains and patch #10 adds support for removing PM domains by provider.
Changes from initial RFC:
- Renamed functions made static per Ulf's feedback.
- Added patch to clean-up provider/xlate APIs per Ulf's feedback
- Re-worked and simplified the association between PM domains and PM
domain providers. Dropped the 'provider_data' variable from the
generic_pm_domain structure in favour of using the fwnode_handle.
- Split patch for removing PM domains into multiple patches per Ulf's
feedback.
Jon Hunter (10):
PM / Domains: Add new helper functions for device-tree
ARM: EXYNOS: Remove calls to of_genpd_get_from_provider()
staging: board: Remove calls to of_genpd_get_from_provider()
PM / Domains: Don't expose generic_pm_domain structure to clients
PM / Domains: Don't expose xlate and provider helper functions
PM / Domains: Verify the PM domain is present when adding a provider
PM / Domains: Prepare for adding support to remove PM domains
PM / Domains: Store the provider in the PM domain structure
PM / Domains: Add support for removing PM domains
PM / Domains: Add support for removing nested PM domains by provider
drivers/base/power/domain.c | 353 +++++++++++++++++++++++++++++++++++----
drivers/soc/samsung/pm_domains.c | 23 +--
drivers/staging/board/board.c | 9 +-
include/linux/pm_domain.h | 74 ++++----
4 files changed, 364 insertions(+), 95 deletions(-)
--
2.1.4
Jon, thanks for being patient and keep working on this series! I have
now completed the review and I am happy with this version.
Also, I remember Krzysztof requested an immutable branch for this
series, I guess Rafael can arrange that if still needed!?
Yes, I can do that.
I think I'll get to these patches later today and if not, then tomorrow.
Thanks,
Rafael