* [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings
@ 2023-02-07 9:08 Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 2/5] arm64: dts: qcom: qcs404: " Krzysztof Kozlowski
` (5 more replies)
0 siblings, 6 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Bindings expect (and most of DTS use) the RPM G-Link node name to be
"rpm-requests".
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index d32c9b2515ee..bbd94025ff5d 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -176,7 +176,7 @@ rpm-glink {
qcom,rpm-msg-ram = <&rpm_msg_ram>;
mboxes = <&apcs_glb 0>;
- rpm_requests: glink-channel {
+ rpm_requests: rpm-requests {
compatible = "qcom,rpm-ipq6018";
qcom,glink-channels = "rpm_requests";
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 2/5] arm64: dts: qcom: qcs404: align RPM G-Link node with bindings
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
@ 2023-02-07 9:08 ` Krzysztof Kozlowski
2023-02-07 10:28 ` Dmitry Baryshkov
2023-02-07 9:08 ` [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller " Krzysztof Kozlowski
` (4 subsequent siblings)
5 siblings, 1 reply; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Bindings expect (and most of DTS use) the RPM G-Link node name to be
"rpm-requests".
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/qcom/qcs404.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/qcs404.dtsi b/arch/arm64/boot/dts/qcom/qcs404.dtsi
index 9c9890cf1b10..bc2ed73afb74 100644
--- a/arch/arm64/boot/dts/qcom/qcs404.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcs404.dtsi
@@ -223,7 +223,7 @@ rpm-glink {
qcom,rpm-msg-ram = <&rpm_msg_ram>;
mboxes = <&apcs_glb 0>;
- rpm_requests: glink-channel {
+ rpm_requests: rpm-requests {
compatible = "qcom,rpm-qcs404";
qcom,glink-channels = "rpm_requests";
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller node with bindings
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 2/5] arm64: dts: qcom: qcs404: " Krzysztof Kozlowski
@ 2023-02-07 9:08 ` Krzysztof Kozlowski
2023-02-07 10:28 ` Dmitry Baryshkov
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
` (3 subsequent siblings)
5 siblings, 1 reply; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
qcom,rpmcc bindings expect RPM clock controller to be named generic
"clock-controller":
rpm-glink: rpm-requests: 'qcom,rpmcc' does not match any of the regexes: '^regulators(-[01])?$', 'pinctrl-[0-9]+'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 55180586f7b6..e0d46dbd498f 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -483,7 +483,7 @@ rpm_requests: rpm-requests {
compatible = "qcom,rpm-msm8996";
qcom,glink-channels = "rpm_requests";
- rpmcc: qcom,rpmcc {
+ rpmcc: clock-controller {
compatible = "qcom,rpmcc-msm8996", "qcom,rpmcc";
#clock-cells = <1>;
clocks = <&xo_board>;
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 2/5] arm64: dts: qcom: qcs404: " Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller " Krzysztof Kozlowski
@ 2023-02-07 9:08 ` Krzysztof Kozlowski
2023-02-07 9:59 ` Krzysztof Kozlowski
` (2 more replies)
2023-02-07 9:08 ` [PATCH 5/5] dt-bindings: remoteproc: qcom,glink-edge: correct label description Krzysztof Kozlowski
` (2 subsequent siblings)
5 siblings, 3 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Convert Qualcomm G-Link RPM edge binding to DT schema. Move it to
remoteproc as it better suits the purpose - communication channel with
remote processor.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../remoteproc/qcom,glink-rpm-edge.yaml | 92 ++++++++++++++++++
.../bindings/soc/qcom/qcom,glink.txt | 94 -------------------
2 files changed, 92 insertions(+), 94 deletions(-)
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
new file mode 100644
index 000000000000..07cd67ec97c1
--- /dev/null
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
@@ -0,0 +1,92 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/remoteproc/qcom,glink-rpm-edge.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Qualcomm G-Link RPM edge
+
+description: |
+ Qualcomm G-Link edge, a FIFO based mechanism for communication with Resource
+ Power Manager (RPM) on various Qualcomm platforms.
+
+maintainers:
+ - Bjorn Andersson <andersson@kernel.org>
+
+properties:
+ compatible:
+ const: qcom,glink-rpm
+
+ label:
+ $ref: /schemas/types.yaml#/definitions/string
+ description:
+ Name of the edge, used for debugging and identification purposes. The
+ node name will be used if this is not present.
+
+ interrupts:
+ maxItems: 1
+
+ mboxes:
+ items:
+ - description: rpm_hlos mailbox in APCS
+
+ qcom,remote-pid:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description:
+ The identifier for the remote processor as known by the rest of the
+ system.
+
+ qcom,rpm-msg-ram:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description: |
+ RPM message memory resource (compatible:: qcom,rpm-msg-ram).
+
+ rpm-requests:
+ type: object
+ $ref: /schemas/soc/qcom/qcom,smd-rpm.yaml#
+ unevaluatedProperties: false
+ description:
+ Qualcomm Resource Power Manager (RPM) over G-Link
+
+ properties:
+ qcom,intents:
+ $ref: /schemas/types.yaml#/definitions/uint32-array
+ description:
+ List of (size, amount) pairs describing what intents should be
+ preallocated for this virtual channel. This can be used to tweak the
+ default intents available for the channel to meet expectations of the
+ remote.
+
+ required:
+ - qcom,glink-channels
+
+required:
+ - compatible
+ - interrupts
+ - mboxes
+
+anyOf:
+ - required:
+ - qcom,remote-pid
+ - required:
+ - qcom,rpm-msg-ram
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+ rpm-glink {
+ compatible = "qcom,glink-rpm";
+ interrupts = <GIC_SPI 168 IRQ_TYPE_EDGE_RISING>;
+ mboxes = <&apcs_glb 0>;
+ qcom,rpm-msg-ram = <&rpm_msg_ram>;
+
+ rpm-requests {
+ compatible = "qcom,rpm-msm8996";
+ qcom,glink-channels = "rpm_requests";
+
+ /* ... */
+ };
+ };
diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
deleted file mode 100644
index 1214192847ac..000000000000
--- a/Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
+++ /dev/null
@@ -1,94 +0,0 @@
-Qualcomm GLINK edge binding
-
-This binding describes a Qualcomm GLINK edge, a fifo based mechanism for
-communication between subsystem-pairs on various Qualcomm platforms. Two types
-of edges can be described by the binding; the GLINK RPM edge and a SMEM based
-edge.
-
-- compatible:
- Usage: required for glink-rpm
- Value type: <stringlist>
- Definition: must be "qcom,glink-rpm"
-
-- label:
- Usage: optional
- Value type: <string>
- Definition: should specify the subsystem name this edge corresponds to.
-
-- interrupts:
- Usage: required
- Value type: <prop-encoded-array>
- Definition: should specify the IRQ used by the remote processor to
- signal this processor about communication related events
-
-- qcom,remote-pid:
- Usage: required for glink-smem
- Value type: <u32>
- Definition: specifies the identifier of the remote endpoint of this edge
-
-- qcom,rpm-msg-ram:
- Usage: required for glink-rpm
- Value type: <prop-encoded-array>
- Definition: handle to RPM message memory resource
-
-- mboxes:
- Usage: required
- Value type: <prop-encoded-array>
- Definition: reference to the "rpm_hlos" mailbox in APCS, as described
- in mailbox/mailbox.txt
-
-= GLINK DEVICES
-Each subnode of the GLINK node represent function tied to a virtual
-communication channel. The name of the nodes are not important. The properties
-of these nodes are defined by the individual bindings for the specific function
-- but must contain the following property:
-
-- qcom,glink-channels:
- Usage: required
- Value type: <stringlist>
- Definition: a list of channels tied to this function, used for matching
- the function to a set of virtual channels
-
-- qcom,intents:
- Usage: optional
- Value type: <prop-encoded-array>
- Definition: a list of size,amount pairs describing what intents should
- be preallocated for this virtual channel. This can be used
- to tweak the default intents available for the channel to
- meet expectations of the remote.
-
-= EXAMPLE
-The following example represents the GLINK RPM node on a MSM8996 device, with
-the function for the "rpm_request" channel defined, which is used for
-regulators and root clocks.
-
- apcs_glb: mailbox@9820000 {
- compatible = "qcom,msm8996-apcs-hmss-global";
- reg = <0x9820000 0x1000>;
-
- #mbox-cells = <1>;
- };
-
- rpm_msg_ram: memory@68000 {
- compatible = "qcom,rpm-msg-ram";
- reg = <0x68000 0x6000>;
- };
-
- rpm-glink {
- compatible = "qcom,glink-rpm";
-
- interrupts = <GIC_SPI 168 IRQ_TYPE_EDGE_RISING>;
-
- qcom,rpm-msg-ram = <&rpm_msg_ram>;
-
- mboxes = <&apcs_glb 0>;
-
- rpm-requests {
- compatible = "qcom,rpm-msm8996";
- qcom,glink-channels = "rpm_requests";
-
- qcom,intents = <0x400 5
- 0x800 1>;
- ...
- };
- };
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 5/5] dt-bindings: remoteproc: qcom,glink-edge: correct label description
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
` (2 preceding siblings ...)
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
@ 2023-02-07 9:08 ` Krzysztof Kozlowski
2023-02-07 10:28 ` [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Dmitry Baryshkov
2023-02-08 8:20 ` Konrad Dybcio
5 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:08 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Correct the description of 'label' property.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/remoteproc/qcom,glink-edge.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,glink-edge.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,glink-edge.yaml
index 8e133ab55ff3..15e6851e1ff8 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,glink-edge.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,glink-edge.yaml
@@ -42,7 +42,9 @@ properties:
maxItems: 1
label:
- description: The names of the state bits used for SMP2P output
+ description:
+ Name of the edge, used for debugging and identification purposes. The
+ node name will be used if this is not present.
mboxes:
maxItems: 1
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
@ 2023-02-07 9:59 ` Krzysztof Kozlowski
2023-02-07 13:00 ` Rob Herring
2023-02-07 18:42 ` Rob Herring
2 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 9:59 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
On 07/02/2023 10:08, Krzysztof Kozlowski wrote:
> Convert Qualcomm G-Link RPM edge binding to DT schema. Move it to
> remoteproc as it better suits the purpose - communication channel with
> remote processor.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../remoteproc/qcom,glink-rpm-edge.yaml | 92 ++++++++++++++++++
> .../bindings/soc/qcom/qcom,glink.txt | 94 -------------------
> 2 files changed, 92 insertions(+), 94 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
> delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
This is expected to trigger warning in other bindings, being fixed here:
https://lore.kernel.org/linux-arm-msm/20230207095639.36537-1-krzysztof.kozlowski@linaro.org/T/#u
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
` (3 preceding siblings ...)
2023-02-07 9:08 ` [PATCH 5/5] dt-bindings: remoteproc: qcom,glink-edge: correct label description Krzysztof Kozlowski
@ 2023-02-07 10:28 ` Dmitry Baryshkov
2023-02-08 8:20 ` Konrad Dybcio
5 siblings, 0 replies; 13+ messages in thread
From: Dmitry Baryshkov @ 2023-02-07 10:28 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Mathieu Poirier, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
On 07/02/2023 11:08, Krzysztof Kozlowski wrote:
> Bindings expect (and most of DTS use) the RPM G-Link node name to be
> "rpm-requests".
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/5] arm64: dts: qcom: qcs404: align RPM G-Link node with bindings
2023-02-07 9:08 ` [PATCH 2/5] arm64: dts: qcom: qcs404: " Krzysztof Kozlowski
@ 2023-02-07 10:28 ` Dmitry Baryshkov
0 siblings, 0 replies; 13+ messages in thread
From: Dmitry Baryshkov @ 2023-02-07 10:28 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Mathieu Poirier, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
On 07/02/2023 11:08, Krzysztof Kozlowski wrote:
> Bindings expect (and most of DTS use) the RPM G-Link node name to be
> "rpm-requests".
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/qcs404.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller node with bindings
2023-02-07 9:08 ` [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller " Krzysztof Kozlowski
@ 2023-02-07 10:28 ` Dmitry Baryshkov
0 siblings, 0 replies; 13+ messages in thread
From: Dmitry Baryshkov @ 2023-02-07 10:28 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Mathieu Poirier, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
On 07/02/2023 11:08, Krzysztof Kozlowski wrote:
> qcom,rpmcc bindings expect RPM clock controller to be named generic
> "clock-controller":
>
> rpm-glink: rpm-requests: 'qcom,rpmcc' does not match any of the regexes: '^regulators(-[01])?$', 'pinctrl-[0-9]+'
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
2023-02-07 9:59 ` Krzysztof Kozlowski
@ 2023-02-07 13:00 ` Rob Herring
2023-02-07 18:42 ` Rob Herring
2 siblings, 0 replies; 13+ messages in thread
From: Rob Herring @ 2023-02-07 13:00 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Bjorn Andersson, linux-remoteproc, linux-arm-msm, Rob Herring,
linux-kernel, Andy Gross, devicetree, Konrad Dybcio,
Mathieu Poirier, Krzysztof Kozlowski
On Tue, 07 Feb 2023 10:08:51 +0100, Krzysztof Kozlowski wrote:
> Convert Qualcomm G-Link RPM edge binding to DT schema. Move it to
> remoteproc as it better suits the purpose - communication channel with
> remote processor.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../remoteproc/qcom,glink-rpm-edge.yaml | 92 ++++++++++++++++++
> .../bindings/soc/qcom/qcom,glink.txt | 94 -------------------
> 2 files changed, 92 insertions(+), 94 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
> delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.example.dtb: rpm-glink: 'mbox-names' does not match any of the regexes: 'pinctrl-[0-9]+'
From schema: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230207090852.28421-4-krzysztof.kozlowski@linaro.org
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
2023-02-07 9:59 ` Krzysztof Kozlowski
2023-02-07 13:00 ` Rob Herring
@ 2023-02-07 18:42 ` Rob Herring
2023-02-08 9:51 ` Krzysztof Kozlowski
2 siblings, 1 reply; 13+ messages in thread
From: Rob Herring @ 2023-02-07 18:42 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Krzysztof Kozlowski, linux-arm-msm, linux-remoteproc, devicetree,
linux-kernel
On Tue, Feb 07, 2023 at 10:08:51AM +0100, Krzysztof Kozlowski wrote:
> Convert Qualcomm G-Link RPM edge binding to DT schema. Move it to
> remoteproc as it better suits the purpose - communication channel with
> remote processor.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../remoteproc/qcom,glink-rpm-edge.yaml | 92 ++++++++++++++++++
> .../bindings/soc/qcom/qcom,glink.txt | 94 -------------------
> 2 files changed, 92 insertions(+), 94 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
> delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
>
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
> new file mode 100644
> index 000000000000..07cd67ec97c1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
> @@ -0,0 +1,92 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/remoteproc/qcom,glink-rpm-edge.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Qualcomm G-Link RPM edge
> +
> +description: |
> + Qualcomm G-Link edge, a FIFO based mechanism for communication with Resource
> + Power Manager (RPM) on various Qualcomm platforms.
> +
> +maintainers:
> + - Bjorn Andersson <andersson@kernel.org>
> +
> +properties:
> + compatible:
> + const: qcom,glink-rpm
> +
> + label:
> + $ref: /schemas/types.yaml#/definitions/string
> + description:
> + Name of the edge, used for debugging and identification purposes. The
> + node name will be used if this is not present.
> +
> + interrupts:
> + maxItems: 1
> +
> + mboxes:
> + items:
> + - description: rpm_hlos mailbox in APCS
> +
> + qcom,remote-pid:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description:
> + The identifier for the remote processor as known by the rest of the
> + system.
> +
> + qcom,rpm-msg-ram:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description: |
> + RPM message memory resource (compatible:: qcom,rpm-msg-ram).
'::' is not a thing in YAML.
> +
> + rpm-requests:
> + type: object
> + $ref: /schemas/soc/qcom/qcom,smd-rpm.yaml#
> + unevaluatedProperties: false
> + description:
> + Qualcomm Resource Power Manager (RPM) over G-Link
> +
> + properties:
> + qcom,intents:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> + description:
> + List of (size, amount) pairs describing what intents should be
pairs? Sounds like a matrix then.
> + preallocated for this virtual channel. This can be used to tweak the
> + default intents available for the channel to meet expectations of the
> + remote.
> +
> + required:
> + - qcom,glink-channels
> +
> +required:
> + - compatible
> + - interrupts
> + - mboxes
> +
> +anyOf:
> + - required:
> + - qcom,remote-pid
> + - required:
> + - qcom,rpm-msg-ram
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> +
> + rpm-glink {
> + compatible = "qcom,glink-rpm";
> + interrupts = <GIC_SPI 168 IRQ_TYPE_EDGE_RISING>;
> + mboxes = <&apcs_glb 0>;
> + qcom,rpm-msg-ram = <&rpm_msg_ram>;
> +
> + rpm-requests {
> + compatible = "qcom,rpm-msm8996";
> + qcom,glink-channels = "rpm_requests";
> +
> + /* ... */
> + };
> + };
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
` (4 preceding siblings ...)
2023-02-07 10:28 ` [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Dmitry Baryshkov
@ 2023-02-08 8:20 ` Konrad Dybcio
5 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-02-08 8:20 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson,
Mathieu Poirier, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel
On 7.02.2023 10:08, Krzysztof Kozlowski wrote:
> Bindings expect (and most of DTS use) the RPM G-Link node name to be
> "rpm-requests".
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
It's not the glink node, it's the rpm subnode of glink.
For the change:
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> index d32c9b2515ee..bbd94025ff5d 100644
> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> @@ -176,7 +176,7 @@ rpm-glink {
> qcom,rpm-msg-ram = <&rpm_msg_ram>;
> mboxes = <&apcs_glb 0>;
>
> - rpm_requests: glink-channel {
> + rpm_requests: rpm-requests {
> compatible = "qcom,rpm-ipq6018";
> qcom,glink-channels = "rpm_requests";
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema
2023-02-07 18:42 ` Rob Herring
@ 2023-02-08 9:51 ` Krzysztof Kozlowski
0 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-08 9:51 UTC (permalink / raw)
To: Rob Herring
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, Mathieu Poirier,
Krzysztof Kozlowski, linux-arm-msm, linux-remoteproc, devicetree,
linux-kernel
On 07/02/2023 19:42, Rob Herring wrote:
> On Tue, Feb 07, 2023 at 10:08:51AM +0100, Krzysztof Kozlowski wrote:
>> Convert Qualcomm G-Link RPM edge binding to DT schema. Move it to
>> remoteproc as it better suits the purpose - communication channel with
>> remote processor.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>> .../remoteproc/qcom,glink-rpm-edge.yaml | 92 ++++++++++++++++++
>> .../bindings/soc/qcom/qcom,glink.txt | 94 -------------------
>> 2 files changed, 92 insertions(+), 94 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
>> delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,glink.txt
>>
>> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
>> new file mode 100644
>> index 000000000000..07cd67ec97c1
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,glink-rpm-edge.yaml
>> @@ -0,0 +1,92 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/remoteproc/qcom,glink-rpm-edge.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Qualcomm G-Link RPM edge
>> +
>> +description: |
>> + Qualcomm G-Link edge, a FIFO based mechanism for communication with Resource
>> + Power Manager (RPM) on various Qualcomm platforms.
>> +
>> +maintainers:
>> + - Bjorn Andersson <andersson@kernel.org>
>> +
>> +properties:
>> + compatible:
>> + const: qcom,glink-rpm
>> +
>> + label:
>> + $ref: /schemas/types.yaml#/definitions/string
>> + description:
>> + Name of the edge, used for debugging and identification purposes. The
>> + node name will be used if this is not present.
>> +
>> + interrupts:
>> + maxItems: 1
>> +
>> + mboxes:
>> + items:
>> + - description: rpm_hlos mailbox in APCS
>> +
>> + qcom,remote-pid:
>> + $ref: /schemas/types.yaml#/definitions/uint32
>> + description:
>> + The identifier for the remote processor as known by the rest of the
>> + system.
>> +
>> + qcom,rpm-msg-ram:
>> + $ref: /schemas/types.yaml#/definitions/phandle
>> + description: |
>> + RPM message memory resource (compatible:: qcom,rpm-msg-ram).
>
> '::' is not a thing in YAML.
ack
>
>> +
>> + rpm-requests:
>> + type: object
>> + $ref: /schemas/soc/qcom/qcom,smd-rpm.yaml#
>> + unevaluatedProperties: false
>> + description:
>> + Qualcomm Resource Power Manager (RPM) over G-Link
>> +
>> + properties:
>> + qcom,intents:
>> + $ref: /schemas/types.yaml#/definitions/uint32-array
>> + description:
>> + List of (size, amount) pairs describing what intents should be
>
> pairs? Sounds like a matrix then.
I'll fix also second appearance of this property
(bindings/soc/qcom/qcom,apr.yaml)
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2023-02-08 9:52 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 9:08 [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 2/5] arm64: dts: qcom: qcs404: " Krzysztof Kozlowski
2023-02-07 10:28 ` Dmitry Baryshkov
2023-02-07 9:08 ` [PATCH 3/5] arm64: dts: qcom: msm8996: align RPM G-Link clock-controller " Krzysztof Kozlowski
2023-02-07 10:28 ` Dmitry Baryshkov
2023-02-07 9:08 ` [PATCH 4/5] dt-bindings: remoteproc: qcom,glink-rpm-edge: convert to DT schema Krzysztof Kozlowski
2023-02-07 9:59 ` Krzysztof Kozlowski
2023-02-07 13:00 ` Rob Herring
2023-02-07 18:42 ` Rob Herring
2023-02-08 9:51 ` Krzysztof Kozlowski
2023-02-07 9:08 ` [PATCH 5/5] dt-bindings: remoteproc: qcom,glink-edge: correct label description Krzysztof Kozlowski
2023-02-07 10:28 ` [PATCH 1/5] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings Dmitry Baryshkov
2023-02-08 8:20 ` Konrad Dybcio
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.