* [PATCH v2] PCI: starfive: Enable PCIe controller's runtime PM before probing host bridge
@ 2024-10-11 23:55 Mayank Rana
2024-10-12 3:50 ` Manivannan Sadhasivam
0 siblings, 1 reply; 3+ messages in thread
From: Mayank Rana @ 2024-10-11 23:55 UTC (permalink / raw)
To: kevin.xie, lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas
Cc: linux-pci, linux-kernel, quic_krichai, Mayank Rana, Marek Szyprowski
PCIe controller device (i.e. PCIe starfive device) is parent to PCIe host
bridge device. To enable runtime PM of PCIe host bridge device (child
device), it is must to enable parent device's runtime PM to avoid seeing
WARN_ON as "Enabling runtime PM for inactive device with active children".
Fix this issue by enabling starfive pcie controller device's runtime PM
before calling into pci_host_probe() through plda_pcie_host_init().
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Mayank Rana <quic_mrana@quicinc.com>
---
v1->v2: Updated commit description based on Bjorn's feedback
Link to v1: https://patchwork.kernel.org/project/linux-pci/patch/20241010202950.3263899-1-quic_mrana@quicinc.com/
drivers/pci/controller/plda/pcie-starfive.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/controller/plda/pcie-starfive.c b/drivers/pci/controller/plda/pcie-starfive.c
index 0567ec373a3e..e73c1b7bc8ef 100644
--- a/drivers/pci/controller/plda/pcie-starfive.c
+++ b/drivers/pci/controller/plda/pcie-starfive.c
@@ -404,6 +404,9 @@ static int starfive_pcie_probe(struct platform_device *pdev)
if (ret)
return ret;
+ pm_runtime_enable(&pdev->dev);
+ pm_runtime_get_sync(&pdev->dev);
+
plda->host_ops = &sf_host_ops;
plda->num_events = PLDA_MAX_EVENT_NUM;
/* mask doorbell event */
@@ -413,11 +416,12 @@ static int starfive_pcie_probe(struct platform_device *pdev)
plda->events_bitmap <<= PLDA_NUM_DMA_EVENTS;
ret = plda_pcie_host_init(&pcie->plda, &starfive_pcie_ops,
&stf_pcie_event);
- if (ret)
+ if (ret) {
+ pm_runtime_put_sync(&pdev->dev);
+ pm_runtime_disable(&pdev->dev);
return ret;
+ }
- pm_runtime_enable(&pdev->dev);
- pm_runtime_get_sync(&pdev->dev);
platform_set_drvdata(pdev, pcie);
return 0;
--
2.25.1
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] PCI: starfive: Enable PCIe controller's runtime PM before probing host bridge
2024-10-11 23:55 [PATCH v2] PCI: starfive: Enable PCIe controller's runtime PM before probing host bridge Mayank Rana
@ 2024-10-12 3:50 ` Manivannan Sadhasivam
2024-10-14 16:18 ` Mayank Rana
0 siblings, 1 reply; 3+ messages in thread
From: Manivannan Sadhasivam @ 2024-10-12 3:50 UTC (permalink / raw)
To: Mayank Rana
Cc: kevin.xie, lpieralisi, kw, robh, bhelgaas, linux-pci,
linux-kernel, quic_krichai, Marek Szyprowski
On Fri, Oct 11, 2024 at 04:55:30PM -0700, Mayank Rana wrote:
> PCIe controller device (i.e. PCIe starfive device) is parent to PCIe host
> bridge device. To enable runtime PM of PCIe host bridge device (child
> device), it is must to enable parent device's runtime PM to avoid seeing
> WARN_ON as "Enabling runtime PM for inactive device with active children".
"to avoid seeing the below warning from PM core:
pcie-starfive 940000000.pcie: Enabling runtime PM for inactive device
with active children"
> Fix this issue by enabling starfive pcie controller device's runtime PM
> before calling into pci_host_probe() through plda_pcie_host_init().
"before calling pci_host_probe() in plda_pcie_host_init()"
>
> Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
> Signed-off-by: Mayank Rana <quic_mrana@quicinc.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
- Mani
> ---
> v1->v2: Updated commit description based on Bjorn's feedback
> Link to v1: https://patchwork.kernel.org/project/linux-pci/patch/20241010202950.3263899-1-quic_mrana@quicinc.com/
>
> drivers/pci/controller/plda/pcie-starfive.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/controller/plda/pcie-starfive.c b/drivers/pci/controller/plda/pcie-starfive.c
> index 0567ec373a3e..e73c1b7bc8ef 100644
> --- a/drivers/pci/controller/plda/pcie-starfive.c
> +++ b/drivers/pci/controller/plda/pcie-starfive.c
> @@ -404,6 +404,9 @@ static int starfive_pcie_probe(struct platform_device *pdev)
> if (ret)
> return ret;
>
> + pm_runtime_enable(&pdev->dev);
> + pm_runtime_get_sync(&pdev->dev);
> +
> plda->host_ops = &sf_host_ops;
> plda->num_events = PLDA_MAX_EVENT_NUM;
> /* mask doorbell event */
> @@ -413,11 +416,12 @@ static int starfive_pcie_probe(struct platform_device *pdev)
> plda->events_bitmap <<= PLDA_NUM_DMA_EVENTS;
> ret = plda_pcie_host_init(&pcie->plda, &starfive_pcie_ops,
> &stf_pcie_event);
> - if (ret)
> + if (ret) {
> + pm_runtime_put_sync(&pdev->dev);
> + pm_runtime_disable(&pdev->dev);
> return ret;
> + }
>
> - pm_runtime_enable(&pdev->dev);
> - pm_runtime_get_sync(&pdev->dev);
> platform_set_drvdata(pdev, pcie);
>
> return 0;
> --
> 2.25.1
>
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] PCI: starfive: Enable PCIe controller's runtime PM before probing host bridge
2024-10-12 3:50 ` Manivannan Sadhasivam
@ 2024-10-14 16:18 ` Mayank Rana
0 siblings, 0 replies; 3+ messages in thread
From: Mayank Rana @ 2024-10-14 16:18 UTC (permalink / raw)
To: Manivannan Sadhasivam
Cc: kevin.xie, lpieralisi, kw, robh, bhelgaas, linux-pci,
linux-kernel, quic_krichai, Marek Szyprowski
On 10/11/2024 8:50 PM, Manivannan Sadhasivam wrote:
> On Fri, Oct 11, 2024 at 04:55:30PM -0700, Mayank Rana wrote:
>> PCIe controller device (i.e. PCIe starfive device) is parent to PCIe host
>> bridge device. To enable runtime PM of PCIe host bridge device (child
>> device), it is must to enable parent device's runtime PM to avoid seeing
>> WARN_ON as "Enabling runtime PM for inactive device with active children".
>
> "to avoid seeing the below warning from PM core:
> pcie-starfive 940000000.pcie: Enabling runtime PM for inactive device
> with active children"
ACK
>> Fix this issue by enabling starfive pcie controller device's runtime PM
>> before calling into pci_host_probe() through plda_pcie_host_init().
>
> "before calling pci_host_probe() in plda_pcie_host_init()"
ACK
>>
>> Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
>> Signed-off-by: Mayank Rana <quic_mrana@quicinc.com>
>
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>
> - Mani
>
>> ---
>> v1->v2: Updated commit description based on Bjorn's feedback
>> Link to v1: https://patchwork.kernel.org/project/linux-pci/patch/20241010202950.3263899-1-quic_mrana@quicinc.com/
>>
>> drivers/pci/controller/plda/pcie-starfive.c | 10 +++++++---
>> 1 file changed, 7 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/pci/controller/plda/pcie-starfive.c b/drivers/pci/controller/plda/pcie-starfive.c
>> index 0567ec373a3e..e73c1b7bc8ef 100644
>> --- a/drivers/pci/controller/plda/pcie-starfive.c
>> +++ b/drivers/pci/controller/plda/pcie-starfive.c
>> @@ -404,6 +404,9 @@ static int starfive_pcie_probe(struct platform_device *pdev)
>> if (ret)
>> return ret;
>>
>> + pm_runtime_enable(&pdev->dev);
>> + pm_runtime_get_sync(&pdev->dev);
>> +
>> plda->host_ops = &sf_host_ops;
>> plda->num_events = PLDA_MAX_EVENT_NUM;
>> /* mask doorbell event */
>> @@ -413,11 +416,12 @@ static int starfive_pcie_probe(struct platform_device *pdev)
>> plda->events_bitmap <<= PLDA_NUM_DMA_EVENTS;
>> ret = plda_pcie_host_init(&pcie->plda, &starfive_pcie_ops,
>> &stf_pcie_event);
>> - if (ret)
>> + if (ret) {
>> + pm_runtime_put_sync(&pdev->dev);
>> + pm_runtime_disable(&pdev->dev);
>> return ret;
>> + }
>>
>> - pm_runtime_enable(&pdev->dev);
>> - pm_runtime_get_sync(&pdev->dev);
>> platform_set_drvdata(pdev, pcie);
>>
>> return 0;
>> --
>> 2.25.1
>>
>
Regards,
Mayank
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-10-14 16:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-10-11 23:55 [PATCH v2] PCI: starfive: Enable PCIe controller's runtime PM before probing host bridge Mayank Rana
2024-10-12 3:50 ` Manivannan Sadhasivam
2024-10-14 16:18 ` Mayank Rana
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®