From patchwork Thu May 31 10:59:58 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 137360 Delivered-To: patches@linaro.org Received: by 2002:a2e:9706:0:0:0:0:0 with SMTP id r6-v6csp6441374lji; Thu, 31 May 2018 04:00:12 -0700 (PDT) X-Received: by 2002:a19:de0a:: with SMTP id v10-v6mr3848775lfg.94.1527764412860; Thu, 31 May 2018 04:00:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1527764412; cv=none; d=google.com; s=arc-20160816; b=Eh5Nxp5ok251G+qw+XMLMvjGuf22S4VwRCM0uk84TALV+XTP1IBzc9rZzXDWfnNmwf 5OlvuT4mL4mnFhipGeqK7tv03O4bb+1AQflQnCPhn2MDv3GldwYj5207EUu0/+unB/hq ufMrBoKr03blw5z12WlbjD7Z+6pCLli9/Rb4xZXjRr2NZNre6QtI/p3GCdE7wJhPa3DJ whXIe4gPND5zrLNMNIG7bMW+QRaz1bfexqtp+XntJ23dCRY6msANphm+1xxP8f0dhwZR cZlCeYRkQ0W17UNU7e08ZsEyhqZfbvCuPtOkA9GIRW4qwBCr2FGQ5rRmD563HuDyt1PA ImZA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature:arc-authentication-results; bh=PNYOIw9NEsuNENF5roerUbH6PEOPf9UriVVq7srO1ao=; b=rG/qswjQi/aOz9cSPBiShJdvH3nVJbUdMsO3WY1eQPB0NLKIHd/ZMVDxkj50JkpzF4 qzjWxel9I8Z/RIDyg3sPDuhlWPuOnbKnHv9Xnzpth4s78tziWWCct85pqsIs6B6DYQwY yxAVyd977y7aHajuGhcOI/Wpt7MQuBB/Nb9QQnJsZuLyzSah2zqZr3hLyziNF6w3qvj8 y2kilvv+/8YfazzJNAN8ILAvUYS3Ar67v5Hiz/SGXa+vxds93qax600/dR1sVfP+4qIN 1ucDjTcia+HTFc5Q25BV6XdqJ88i0afInMB8WeX/s+8kun7/1p7OwltmDHL5UVeMA9Xj ECwQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=C2m386/Z; spf=pass (google.com: domain of ulf.hansson@linaro.org designates 209.85.220.65 as permitted sender) smtp.mailfrom=ulf.hansson@linaro.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id a76-v6sor8310206lfa.79.2018.05.31.04.00.12 for (Google Transport Security); Thu, 31 May 2018 04:00:12 -0700 (PDT) Received-SPF: pass (google.com: domain of ulf.hansson@linaro.org designates 209.85.220.65 as permitted sender) client-ip=209.85.220.65; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=C2m386/Z; spf=pass (google.com: domain of ulf.hansson@linaro.org designates 209.85.220.65 as permitted sender) smtp.mailfrom=ulf.hansson@linaro.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=PNYOIw9NEsuNENF5roerUbH6PEOPf9UriVVq7srO1ao=; b=C2m386/ZCM+hg1E2O+VE+FkwYAtu5xNQ26xZF9Fz8mG+u6jkyFahRscn4u+LZRuyN/ ipQ18eKIs+dfRnSlXO3Zxaht2kUjOxyf/ifF+yE4gUqKkzZWUfGAi7AAOTsaOuHbWWG4 65DOfsmdxnrwiQ/5BtwmNEKi09XL5ri93Cmpw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=PNYOIw9NEsuNENF5roerUbH6PEOPf9UriVVq7srO1ao=; b=BmvUtxPsFnPGmbgsoXDSCO803a/zu1PqO50sKKtJq8DnD3m/URV42Hj+2eH7UGDk55 R2ACJkK54Nakkb/tjc/ACkVq2+LBfXTApi1oGQYwZnzBbNoRMwyFX1Y/hD9EDbu47ceG qYWhVvnN/04wvseyfZe9LpK504OEIm/rX2DbT9aY+65W2lv8UtqsvM9XV1DzpPv7mqoB rCkkd1efUbkdlsH8AWJU42zjlgF27hhs+ADp8YheUM5aWI2OSQDERuOuSx2XlNQ4porl 1vAb4NwGcK2o23lwS7UomVeoY4pgvyjTQRwsN6IgBRyDjazHaQKTfWGqCpBQwkongAeJ NFMA== X-Gm-Message-State: ALKqPwcLmBup3UPjWaTPK++fI9i9UMKFC0U8UBt09M9V0nJZQd0Zx5+R ULBdCnLcN2ziVxwU4cs37cIH5ZOa X-Google-Smtp-Source: ADUXVKJuk4Gi5a4S+ThnajpbClUgsgcJqET+6SIyP7UDuFt9kiFWKtUUYGPN5weYlJr7YDLgQrH2Sg== X-Received: by 2002:a19:f611:: with SMTP id x17-v6mr4141689lfe.116.1527764412581; Thu, 31 May 2018 04:00:12 -0700 (PDT) Return-Path: Received: from localhost.localdomain (h-158-174-22-210.NA.cust.bahnhof.se. [158.174.22.210]) by smtp.gmail.com with ESMTPSA id b20-v6sm7454383ljf.51.2018.05.31.04.00.11 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 31 May 2018 04:00:11 -0700 (PDT) From: Ulf Hansson To: "Rafael J . Wysocki" , linux-pm@vger.kernel.org Cc: Ulf Hansson , Greg Kroah-Hartman , Jon Hunter , Geert Uytterhoeven , Todor Tomov , Rajendra Nayak , Viresh Kumar , Vincent Guittot , Kevin Hilman , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-tegra@vger.kernel.org Subject: [PATCH v3 4/5] PM / Domains: Add support for multi PM domains per device to genpd Date: Thu, 31 May 2018 12:59:58 +0200 Message-Id: <20180531105959.14843-5-ulf.hansson@linaro.org> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180531105959.14843-1-ulf.hansson@linaro.org> References: <20180531105959.14843-1-ulf.hansson@linaro.org> To support devices being partitioned across multiple PM domains, let's begin with extending genpd to cope with these kind of configurations. Therefore, add a new exported function genpd_dev_pm_attach_by_id(), which is similar to the existing genpd_dev_pm_attach(), but with the difference that it allows its callers to provide an index to the PM domain that it wants to attach. Note that, genpd_dev_pm_attach_by_id() shall only be called by the driver core / PM core, similar to how the existing dev_pm_domain_attach() makes use of genpd_dev_pm_attach(). However, this is implemented by following changes on top. Because, only one PM domain can be attached per device, genpd needs to create a virtual device that it can attach/detach instead. More precisely, let the new function genpd_dev_pm_attach_by_id() register a virtual struct device via calling device_register(). Then let it attach this device to the corresponding PM domain, rather than the one that is provided by the caller. The actual attaching is done via re-using the existing genpd OF functions. At successful attachment, genpd_dev_pm_attach_by_id() returns the created virtual device, which allows the caller to operate on it to deal with power management. Following changes on top, provides more details in this regards. To deal with detaching of a PM domain for the multiple PM domains case, let's also extend the existing genpd_dev_pm_detach() function, to cover the cleanup of the created virtual device, via make it call device_unregister() on it. In this way, there is no need to introduce a new function to deal with detach for the multiple PM domain case, but instead the existing one is re-used. Signed-off-by: Ulf Hansson Acked-by: Jon Hunter Tested-by: Jon Hunter Reviewed-by: Viresh Kumar --- drivers/base/power/domain.c | 80 +++++++++++++++++++++++++++++++++++++ include/linux/pm_domain.h | 8 ++++ 2 files changed, 88 insertions(+) -- 2.17.0 diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index b1fcbf917974..4925af5c4cf0 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -2171,6 +2171,15 @@ struct generic_pm_domain *of_genpd_remove_last(struct device_node *np) } EXPORT_SYMBOL_GPL(of_genpd_remove_last); +static void genpd_release_dev(struct device *dev) +{ + kfree(dev); +} + +static struct bus_type genpd_bus_type = { + .name = "genpd", +}; + /** * genpd_dev_pm_detach - Detach a device from its PM domain. * @dev: Device to detach. @@ -2208,6 +2217,10 @@ static void genpd_dev_pm_detach(struct device *dev, bool power_off) /* Check if PM domain can be powered off after removing this device. */ genpd_queue_power_off_work(pd); + + /* Unregister the device if it was created by genpd. */ + if (dev->bus == &genpd_bus_type) + device_unregister(dev); } static void genpd_dev_pm_sync(struct device *dev) @@ -2298,6 +2311,67 @@ int genpd_dev_pm_attach(struct device *dev) } EXPORT_SYMBOL_GPL(genpd_dev_pm_attach); +/** + * genpd_dev_pm_attach_by_id - Associate a device with one of its PM domains. + * @dev: The device used to lookup the PM domain. + * @index: The index of the PM domain. + * + * Parse device's OF node to find a PM domain specifier at the provided @index. + * If such is found, creates a virtual device and attaches it to the retrieved + * pm_domain ops. To deal with detaching of the virtual device, the ->detach() + * callback in the struct dev_pm_domain are assigned to genpd_dev_pm_detach(). + * + * Returns the created virtual device if successfully attached PM domain, NULL + * when the device don't need a PM domain, else an ERR_PTR() in case of + * failures. If a power-domain exists for the device, but cannot be found or + * turned on, then ERR_PTR(-EPROBE_DEFER) is returned to ensure that the device + * is not probed and to re-try again later. + */ +struct device *genpd_dev_pm_attach_by_id(struct device *dev, + unsigned int index) +{ + struct device *genpd_dev; + int num_domains; + int ret; + + if (!dev->of_node) + return NULL; + + /* Deal only with devices using multiple PM domains. */ + num_domains = of_count_phandle_with_args(dev->of_node, "power-domains", + "#power-domain-cells"); + if (num_domains < 2 || index >= num_domains) + return NULL; + + /* Allocate and register device on the genpd bus. */ + genpd_dev = kzalloc(sizeof(*genpd_dev), GFP_KERNEL); + if (!genpd_dev) + return ERR_PTR(-ENOMEM); + + dev_set_name(genpd_dev, "genpd:%u:%s", index, dev_name(dev)); + genpd_dev->bus = &genpd_bus_type; + genpd_dev->release = genpd_release_dev; + + ret = device_register(genpd_dev); + if (ret) { + kfree(genpd_dev); + return ERR_PTR(ret); + } + + /* Try to attach the device to the PM domain at the specified index. */ + ret = __genpd_dev_pm_attach(genpd_dev, dev->of_node, index); + if (ret < 1) { + device_unregister(genpd_dev); + return ret ? ERR_PTR(ret) : NULL; + } + + pm_runtime_set_active(genpd_dev); + pm_runtime_enable(genpd_dev); + + return genpd_dev; +} +EXPORT_SYMBOL_GPL(genpd_dev_pm_attach_by_id); + static const struct of_device_id idle_state_match[] = { { .compatible = "domain-idle-state", }, { } @@ -2457,6 +2531,12 @@ unsigned int of_genpd_opp_to_performance_state(struct device *dev, } EXPORT_SYMBOL_GPL(of_genpd_opp_to_performance_state); +static int __init genpd_bus_init(void) +{ + return bus_register(&genpd_bus_type); +} +core_initcall(genpd_bus_init); + #endif /* CONFIG_PM_GENERIC_DOMAINS_OF */ diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h index 42e0d649e653..82458e8e2e01 100644 --- a/include/linux/pm_domain.h +++ b/include/linux/pm_domain.h @@ -237,6 +237,8 @@ unsigned int of_genpd_opp_to_performance_state(struct device *dev, struct device_node *opp_node); int genpd_dev_pm_attach(struct device *dev); +struct device *genpd_dev_pm_attach_by_id(struct device *dev, + unsigned int index); #else /* !CONFIG_PM_GENERIC_DOMAINS_OF */ static inline int of_genpd_add_provider_simple(struct device_node *np, struct generic_pm_domain *genpd) @@ -282,6 +284,12 @@ static inline int genpd_dev_pm_attach(struct device *dev) return 0; } +static inline struct device *genpd_dev_pm_attach_by_id(struct device *dev, + unsigned int index) +{ + return NULL; +} + static inline struct generic_pm_domain *of_genpd_remove_last(struct device_node *np) {