From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: georgi.djakov@linaro.org
Cc: Sibi Sankar <sibis@codeaurora.org>,
robh+dt@kernel.org, evgreen@chromium.org, agross@kernel.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-msm@vger.kernel.org, mark.rutland@arm.com,
daidavid1@codeaurora.org, saravanak@google.com, mka@chromium.org,
linux-pm@vger.kernel.org
Subject: Re: [PATCH v3 0/6] Split SDM845 interconnect nodes and consolidate RPMh support
Date: Mon, 24 Feb 2020 21:16:45 -0800 [thread overview]
Message-ID: <20200225051645.GX3948@builder> (raw)
In-Reply-To: <20200209183411.17195-1-sibis@codeaurora.org>
On Sun 09 Feb 10:34 PST 2020, Sibi Sankar wrote:
Hi Georgi,
Will you pick up the bindings and driver patches for v5.7? So that I can
apply the dts patches.
Regards,
Bjorn
> While there are no current consumers of the SDM845 interconnect device in
> devicetree, take this opportunity to redefine the interconnect device nodes
> as the previous definitions of using a single child node under the apps_rsc
> device did not accurately capture the description of the hardware.
> The Network-On-Chip (NoC) interconnect devices should be represented in a
> manner akin to QCS404 platforms[1] where there is a separation of NoC devices
> and its RPM/RPMh counterparts.
>
> The bcm-voter devices are representing the RPMh devices that the interconnect
> providers need to communicate with and there can be more than one instance of
> the Bus Clock Manager (BCM) which can live under different instances of Resource
> State Coordinators (RSC). There are display use cases where consumers may need
> to target a different bcm-voter (Some display specific RSC) than the default,
> and there needs to be a way to represent this connection in devicetree.
>
> This patches series extends the discussions[2][3] involving the SDM845
> interconnect bindings by adding accompanying driver implementations
> using the split NoC devices. Some of the code used to support the SDM845
> provider driver are refactored into common modules that can used by other
> RPMh based interconnect providers such as SC7180[4]. This patch series also
> updates existing sdm845 binding documentation to DT schema format using
> json-schema.
>
> v3:
> - Picked up Robs R-b for patch 1
> - Fixup qcom,bcm-voter.yaml. comments (Rob)
> - Use qcom,bcm-voter instead of SoC specific compatible for
> SDM845 and SC7180 (Odelu/Sibi)
> - Fixup bindings check failures for qcom,sdm845.yaml
> - Fixup the misc bugs. comments (Evan/Sibi)
> - Fixup reg size for aggre1/2_noc
>
> v2:
> - Reorganized dt-binding patches
> - Fixed a bug that adds duplicate BCM node to voter (Georgi)
> - Addressed misc. comments (Georgi)
>
> v1: https://lkml.org/lkml/2019/12/16/15
>
> [1]: https://lkml.org/lkml/2019/6/13/143
> [2]: https://lkml.org/lkml/2019/7/19/1063
> [3]: https://lkml.org/lkml/2019/10/16/1793
> [4]: https://lkml.org/lkml/2019/11/26/389
>
> David Dai (6):
> dt-bindings: interconnect: Convert qcom,sdm845 to DT schema
> dt-bindings: interconnect: Add YAML schemas for QCOM bcm-voter
> dt-bindings: interconnect: Update Qualcomm SDM845 DT bindings
> interconnect: qcom: Consolidate interconnect RPMh support
> interconnect: qcom: sdm845: Split qnodes into their respective NoCs
> arm64: dts: sdm845: Redefine interconnect provider DT nodes
>
> .../bindings/interconnect/qcom,bcm-voter.yaml | 45 +
> .../bindings/interconnect/qcom,sdm845.txt | 24 -
> .../bindings/interconnect/qcom,sdm845.yaml | 74 +
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 65 +-
> drivers/interconnect/qcom/Kconfig | 13 +-
> drivers/interconnect/qcom/Makefile | 4 +
> drivers/interconnect/qcom/bcm-voter.c | 366 +++++
> drivers/interconnect/qcom/bcm-voter.h | 27 +
> drivers/interconnect/qcom/icc-rpmh.c | 147 ++
> drivers/interconnect/qcom/icc-rpmh.h | 149 +++
> drivers/interconnect/qcom/sdm845.c | 1185 +++++++----------
> .../dt-bindings/interconnect/qcom,sdm845.h | 263 ++--
> 12 files changed, 1516 insertions(+), 846 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/interconnect/qcom,bcm-voter.yaml
> delete mode 100644 Documentation/devicetree/bindings/interconnect/qcom,sdm845.txt
> create mode 100644 Documentation/devicetree/bindings/interconnect/qcom,sdm845.yaml
> create mode 100644 drivers/interconnect/qcom/bcm-voter.c
> create mode 100644 drivers/interconnect/qcom/bcm-voter.h
> create mode 100644 drivers/interconnect/qcom/icc-rpmh.c
> create mode 100644 drivers/interconnect/qcom/icc-rpmh.h
>
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
next prev parent reply other threads:[~2020-02-25 5:16 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-09 18:34 [PATCH v3 0/6] Split SDM845 interconnect nodes and consolidate RPMh support Sibi Sankar
2020-02-09 18:34 ` [PATCH v3 1/6] dt-bindings: interconnect: Convert qcom,sdm845 to DT schema Sibi Sankar
2020-02-09 18:34 ` [PATCH v3 2/6] dt-bindings: interconnect: Add YAML schemas for QCOM bcm-voter Sibi Sankar
2020-02-18 21:31 ` Rob Herring
2020-02-09 18:34 ` [PATCH v3 3/6] dt-bindings: interconnect: Update Qualcomm SDM845 DT bindings Sibi Sankar
2020-02-18 21:31 ` Rob Herring
2020-02-09 18:34 ` [PATCH v3 4/6] interconnect: qcom: Consolidate interconnect RPMh support Sibi Sankar
2020-02-27 13:44 ` Georgi Djakov
2020-02-27 21:52 ` Evan Green
2020-02-09 18:34 ` [PATCH v3 5/6] interconnect: qcom: sdm845: Split qnodes into their respective NoCs Sibi Sankar
2020-02-27 21:52 ` Evan Green
2020-02-09 18:34 ` [PATCH v3 6/6] arm64: dts: sdm845: Redefine interconnect provider DT nodes Sibi Sankar
2020-02-27 13:45 ` Georgi Djakov
2020-02-27 21:52 ` Evan Green
2020-02-25 5:16 ` Bjorn Andersson [this message]
2020-03-04 16:23 ` [PATCH v3 0/6] Split SDM845 interconnect nodes and consolidate RPMh support Georgi Djakov
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=20200225051645.GX3948@builder \
--to=bjorn.andersson@linaro.org \
--cc=agross@kernel.org \
--cc=daidavid1@codeaurora.org \
--cc=devicetree@vger.kernel.org \
--cc=evgreen@chromium.org \
--cc=georgi.djakov@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mka@chromium.org \
--cc=robh+dt@kernel.org \
--cc=saravanak@google.com \
--cc=sibis@codeaurora.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).