2021-03-26 09:44:19

by Arnaud Pouliquen

Subject: [PATCH v3 1/2] dt-bindings: remoteproc: stm32-rproc: add new mailbox channel for detach

Add the "detach" mailbox item, that allows to define a mailbox to
send a IPCC signal to the remote processor on remoteproc detach action.

Signed-off-by: Arnaud Pouliquen <[email protected]>
---
No Update from V2.

Update from V1:
Fix indentation error reported by 'make dt_binding_check'.

---
.../bindings/remoteproc/st,stm32-rproc.yaml | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
index a1171dfba024..64afdcfb613d 100644
--- a/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
@@ -65,16 +65,23 @@ properties:
Unidirectional channel:
- from local to remote, where ACK from the remote means that it is
ready for shutdown
+ - description: |
+ A channel (d) used by the local proc to notify the remote proc that it
+ has to stop interprocessor communnication.
+ Unidirectional channel:
+ - from local to remote, where ACK from the remote means that communnication
+ as been stopped on the remote side.
minItems: 1
- maxItems: 3
+ maxItems: 4

mbox-names:
items:
- const: vq0
- const: vq1
- const: shutdown
+ - const: detach
minItems: 1
- maxItems: 3
+ maxItems: 4

memory-region:
description:
--
2.17.1

2021-03-30 07:16:32

by Arnaud Pouliquen

Subject: Re: [PATCH v3 1/2] dt-bindings: remoteproc: stm32-rproc: add new mailbox channel for detach

Hello Rob,

Seems I made a mistake in my mailing list, you are not in...
Please could you review the bindings?
if it is easier for you i can resend the series.

Thanks,
Arnaud

On 3/26/21 10:42 AM, Arnaud Pouliquen wrote:
> Add the "detach" mailbox item, that allows to define a mailbox to
> send a IPCC signal to the remote processor on remoteproc detach action.
>
> Signed-off-by: Arnaud Pouliquen <[email protected]>
> ---
> No Update from V2.
>
> Update from V1:
> Fix indentation error reported by 'make dt_binding_check'.
>
> ---
> .../bindings/remoteproc/st,stm32-rproc.yaml | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
> index a1171dfba024..64afdcfb613d 100644
> --- a/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/st,stm32-rproc.yaml
> @@ -65,16 +65,23 @@ properties:
> Unidirectional channel:
> - from local to remote, where ACK from the remote means that it is
> ready for shutdown
> + - description: |
> + A channel (d) used by the local proc to notify the remote proc that it
> + has to stop interprocessor communnication.
> + Unidirectional channel:
> + - from local to remote, where ACK from the remote means that communnication
> + as been stopped on the remote side.
> minItems: 1
> - maxItems: 3
> + maxItems: 4
>
> mbox-names:
> items:
> - const: vq0
> - const: vq1
> - const: shutdown
> + - const: detach
> minItems: 1
> - maxItems: 3
> + maxItems: 4
>
> memory-region:
> description:
>