From mboxrd@z Thu Jan 1 00:00:00 1970 From: Georgi Djakov Subject: [PATCH v4 4/7] interconnect: qcom: Add RPM communication Date: Wed, 6 Jun 2018 18:00:58 +0300 Message-ID: <43302039-a056-b73d-50a9-d54f7dc90219@linaro.org> References: <20180309210958.16672-1-georgi.djakov@linaro.org> <20180309210958.16672-5-georgi.djakov@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Evan Green Cc: linux-pm@vger.kernel.org, gregkh@linuxfoundation.org, rjw@rjwysocki.net, robh+dt@kernel.org, Michael Turquette , khilman@baylibre.com, vincent.guittot@linaro.org, skannan@codeaurora.org, Bjorn Andersson , amit.kucheria@linaro.org, seansw@qti.qualcomm.com, davidai@quicinc.com, mark.rutland@arm.com, lorenzo.pieralisi@arm.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org Hi Evan, On 12.05.18 г. 0:30, Evan Green wrote: > On Fri, Mar 9, 2018 at 1:11 PM Georgi Djakov > wrote: > >> On some Qualcomm SoCs, there is a remote processor, which controls some of >> the Network-On-Chip interconnect resources. Other CPUs express their needs >> by communicating with this processor. Add a driver to handle comminication >> with this remote processor. > >> Signed-off-by: Georgi Djakov >> --- >> .../devicetree/bindings/interconnect/qcom-smd.txt | 31 ++++++++ >> drivers/interconnect/qcom/Makefile | 1 + >> drivers/interconnect/qcom/smd-rpm.c | 90 > ++++++++++++++++++++++ >> drivers/interconnect/qcom/smd-rpm.h | 15 ++++ >> 4 files changed, 137 insertions(+) >> create mode 100644 > Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> create mode 100644 drivers/interconnect/qcom/Makefile >> create mode 100644 drivers/interconnect/qcom/smd-rpm.c >> create mode 100644 drivers/interconnect/qcom/smd-rpm.h > >> diff --git a/Documentation/devicetree/bindings/interconnect/qcom-smd.txt > b/Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> new file mode 100644 >> index 000000000000..14e83ed7019b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> @@ -0,0 +1,31 @@ >> +Qualcomm SMD-RPM interconnect driver binding >> +------------------------------------------------ >> +The RPM is a dedicated hardware engine for managing the shared >> +SoC resources in order to keep the lowest power profile. It >> +communicates with other hardware subsystems via shared memory >> +and accepts requests for various resources. > > You never say what RPM or SMD stands for. RPM is Resource Power Manager, > right? But I'm not in the know about SMD. Can you define these somewhere? > Sure, it's the Shared Memory Driver back-end used for communicating with RPM. Now added this explanation into the docs. Thanks, Georgi From mboxrd@z Thu Jan 1 00:00:00 1970 From: georgi.djakov@linaro.org (Georgi Djakov) Date: Wed, 6 Jun 2018 18:00:58 +0300 Subject: [PATCH v4 4/7] interconnect: qcom: Add RPM communication In-Reply-To: References: <20180309210958.16672-1-georgi.djakov@linaro.org> <20180309210958.16672-5-georgi.djakov@linaro.org> Message-ID: <43302039-a056-b73d-50a9-d54f7dc90219@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Evan, On 12.05.18 ?. 0:30, Evan Green wrote: > On Fri, Mar 9, 2018 at 1:11 PM Georgi Djakov > wrote: > >> On some Qualcomm SoCs, there is a remote processor, which controls some of >> the Network-On-Chip interconnect resources. Other CPUs express their needs >> by communicating with this processor. Add a driver to handle comminication >> with this remote processor. > >> Signed-off-by: Georgi Djakov >> --- >> .../devicetree/bindings/interconnect/qcom-smd.txt | 31 ++++++++ >> drivers/interconnect/qcom/Makefile | 1 + >> drivers/interconnect/qcom/smd-rpm.c | 90 > ++++++++++++++++++++++ >> drivers/interconnect/qcom/smd-rpm.h | 15 ++++ >> 4 files changed, 137 insertions(+) >> create mode 100644 > Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> create mode 100644 drivers/interconnect/qcom/Makefile >> create mode 100644 drivers/interconnect/qcom/smd-rpm.c >> create mode 100644 drivers/interconnect/qcom/smd-rpm.h > >> diff --git a/Documentation/devicetree/bindings/interconnect/qcom-smd.txt > b/Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> new file mode 100644 >> index 000000000000..14e83ed7019b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/interconnect/qcom-smd.txt >> @@ -0,0 +1,31 @@ >> +Qualcomm SMD-RPM interconnect driver binding >> +------------------------------------------------ >> +The RPM is a dedicated hardware engine for managing the shared >> +SoC resources in order to keep the lowest power profile. It >> +communicates with other hardware subsystems via shared memory >> +and accepts requests for various resources. > > You never say what RPM or SMD stands for. RPM is Resource Power Manager, > right? But I'm not in the know about SMD. Can you define these somewhere? > Sure, it's the Shared Memory Driver back-end used for communicating with RPM. Now added this explanation into the docs. Thanks, Georgi