From: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
To: "Lorenzo Pieralisi" <lpieralisi@kernel.org>,
"Krzysztof Wilczyński" <kw@linux.com>,
"Manivannan Sadhasivam" <manivannan.sadhasivam@linaro.org>,
"Rob Herring" <robh@kernel.org>,
"Bjorn Helgaas" <bhelgaas@google.com>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konradybcio@kernel.org>,
cros-qcom-dts-watchers@chromium.org
Cc: linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
quic_vbadigan@quicinc.com, quic_mrana@quicinc.com,
Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
Subject: [PATCH v2 0/3] PCI: qcom: Move PERST# GPIO & phy retrieval from controller to PCIe bridge node
Date: Mon, 14 Apr 2025 11:09:11 +0530 [thread overview]
Message-ID: <20250414-perst-v2-0-89247746d755@oss.qualcomm.com> (raw)
There are many places we agreed to move the wake and perst gpio's
and phy etc to the pcie root port node instead of bridge node[1].
So move the phy, phy-names, wake-gpio's in the root port.
There is already reset-gpio defined for PERST# in pci-bus-common.yaml,
start using that property instead of perst-gpio.
For backward compatibility, not removing any existing properties in the
bridge node.
There are some other properties like num-lanes, max-link-speed which
needs to be moved to the root port nodes, but in this series we are
excluding them for now as this requires more changes in dwc layer and
can complicate the things.
The main intention of this series is to move wake# to the root port node.
After this series we wil come up with a patch which regiters for wake IRQ
from the pcieport driver. The wake IRQ is needed for the endpoint to wakeup
the host from D3cold.
[1] https://lore.kernel.org/linux-pci/20241211192014.GA3302752@bhelgaas/
Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
---
Changes in v2:
- Remove phy-names property and change the driver, dtsi accordingly (Rob)
- Link to v1: https://lore.kernel.org/r/20250322-perst-v1-0-e5e4da74a204@oss.qualcomm.com
---
Krishna Chaitanya Chundru (3):
dt-bindings: PCI: qcom: Move phy, wake & reset gpio's to root port
arm64: qcom: sc7280: Move phy, perst to root port node
PCI: qcom: Add support for multi-root port
.../devicetree/bindings/pci/qcom,pcie-common.yaml | 18 +++
.../devicetree/bindings/pci/qcom,pcie-sc7280.yaml | 17 ++-
arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 5 +-
arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 5 +-
arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 5 +-
arch/arm64/boot/dts/qcom/sc7280.dtsi | 6 +-
drivers/pci/controller/dwc/pcie-qcom.c | 149 +++++++++++++++++----
7 files changed, 168 insertions(+), 37 deletions(-)
---
base-commit: 8ffd015db85fea3e15a77027fda6c02ced4d2444
change-id: 20250101-perst-cb885b5a6129
Best regards,
--
Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
next reply other threads:[~2025-04-14 5:39 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-14 5:39 Krishna Chaitanya Chundru [this message]
2025-04-14 5:39 ` [PATCH v2 1/3] dt-bindings: PCI: qcom: Move phy, wake & reset gpio's to root port Krishna Chaitanya Chundru
2025-04-14 8:04 ` Dmitry Baryshkov
2025-04-14 12:50 ` Caleb Connolly
2025-04-15 7:41 ` Manivannan Sadhasivam
2025-04-15 22:00 ` Rob Herring
2025-04-14 5:39 ` [PATCH v2 2/3] arm64: qcom: sc7280: Move phy, perst to root port node Krishna Chaitanya Chundru
2025-04-14 8:06 ` Dmitry Baryshkov
2025-04-14 12:49 ` Caleb Connolly
2025-04-15 7:46 ` Manivannan Sadhasivam
2025-04-15 7:59 ` Dmitry Baryshkov
2025-04-15 9:15 ` Caleb Connolly
2025-04-14 5:39 ` [PATCH v2 3/3] PCI: qcom: Add support for multi-root port Krishna Chaitanya Chundru
2025-04-15 7:59 ` Manivannan Sadhasivam
2025-04-14 8:03 ` [PATCH v2 0/3] PCI: qcom: Move PERST# GPIO & phy retrieval from controller to PCIe bridge node Dmitry Baryshkov
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=20250414-perst-v2-0-89247746d755@oss.qualcomm.com \
--to=krishna.chundru@oss.qualcomm.com \
--cc=andersson@kernel.org \
--cc=bhelgaas@google.com \
--cc=conor+dt@kernel.org \
--cc=cros-qcom-dts-watchers@chromium.org \
--cc=devicetree@vger.kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=kw@linux.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=quic_mrana@quicinc.com \
--cc=quic_vbadigan@quicinc.com \
--cc=robh@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