From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752955AbcIIIeo (ORCPT ); Fri, 9 Sep 2016 04:34:44 -0400 Received: from mail-wm0-f53.google.com ([74.125.82.53]:32774 "EHLO mail-wm0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751693AbcIIIel (ORCPT ); Fri, 9 Sep 2016 04:34:41 -0400 MIME-Version: 1.0 In-Reply-To: References: <20160819235653.26355-1-nm@ti.com> <20160819235653.26355-4-nm@ti.com> <57C0D2C9.1030801@ti.com> <57C5E1DA.2040405@ti.com> <57CF26DB.4020807@ti.com> <7h4m5rv9kz.fsf@baylibre.com> From: Ulf Hansson Date: Fri, 9 Sep 2016 10:34:38 +0200 Message-ID: Subject: Re: [PATCH 3/3] soc: ti: Add ti_sci_pm_domains driver To: Kevin Hilman Cc: Dave Gerlach , Nishanth Menon , "Rafael J. Wysocki" , Keerthy , Peter Ujfalusi , Tero Kristo , Russell King , Sudeep Holla , Santosh Shilimkar , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-pm@vger.kernel.org" , Jon Hunter Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8 September 2016 at 19:38, Kevin Hilman wrote: > Ulf Hansson writes: > >> [...] >> >>> >>> One more idea... >>> >>> Since you don't really have a domain (a group of devices), what you >>> really have is each device having an independent power switch, so as Ulf >>> suggested, what you really need is for all the devices to share the same >>> set of runtime PM callbacks that call SCI. The only difference is the >>> unique ID. >>> >>> Rather than using all of genpd, you could also just use a pm_domain >>> which is what genpd is built on top of (and also omap_device, which >>> you're probably familiar with also.) >> >> Even if this would work as well, the downside would be that you need >> to re-invent the parts related to the DT parsing, the probing/removal >> and attaching/detaching of the device to the PM domain. >> >> You probably don't want to go there... :-) > > All you'd need to read from DT would be the device-specific ID for > TI-SCI, and that could be done at bind time with a notifier. The, in > that same notifier, if a TI-SCI ID exists, it would get added to the > pm_domain. > > Anyways, your original proposal is much preferred if it can work. I'm > just throwing out another option because I really don't like one genpd > per device. Okay, then we are in full agreement! BTW, I have just been trying to convince other people working on Rockchip SoCs, to also avoid using one genpd per device. Feel free to join those discussions [1] as well. :-) Kind regards Uffe [1] https://lkml.org/lkml/2016/9/1/377