All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@baylibre.com>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: "cristian.marussi@arm.com" <cristian.marussi@arm.com>,
	"souvik.chakravarty@arm.com" <souvik.chakravarty@arm.com>,
	Sudeep Holla <sudeep.holla@arm.com>,
	"arm-scmi@vger.kernel.org" <arm-scmi@vger.kernel.org>,
	Dhruva Gole <d-gole@ti.com>, Sebin Francis <sebin.francis@ti.com>
Subject: Re: mixing SCMI and PSCI power domain hierarchy
Date: Wed, 16 Apr 2025 12:28:51 -0700	[thread overview]
Message-ID: <7h1ptrdhxo.fsf@baylibre.com> (raw)
In-Reply-To: <7h7c3kcbg2.fsf@baylibre.com>

Kevin Hilman <khilman@baylibre.com> writes:

> Ulf Hansson <ulf.hansson@linaro.org> writes:

[...]

>>
>> Attaching a single PM domain is managed by bus-level code via
>> dev_pm_domain_attach().
>>
>> If a device has multiple PM domains, the driver/bus for it needs to
>> manage that specifically, typically by using
>> dev_pm_domain_attach_list().
>
> OK, I see.  But in my quick experiment with this, I found that adding a
> 2nd domain to a devices power-domain property, it disappeared from the
> first domain also.  Any idea what is happening there?

Just to answer my own question, I found that genpd_dev_pm_attach() has
an explicit check for the number of PM domains a device has[1].  If it
is not 1, the device is not attached to any domain.

I had assumed that maybe the first domain would get attached and the
others are ignored, but in fact all the domains are ignored.

Kevin

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/pmdomain/core.c?h=v6.14#n3035

  reply	other threads:[~2025-04-16 19:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-07 23:31 mixing SCMI and PSCI power domain hierarchy Kevin Hilman
2025-04-10 13:07 ` Ulf Hansson
2025-04-16  0:22   ` Kevin Hilman
2025-04-16 13:22     ` Ulf Hansson
2025-04-16 16:34       ` Kevin Hilman
2025-04-16 19:28         ` Kevin Hilman [this message]
2025-04-25 10:48           ` Ulf Hansson
2025-04-16 23:57       ` Kevin Hilman
2025-04-25 11:08         ` Ulf Hansson
2025-04-25 14:39           ` Kevin Hilman
2025-05-16 12:36             ` Ulf Hansson
2025-05-28 20:09               ` Kevin Hilman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7h1ptrdhxo.fsf@baylibre.com \
    --to=khilman@baylibre.com \
    --cc=arm-scmi@vger.kernel.org \
    --cc=cristian.marussi@arm.com \
    --cc=d-gole@ti.com \
    --cc=sebin.francis@ti.com \
    --cc=souvik.chakravarty@arm.com \
    --cc=sudeep.holla@arm.com \
    --cc=ulf.hansson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.