From: Govind Singh <govinds@codeaurora.org> To: Rob Herring <robh@kernel.org> Cc: Stephen Boyd <sboyd@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, linux-arm-msm <linux-arm-msm@vger.kernel.org>, linux-clk <linux-clk@vger.kernel.org>, devicetree@vger.kernel.org, "ARM/QUALCOMM SUPPORT <linux-soc@vger.kernel.org>, Andy Gross" <andy.gross@linaro.org>, "open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM" <linux-remoteproc@vger.kernel.org> Subject: Re: [v2 1/2] dt-bindings: clock: qcom: Add QCOM Q6SSTOP clock controller bindings Date: Fri, 23 Aug 2019 18:48:57 +0530 [thread overview] Message-ID: <2454a240a165862c551b52b224ceedab@codeaurora.org> (raw) In-Reply-To: <CAL_JsqK-GK8arfRu6sqP9UjNrwc0=aUWXymMRF5fQhg+M2TNng@mail.gmail.com> Hi Rob, On 2019-08-13 19:13, Rob Herring wrote: > On Tue, Aug 13, 2019 at 7:10 AM Govind Singh <govinds@codeaurora.org> > wrote: >> >> Add devicetree binding for the Q6SSTOP clock controller found in >> QCS404. > > You need to test this with 'make dt_binding_check' and fix the errors. > Fixed in v3. >> >> Signed-off-by: Govind Singh <govinds@codeaurora.org> >> --- >> .../bindings/clock/qcom,q6sstopcc.yaml | 45 >> +++++++++++++++++++ >> 1 file changed, 45 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> >> diff --git >> a/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> b/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> new file mode 100644 >> index 000000000000..861e9ba97ca3 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> @@ -0,0 +1,45 @@ >> +# SPDX-License-Identifier: BSD-2-Clause > > GPL-2.0-only OR BSD-2-Clause > I have kept BSD-2-Clause as its new binding. >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/interconnect/qcom,osm-l3.yaml# > > needs updating > Fixed in V3. >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Q6SSTOP clock Controller >> + >> +maintainers: >> + - Govind Singh <govinds@codeaurora.org> >> + >> +description: >> + Q6SSTOP clock controller is used by WCSS remoteproc driver >> + to bring WDSP out of reset. >> + >> +properties: >> + compatible: >> + const: "qcom,qcs404-q6sstopcc" >> + >> + reg: >> + maxItems: 2 >> + description: Q6SSTOP clocks register region >> + description: Q6SSTOP_TCSR register region > > Not valid json-schema > Fixed in V3. >> + >> + clocks: >> + items: >> + - description: ahb clock for the q6sstopCC > > Single item just needs 'maxItems: 1' > >> + >> + '#clock-cells': >> + const: 1 >> + >> +required: >> + - compatible >> + - reg >> + - clocks >> + - '#clock-cells' > > Should have an 'additionalProperties: false' here. > Fixed in v3. >> + >> +examples: >> + - | >> + q6sstopcc: clock-controller@7500000 { >> + compatible = "qcom,qcs404-q6sstopcc"; >> + reg = <0x07500000 0x4e000>, <0x07550000 0x10000>; >> + clocks = <&gcc GCC_WCSS_Q6_AHB_CLK>; >> + #clock-cells = <1>; >> + }; >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora >> Forum, >> a Linux Foundation Collaborative Project >> BR, Govind
WARNING: multiple messages have this Message-ID (diff)
From: Govind Singh <govinds@codeaurora.org> To: Rob Herring <robh@kernel.org> Cc: Stephen Boyd <sboyd@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, linux-arm-msm <linux-arm-msm@vger.kernel.org>, linux-clk <linux-clk@vger.kernel.org>, devicetree@vger.kernel.org, "open list:ARM/QUALCOMM SUPPORT" <linux-soc@vger.kernel.org>, Andy Gross <andy.gross@linaro.org>, "open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM" <linux-remoteproc@vger.kernel.org> Subject: Re: [v2 1/2] dt-bindings: clock: qcom: Add QCOM Q6SSTOP clock controller bindings Date: Fri, 23 Aug 2019 18:48:57 +0530 [thread overview] Message-ID: <2454a240a165862c551b52b224ceedab@codeaurora.org> (raw) In-Reply-To: <CAL_JsqK-GK8arfRu6sqP9UjNrwc0=aUWXymMRF5fQhg+M2TNng@mail.gmail.com> Hi Rob, On 2019-08-13 19:13, Rob Herring wrote: > On Tue, Aug 13, 2019 at 7:10 AM Govind Singh <govinds@codeaurora.org> > wrote: >> >> Add devicetree binding for the Q6SSTOP clock controller found in >> QCS404. > > You need to test this with 'make dt_binding_check' and fix the errors. > Fixed in v3. >> >> Signed-off-by: Govind Singh <govinds@codeaurora.org> >> --- >> .../bindings/clock/qcom,q6sstopcc.yaml | 45 >> +++++++++++++++++++ >> 1 file changed, 45 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> >> diff --git >> a/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> b/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> new file mode 100644 >> index 000000000000..861e9ba97ca3 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/clock/qcom,q6sstopcc.yaml >> @@ -0,0 +1,45 @@ >> +# SPDX-License-Identifier: BSD-2-Clause > > GPL-2.0-only OR BSD-2-Clause > I have kept BSD-2-Clause as its new binding. >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/interconnect/qcom,osm-l3.yaml# > > needs updating > Fixed in V3. >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Q6SSTOP clock Controller >> + >> +maintainers: >> + - Govind Singh <govinds@codeaurora.org> >> + >> +description: >> + Q6SSTOP clock controller is used by WCSS remoteproc driver >> + to bring WDSP out of reset. >> + >> +properties: >> + compatible: >> + const: "qcom,qcs404-q6sstopcc" >> + >> + reg: >> + maxItems: 2 >> + description: Q6SSTOP clocks register region >> + description: Q6SSTOP_TCSR register region > > Not valid json-schema > Fixed in V3. >> + >> + clocks: >> + items: >> + - description: ahb clock for the q6sstopCC > > Single item just needs 'maxItems: 1' > >> + >> + '#clock-cells': >> + const: 1 >> + >> +required: >> + - compatible >> + - reg >> + - clocks >> + - '#clock-cells' > > Should have an 'additionalProperties: false' here. > Fixed in v3. >> + >> +examples: >> + - | >> + q6sstopcc: clock-controller@7500000 { >> + compatible = "qcom,qcs404-q6sstopcc"; >> + reg = <0x07500000 0x4e000>, <0x07550000 0x10000>; >> + clocks = <&gcc GCC_WCSS_Q6_AHB_CLK>; >> + #clock-cells = <1>; >> + }; >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora >> Forum, >> a Linux Foundation Collaborative Project >> BR, Govind
next prev parent reply other threads:[~2019-08-23 13:18 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-13 13:09 [v2 0/2] Add Q6SSTOP clock controller for QCS404 Govind Singh 2019-08-13 13:09 ` [v2 1/2] dt-bindings: clock: qcom: Add QCOM Q6SSTOP clock controller bindings Govind Singh 2019-08-13 13:43 ` Rob Herring 2019-08-13 13:43 ` Rob Herring 2019-08-14 6:41 ` Bjorn Andersson 2019-08-14 6:41 ` Bjorn Andersson 2019-08-14 6:41 ` Bjorn Andersson 2019-08-14 23:17 ` Rob Herring 2019-08-14 23:17 ` Rob Herring 2019-08-23 13:18 ` Govind Singh [this message] 2019-08-23 13:18 ` Govind Singh 2019-08-13 13:09 ` [v2 2/2] clk: qcom: Add Q6SSTOP clock controller for QCS404 Govind Singh 2019-08-13 18:32 ` [v2 0/2] " Stephen Boyd 2019-08-13 18:32 ` Stephen Boyd
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=2454a240a165862c551b52b224ceedab@codeaurora.org \ --to=govinds@codeaurora.org \ --cc=andy.gross@linaro.org \ --cc=bjorn.andersson@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-remoteproc@vger.kernel.org \ --cc=robh@kernel.org \ --cc=sboyd@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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.