From patchwork Thu Dec 27 17:14:48 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aisheng Dong X-Patchwork-Id: 10743881 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 93CB791E for ; Thu, 27 Dec 2018 17:15:12 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7284828A5C for ; Thu, 27 Dec 2018 17:15:12 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 615A41FFD9; Thu, 27 Dec 2018 17:15:12 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 4C4961FFD9 for ; Thu, 27 Dec 2018 17:15:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=dYhvZ982UINCHr+i/jULGS77lB0S/37ziPFiSgTM9tw=; b=bvqsOxtZrkvydq cOja1j8pPorZZ48MsvBQ+qdRYIddNRctp6vPXDyAkjiMAyJwsR88YNNPJmDn4RFgoO8AuxzCcT3x1 3klpwRvjv1x5zIErX578rcgT8n/UcbHVzKzRnP19Ar3PkFVL5AdPxr0nxZs4lESPA8z7ZBMzjyNBB dXuUzU9cCBYCsv1ukzUvqvJZHTCxARm2w7wsqdnIN19OAnIAKQU6qYh1nAnEMvL+bVkBsLYg/R+ks M9iyYgB4pAx2l2gahOh63HWs4E9awuo6RgYXiDRJsy/uWf22Udnmo+bBrofAli8ueINe8/HYRHZdU sFy9NpDzIequx9KteB0A==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gcZFU-00041i-I0; Thu, 27 Dec 2018 17:15:08 +0000 Received: from mail-eopbgr00051.outbound.protection.outlook.com ([40.107.0.51] helo=EUR02-AM5-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gcZFQ-0003Gl-7q for linux-arm-kernel@lists.infradead.org; Thu, 27 Dec 2018 17:15:06 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+cTy30umQRbngbCO0UFlX+2TS1jCNAFUoXKXdYwMEUg=; b=eJaYp7DnukWeLtLcuN6oO6/YvT1slPBjNY0LOIy9yojGqVYyxhs392YQdJXKFqlIu1BH3dLvIL1L0juhFG/CcVgqFLN95Cxa6UVg/WevQcYujJda4SO7Ndi7p3qVREMdwoFk/Bu9tQhiK9ucFUI1XxZaISdj+2dC29Mz5FJK7NI= Received: from AM0PR04MB4211.eurprd04.prod.outlook.com (52.134.126.21) by AM0PR04MB5427.eurprd04.prod.outlook.com (20.178.116.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1471.20; Thu, 27 Dec 2018 17:14:48 +0000 Received: from AM0PR04MB4211.eurprd04.prod.outlook.com ([fe80::9125:e806:7303:d8e6]) by AM0PR04MB4211.eurprd04.prod.outlook.com ([fe80::9125:e806:7303:d8e6%7]) with mapi id 15.20.1446.027; Thu, 27 Dec 2018 17:14:48 +0000 From: Aisheng Dong To: "linux-pm@vger.kernel.org" Subject: [RFC PATCH 1/1] PM / Domains: Add multi PM domains support for attach_dev Thread-Topic: [RFC PATCH 1/1] PM / Domains: Add multi PM domains support for attach_dev Thread-Index: AQHUngerKaedheUOE0OdzOpUuVXDPw== Date: Thu, 27 Dec 2018 17:14:48 +0000 Message-ID: <1545930550-9906-1-git-send-email-aisheng.dong@nxp.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-mailer: git-send-email 2.7.4 x-clientproxiedby: HK0PR01CA0017.apcprd01.prod.exchangelabs.com (2603:1096:203:92::29) To AM0PR04MB4211.eurprd04.prod.outlook.com (2603:10a6:208:66::21) authentication-results: spf=none (sender IP is ) smtp.mailfrom=aisheng.dong@nxp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [119.31.174.66] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; AM0PR04MB5427; 6:+fskWIJCrt5qETjnmfbePtE3GceutJsWTQOwHLvqExwPdOJnvE2S/4ucOkvv/hjbGcM7VrTCFI1I0HRMeecPlFjJnaghjx5HxTngjBCHyYGAV7B3DhOF8OV75TFJvm3PlD2hy6s+3XWEtRPAcN9wZWUSUTd4x6NXjKwt2JwYeuIXtMRumev/Vp6oaBjGRxfFShU7XAaPmddOmoDmRed1sYF4YKrophNzFM00YUmWpKt3oUUYQck5KXIVC+zvp1dhOqHlR+mxRXNAFlaOckoOOxhpMUG8S0JCkQSk+cm4cs3mGly4Oww83iy11X+ifKa5p0GKKoAXtrWANg4gHyzWNfHFcxQ1EA620JG4xbCQqTL97Ay8HI2W/JKOX8CFfJjhBvxoBkU15FLOY7RKWq2hKLKnSMps56dUhh+A36dBE1vYsqHRaq4ZVxU1pJR6pgjse+iXv9oXWGUtLQzrzRZgLQ==; 5:McyEVeiMVs4vMHil1P0iCoXyigCCE306ZrWRFN+GyGe7MqNP/KAMPTL3xgJVd+/Wdh4MKpzqt79msDRJ5LqxiiOdNhmoodpHqG070ro9mERV3/83eujHq2Nayzqv+nbBvv4Ba7t66tjktnr+cAhmxRjgJY1X2GKORYB/ki5W8j4=; 7:z1PC1DJ8NZ4fGm/8RBmAkHJ37p5kKKWjLdXraay2TOKi0yLWsjMGQEwVfYIFtXOPMj0TGTh7zqqk4ztIjss+N50uVbhiyuWSZz1Yt48zQD0sdilJE8WngIOIIzXF+GDHc2FbLFmicrrvkAA40PyQrg== x-ms-office365-filtering-correlation-id: c2c9c871-2dd3-43b5-34fe-08d66c1ece28 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:AM0PR04MB5427; x-ms-traffictypediagnostic: AM0PR04MB5427: x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(3230021)(908002)(999002)(5005026)(6040522)(2401047)(8121501046)(10201501046)(3002001)(93006095)(93001095)(3231475)(944501520)(52105112)(6055026)(6041310)(20161123562045)(20161123560045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(201708071742011)(7699051)(76991095); SRVR:AM0PR04MB5427; BCL:0; PCL:0; RULEID:; SRVR:AM0PR04MB5427; x-forefront-prvs: 0899B47777 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(366004)(396003)(346002)(136003)(376002)(52314003)(199004)(189003)(8936002)(97736004)(5660300001)(53936002)(4326008)(5024004)(3846002)(50226002)(86362001)(6916009)(316002)(966005)(256004)(14454004)(2616005)(14444005)(71190400001)(71200400001)(476003)(478600001)(7736002)(36756003)(305945005)(6116002)(39060400002)(6486002)(217873002)(68736007)(486006)(2501003)(186003)(2906002)(99286004)(25786009)(6306002)(54906003)(102836004)(6512007)(44832011)(6436002)(5640700003)(7416002)(8676002)(106356001)(105586002)(2351001)(6506007)(386003)(81156014)(81166006)(66066001)(26005)(52116002)(21314003); DIR:OUT; SFP:1101; SCL:1; SRVR:AM0PR04MB5427; H:AM0PR04MB4211.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: gzUPVIHohWAlRn6dwanv20LkYOQI9GTBvLOGTWgSBFRqz9lq1k+bJZC6Tbzm05A4JIu9QgUdWDBibX3+HLgEP2ssZtw5jK31/nXSBs+jqE3VOK2eD2QME1KYyW3NRGyXu2vaJ4EbzbrM1n88dujHz04PWNGGNWJDxH0CakW25P03i/rQdfUfiQ2UNRSyFT4m5UStSKforYiceLhPkZKV1ix4YyT/zuLT+iCsh3PghnOlJRe6Dekxkvvpt3s4OuvJ7W363WmwWlir8Ezdno3vr8IXS8SiECFhbru2M1f5x96AWIRAS89M0wp+wL1P5LjA spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: c2c9c871-2dd3-43b5-34fe-08d66c1ece28 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Dec 2018 17:14:48.7367 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB5427 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181227_091504_294450_2989207E X-CRM114-Status: GOOD ( 22.71 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Aisheng Dong , "ulf.hansson@linaro.org" , "dongas86@gmail.com" , "khilman@kernel.org" , Greg Kroah-Hartman , "rjw@rjwysocki.net" , "linux-kernel@vger.kernel.org" , dl-linux-imx , "kernel@pengutronix.de" , Fabio Estevam , "shawnguo@kernel.org" , "linux-arm-kernel@lists.infradead.org" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Currently attach_dev() in power domain infrastructure still does not support multi domains case as the struct device *dev passed down from genpd_dev_pm_attach_by_id() is a virtual PD device, it does not help for parsing the real device information from device tree, e.g. Device/Power IDs, Clocks and it's unware of which real power domain the device should attach. Extend the framework a bit to store the multi PM domains information in per-device struct generic_pm_domain_data, then power domain driver could retrieve it for necessary operations during attach_dev(). Two new APIs genpd_is_mpd_device() and dev_gpd_mpd_data() are also introduced to ease the driver operation. Cc: "Rafael J. Wysocki" Cc: Kevin Hilman Cc: Ulf Hansson Cc: Greg Kroah-Hartman Signed-off-by: Dong Aisheng --- This patch is a follow-up work of the earlier discussion with Ulf Hansson about the multi PM domains support for the attach_dev() function [1]. After a bit more thinking, this is a less intrusive implementation with the mininum impact on the exist function definitions and calling follows. One known little drawback is that we have to use the device driver private data (device.drvdata) to pass down the multi domains information in a earlier time. However, as multi PD devices are created by domain framework, this seems to be safe to use it in domain core code as device driver is not likely going to use it. Anyway, if any better ideas, please let me know. With the two new APIs, the using can be simply as: static int xxx_attach_dev(struct generic_pm_domain *domain, struct device *dev) { ... if (genpd_is_mpd_device(dev)) { mpd_data = dev_gpd_mpd_data(dev); np = mpd_data->parent->of_node; idx = mpd_data->index; //dts parsing ... } ... } [1] https://patchwork.kernel.org/patch/10658669/ --- drivers/base/power/domain.c | 31 +++++++++++++++++++++++++++++++ include/linux/pm_domain.h | 23 +++++++++++++++++++++++ 2 files changed, 54 insertions(+) diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index 7f38a92..1aa0918 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -1343,6 +1343,9 @@ static struct generic_pm_domain_data *genpd_alloc_dev_data(struct device *dev, gpd_data->td.effective_constraint_ns = PM_QOS_RESUME_LATENCY_NO_CONSTRAINT_NS; gpd_data->nb.notifier_call = genpd_dev_pm_qos_notifier; + if (genpd_is_mpd_device(dev)) + gpd_data->mpd_data = dev_get_drvdata(dev); + spin_lock_irq(&dev->power.lock); if (dev->power.subsys_data->domain_data) { @@ -2179,6 +2182,7 @@ EXPORT_SYMBOL_GPL(of_genpd_remove_last); static void genpd_release_dev(struct device *dev) { + kfree(dev->driver_data); kfree(dev); } @@ -2320,6 +2324,20 @@ int genpd_dev_pm_attach(struct device *dev) EXPORT_SYMBOL_GPL(genpd_dev_pm_attach); /** + * genpd_is_mpd_device - Check if a device is associated with multi PM domains + * @dev: Device to check. + */ + +bool genpd_is_mpd_device(struct device *dev) +{ + if (!dev || (dev && !dev->bus)) + return false; + + return dev->bus == &genpd_bus_type; +}; +EXPORT_SYMBOL_GPL(genpd_is_mpd_device); + +/** * 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. @@ -2338,6 +2356,7 @@ EXPORT_SYMBOL_GPL(genpd_dev_pm_attach); struct device *genpd_dev_pm_attach_by_id(struct device *dev, unsigned int index) { + struct pm_domain_mpd_data *mpd_data; struct device *genpd_dev; int num_domains; int ret; @@ -2366,6 +2385,18 @@ struct device *genpd_dev_pm_attach_by_id(struct device *dev, return ERR_PTR(ret); } + /* Allocate multi power domains data */ + mpd_data = kzalloc(sizeof(*mpd_data), GFP_KERNEL); + if (!mpd_data) { + device_unregister(genpd_dev); + return ERR_PTR(-ENOMEM); + } + + mpd_data->parent = dev; + mpd_data->index = index; + + dev_set_drvdata(genpd_dev, mpd_data); + /* 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); if (ret < 1) { diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h index 3b5d728..106d4e7 100644 --- a/include/linux/pm_domain.h +++ b/include/linux/pm_domain.h @@ -144,6 +144,11 @@ struct gpd_timing_data { bool cached_suspend_ok; }; +struct pm_domain_mpd_data { + struct device *parent; + unsigned int index; +}; + struct pm_domain_data { struct list_head list_node; struct device *dev; @@ -151,6 +156,7 @@ struct pm_domain_data { struct generic_pm_domain_data { struct pm_domain_data base; + struct pm_domain_mpd_data *mpd_data; struct gpd_timing_data td; struct notifier_block nb; unsigned int performance_state; @@ -262,10 +268,17 @@ unsigned int of_genpd_opp_to_performance_state(struct device *dev, struct device_node *np); int genpd_dev_pm_attach(struct device *dev); +bool genpd_is_mpd_device(struct device *dev); struct device *genpd_dev_pm_attach_by_id(struct device *dev, unsigned int index); struct device *genpd_dev_pm_attach_by_name(struct device *dev, char *name); + +static inline struct pm_domain_mpd_data *dev_gpd_mpd_data(struct device *dev) +{ + return dev_gpd_data(dev)->mpd_data; +} + #else /* !CONFIG_PM_GENERIC_DOMAINS_OF */ static inline int of_genpd_add_provider_simple(struct device_node *np, struct generic_pm_domain *genpd) @@ -311,6 +324,11 @@ static inline int genpd_dev_pm_attach(struct device *dev) return 0; } +static bool genpd_is_mpd_device(struct device *dev) +{ + return false; +} + static inline struct device *genpd_dev_pm_attach_by_id(struct device *dev, unsigned int index) { @@ -323,6 +341,11 @@ static inline struct device *genpd_dev_pm_attach_by_name(struct device *dev, return NULL; } +static inline struct pm_domain_mpd_data *dev_gpd_mpd_data(struct device *dev) +{ + return ERR_PTR(-ENOSYS); +} + static inline struct generic_pm_domain *of_genpd_remove_last(struct device_node *np) {