From: Georgi Djakov <georgi.djakov@linaro.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: linux-pm@vger.kernel.org, gregkh@linuxfoundation.org,
rjw@rjwysocki.net, robh+dt@kernel.org, mturquette@baylibre.com,
khilman@baylibre.com, vincent.guittot@linaro.org,
skannan@codeaurora.org, 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
Subject: Re: [PATCH v4 2/7] dt-bindings: Introduce interconnect provider bindings
Date: Mon, 19 Mar 2018 17:34:24 +0800 [thread overview]
Message-ID: <45cba7ca-3a83-0ef8-eff7-a8b33bce9185@linaro.org> (raw)
In-Reply-To: <20180318225029.GQ5626@tuxbook-pro>
Hi Bjorn,
On 03/19/2018 06:50 AM, Bjorn Andersson wrote:
> On Fri 09 Mar 13:09 PST 2018, Georgi Djakov wrote:
>> diff --git a/Documentation/devicetree/bindings/interconnect/interconnect.txt b/Documentation/devicetree/bindings/interconnect/interconnect.txt
> [..]
>> +Required properties:
>> +- compatible : contains the interconnect provider vendor specific compatible
>> + string
>> +- reg : register space of the interconnect controller hardware
>
> These properties doesn't relate to the interconnect provider, so there's
> no need to describe them here.
Agree.
>
>> +
>> +Examples:
>> +
>> + snoc: snoc@580000 {
>
> These nodes should be described in a qcom/msm8916 binding document.
Ok, will move this and the above to the platform specific binding
documentation.
>
>> + compatible = "qcom,msm8916-snoc";
>> + reg = <0x580000 0x14000>;
>> + clock-names = "bus_clk", "bus_a_clk";
>> + clocks = <&rpmcc RPM_SMD_SNOC_CLK>, <&rpmcc RPM_SMD_SNOC_A_CLK>;
>> + status = "okay";
>> + };
Thanks for the comments!
BR,
Georgi
next prev parent reply other threads:[~2018-03-19 9:34 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-09 21:09 [PATCH v4 0/7] Introduce on-chip interconnect API Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 1/7] interconnect: Add generic " Georgi Djakov
2018-04-06 17:38 ` Matthias Kaehlcke
2018-04-12 13:06 ` Georgi Djakov
2018-05-11 21:30 ` Evan Green
2018-06-06 14:59 ` Georgi Djakov
2018-06-06 18:09 ` Georgi Djakov
2018-06-07 1:06 ` Evan Green
2018-05-25 8:26 ` Amit Kucheria
2018-06-06 15:08 ` Georgi Djakov
2018-06-08 15:57 ` Alexandre Bailon
2018-06-09 19:15 ` Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 2/7] dt-bindings: Introduce interconnect provider bindings Georgi Djakov
2018-03-18 22:50 ` Bjorn Andersson
2018-03-19 9:34 ` Georgi Djakov [this message]
2018-04-12 13:15 ` Neil Armstrong
2018-06-06 15:23 ` Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 3/7] interconnect: Add debugfs support Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 4/7] interconnect: qcom: Add RPM communication Georgi Djakov
2018-05-11 21:30 ` Evan Green
2018-06-06 15:00 ` Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 5/7] interconnect: qcom: Add msm8916 interconnect provider driver Georgi Djakov
2018-04-05 22:58 ` Matthias Kaehlcke
2018-04-12 13:09 ` Georgi Djakov
2018-05-11 21:29 ` Evan Green
2018-06-06 15:03 ` Georgi Djakov
2018-05-25 8:27 ` Amit Kucheria
2018-06-06 15:14 ` Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 6/7] dt-bindings: Introduce interconnect consumers bindings Georgi Djakov
2018-03-18 22:49 ` Bjorn Andersson
2018-03-19 9:41 ` Georgi Djakov
2018-03-09 21:09 ` [PATCH v4 7/7] interconnect: Allow endpoints translation via DT Georgi Djakov
2018-05-11 21:29 ` Evan Green
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=45cba7ca-3a83-0ef8-eff7-a8b33bce9185@linaro.org \
--to=georgi.djakov@linaro.org \
--cc=amit.kucheria@linaro.org \
--cc=bjorn.andersson@linaro.org \
--cc=davidai@quicinc.com \
--cc=gregkh@linuxfoundation.org \
--cc=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=mturquette@baylibre.com \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=seansw@qti.qualcomm.com \
--cc=skannan@codeaurora.org \
--cc=vincent.guittot@linaro.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).