public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
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

-- 
மணிவண்ணன் சதாசிவம்

      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