On 8 September 2016 at 19:38, Kevin Hilman <khilman@xxxxxxxxxxxx> wrote: > Ulf Hansson <ulf.hansson@xxxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
