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 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.