From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: rafael@kernel.org
Cc: linux-pm@vger.kernel.org,
"linux-pci@vger.kernel.org Bjorn Helgaas" <bhelgaas@google.com>,
Krishna Chaitanya Chundru <quic_krichai@quicinc.com>
Subject: Re: Behavior of the PM framework w.r.t the parent-child relationship
Date: Tue, 21 May 2024 21:10:36 +0530 [thread overview]
Message-ID: <20240521154036.GA59482@thinkpad> (raw)
In-Reply-To: <c0c9fd70-69d4-a7dc-8c4a-773dc742f6c8@quicinc.com>
+ linux-pci
On Mon, Apr 22, 2024 at 03:30:18PM +0530, Krishna Chaitanya Chundru wrote:
> Hi Rafael,
>
> We are planning to add runtime PM support to the PCIe controller driver
> and facing some issues with runtime PM. These are the details.
>
> PCIe controller is a parent of the PCIe bridge dev node and PCIe
> endpoint driver as shown below
>
> PCIe controller(Top level parent & parent of host bridge)
> |
> v
> PCIe Host bridge(Parent of PCI-PCI bridge driver)
> |
> v
> PCI-PCI bridge(Parent of endpoint driver)
> |
> v
> PCIe endpoint driver
>
> There is no driver associated with the PCIe Host bridge device node.
> And the runtime pm for the bridge node is not enabled explicitly.
>
> we are seeing PCIe controller driver is getting runtime suspended even
> when the PCI-PCI bridge and PCIe endpoint driver are active.
> Our expectation is controller driver runtime suspend will be called only
> when the child drivers went to the runtime suspend as the controller
> is parent device.
>
> For the things to work properly we are calling pm_runtime_set_active() &
> pm_runtime_enable() in [1]
>
> As bridge device node doesn't have driver associated and it is kind of
> integrated with the controller, So we are expecting when the controller
> driver enables its runtime pm it should enable the bridge runtime pm
> also and vice versa.
>
> Can you please check this behavior and see if it is something which
> needs to be fixed at the PM framework.
>
Hi Rafael,
Gentle ping on this thread!
- Mani
--
மணிவண்ணன் சதாசிவம்
prev parent reply other threads:[~2024-05-21 15:40 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-22 10:00 Behavior of the PM framework w.r.t the parent-child relationship Krishna Chaitanya Chundru
2024-05-21 15:40 ` Manivannan Sadhasivam [this message]
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=20240521154036.GA59482@thinkpad \
--to=manivannan.sadhasivam@linaro.org \
--cc=bhelgaas@google.com \
--cc=linux-pm@vger.kernel.org \
--cc=quic_krichai@quicinc.com \
--cc=rafael@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox