diff mbox series

[v2,4/7] dt-bindings: mailbox: qcom,apcs-kpss-global: drop mbox-names from example

Message ID 20230208101545.45711-4-krzysztof.kozlowski@linaro.org (mailing list archive)
State Accepted
Commit 7c855ef7137a67bcff0e039691b969428dd8ef6a
Headers show
Series [v2,1/7] arm64: dts: qcom: ipq6018: align RPM G-Link node with bindings | expand

Commit Message

Krzysztof Kozlowski Feb. 8, 2023, 10:15 a.m. UTC
Qualcomm G-Link RPM edge bindings do not allow and do not use mbox-names
property.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

Changes since v1:
1. None.
2. Previously was sent as separate patch.

There are no strict dependencies. This can go anytime. The next patch
(glink-rpm-edge) should be applied in the same or later cycle (could be
via different trees).
---
 .../devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml       | 1 -
 1 file changed, 1 deletion(-)

Comments

Rob Herring (Arm) Feb. 9, 2023, 6:01 p.m. UTC | #1
On Wed, 08 Feb 2023 11:15:42 +0100, Krzysztof Kozlowski wrote:
> Qualcomm G-Link RPM edge bindings do not allow and do not use mbox-names
> property.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Changes since v1:
> 1. None.
> 2. Previously was sent as separate patch.
> 
> There are no strict dependencies. This can go anytime. The next patch
> (glink-rpm-edge) should be applied in the same or later cycle (could be
> via different trees).
> ---
>  .../devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml       | 1 -
>  1 file changed, 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Rob Herring Feb. 15, 2023, 2:09 p.m. UTC | #2
On Wed, Feb 8, 2023 at 4:16 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> Qualcomm G-Link RPM edge bindings do not allow and do not use mbox-names
> property.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> ---
>
> Changes since v1:
> 1. None.
> 2. Previously was sent as separate patch.
>
> There are no strict dependencies. This can go anytime. The next patch
> (glink-rpm-edge) should be applied in the same or later cycle (could be
> via different trees).

Except that without this patch there is now a warning in linux-next.

Rob
Bjorn Andersson Feb. 15, 2023, 7:21 p.m. UTC | #3
On Wed, Feb 08, 2023 at 11:15:42AM +0100, Krzysztof Kozlowski wrote:
> Qualcomm G-Link RPM edge bindings do not allow and do not use mbox-names
> property.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Changes since v1:
> 1. None.
> 2. Previously was sent as separate patch.
> 
> There are no strict dependencies. This can go anytime. The next patch
> (glink-rpm-edge) should be applied in the same or later cycle (could be
> via different trees).

Jassi, I missed this note and merged the following patches, introducing
the forecasted issue in linux-next. To remedy this, I took the liberty
of merging this patch as well into the remoteproc tree.

Regards,
Bjorn

> ---
>  .../devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml       | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> index 943f9472ae10..56b386b688b3 100644
> --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> @@ -155,7 +155,6 @@ examples:
>          interrupts = <GIC_SPI 168 IRQ_TYPE_EDGE_RISING>;
>          qcom,rpm-msg-ram = <&rpm_msg_ram>;
>          mboxes = <&apcs_glb 0>;
> -        mbox-names = "rpm_hlos";
>      };
>  
>    # Example apcs with qcs404
> -- 
> 2.34.1
>
Jassi Brar Feb. 15, 2023, 7:46 p.m. UTC | #4
On Wed, Feb 15, 2023 at 1:19 PM Bjorn Andersson <andersson@kernel.org> wrote:
>
> On Wed, Feb 08, 2023 at 11:15:42AM +0100, Krzysztof Kozlowski wrote:
> > Qualcomm G-Link RPM edge bindings do not allow and do not use mbox-names
> > property.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >
> > ---
> >
> > Changes since v1:
> > 1. None.
> > 2. Previously was sent as separate patch.
> >
> > There are no strict dependencies. This can go anytime. The next patch
> > (glink-rpm-edge) should be applied in the same or later cycle (could be
> > via different trees).
>
> Jassi, I missed this note and merged the following patches, introducing
> the forecasted issue in linux-next. To remedy this, I took the liberty
> of merging this patch as well into the remoteproc tree.
>
No problem.

Cheers.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
index 943f9472ae10..56b386b688b3 100644
--- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
+++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
@@ -155,7 +155,6 @@  examples:
         interrupts = <GIC_SPI 168 IRQ_TYPE_EDGE_RISING>;
         qcom,rpm-msg-ram = <&rpm_msg_ram>;
         mboxes = <&apcs_glb 0>;
-        mbox-names = "rpm_hlos";
     };
 
   # Example apcs with qcs404