From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Prasad Malisetty <pmaliset@codeaurora.org>
Cc: agross@kernel.org, bhelgaas@google.com, robh+dt@kernel.org,
swboyd@chromium.org, lorenzo.pieralisi@arm.com,
svarbanov@mm-sol.com, devicetree@vger.kernel.org,
linux-arm-msm@vger.kernel.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org, mgautam@codeaurora.org,
dianders@chromium.org, mka@chromium.org, sanm@codeaurora.org
Subject: Re: [PATCH v2 4/4] dt-bindings: pci: qcom: Document PCIe bindings for SC720
Date: Sat, 5 Jun 2021 21:19:07 -0500 [thread overview]
Message-ID: <YLwwm8UxDTSgBjy4@builder.lan> (raw)
In-Reply-To: <1622904059-21244-5-git-send-email-pmaliset@codeaurora.org>
On Sat 05 Jun 09:40 CDT 2021, Prasad Malisetty wrote:
> Document the PCIe DT bindings for SC7280 SoC.The PCIe IP is similar
> to the one used on SM8250. Add the compatible for SC7280.
>
> Signed-off-by: Prasad Malisetty <pmaliset@codeaurora.org>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
> Documentation/devicetree/bindings/pci/qcom,pcie.txt | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.txt b/Documentation/devicetree/bindings/pci/qcom,pcie.txt
> index 25f4def..9c0908f 100644
> --- a/Documentation/devicetree/bindings/pci/qcom,pcie.txt
> +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.txt
> @@ -12,6 +12,7 @@
> - "qcom,pcie-ipq4019" for ipq4019
> - "qcom,pcie-ipq8074" for ipq8074
> - "qcom,pcie-qcs404" for qcs404
> + - "qcom,pcie-sc7280" for sc7280
> - "qcom,pcie-sdm845" for sdm845
> - "qcom,pcie-sm8250" for sm8250
> - "qcom,pcie-ipq6018" for ipq6018
> @@ -144,6 +145,22 @@
> - "slave_bus" AXI Slave clock
>
> - clock-names:
> + Usage: required for sc7280
> + Value type: <stringlist>
> + Definition: Should contain the following entries
> + - "aux" Auxiliary clock
> + - "cfg" Configuration clock
> + - "bus_master" Master AXI clock
> + - "bus_slave" Slave AXI clock
> + - "slave_q2a" Slave Q2A clock
> + - "tbu" PCIe TBU clock
> + - "ddrss_sf_tbu" PCIe SF TBU clock
> + - "pipe" PIPE clock
> + - "pipe_src" PIPE MUX
If you describe it as pipe _mux_, then name it pipe_mux.
> + - "pipe_ext" PIPE output clock
Better to name this in a way that indicates that it is the pipe clock
coming out of the phy, perhaps "phy_pipe".
But that said, as Stephen points out ensuring that the pipe_clk_src's
parent is the phy's clock can be done in DT directly.
Regards,
Bjorn
> + - "ref" REFERENCE clock
> +
> +- clock-names:
> Usage: required for sdm845
> Value type: <stringlist>
> Definition: Should contain the following entries
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
prev parent reply other threads:[~2021-06-06 2:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-05 14:40 [PATCH v2 0/4] Add DT bindings and DT nodes for PCIe and PHY in SC7280 Prasad Malisetty
2021-06-05 14:40 ` [PATCH v2 1/4] arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board Prasad Malisetty
2021-06-06 2:20 ` Bjorn Andersson
2021-06-05 14:40 ` [PATCH v2 2/4] arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes Prasad Malisetty
2021-06-06 2:26 ` Bjorn Andersson
2021-06-18 5:52 ` Prasad Malisetty
2021-06-05 14:40 ` [PATCH v2 3/4] PCIe: qcom: Add support to control pipe clk mux Prasad Malisetty
2021-06-05 21:26 ` Stephen Boyd
2021-06-18 13:00 ` Prasad Malisetty
2021-06-06 2:15 ` Bjorn Andersson
2021-06-22 12:54 ` Prasad Malisetty
2021-06-05 14:40 ` [PATCH v2 4/4] dt-bindings: pci: qcom: Document PCIe bindings for SC720 Prasad Malisetty
2021-06-06 2:19 ` Bjorn Andersson [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=YLwwm8UxDTSgBjy4@builder.lan \
--to=bjorn.andersson@linaro.org \
--cc=agross@kernel.org \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=mgautam@codeaurora.org \
--cc=mka@chromium.org \
--cc=pmaliset@codeaurora.org \
--cc=robh+dt@kernel.org \
--cc=sanm@codeaurora.org \
--cc=svarbanov@mm-sol.com \
--cc=swboyd@chromium.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;
as well as URLs for NNTP newsgroup(s).