diff mbox series

[1/2] dt-bindings: dma: altera-msgdma: make response port optional

Message ID fb28146a23a182be9e5435c1d3e5cac36b372294.1623898678.git.olivier.dautricourt@orolia.com
State Accepted
Commit 4aece33cacf726d34ecd8824aee369652ec2beec
Headers show
Series *** altera-msgdma: make response port optional *** | expand

Commit Message

Olivier Dautricourt June 17, 2021, 7:52 p.m. UTC
Response port is not required in some configuration of the IP core.

Signed-off-by: Olivier Dautricourt <olivier.dautricourt@orolia.com>
---
 Documentation/devicetree/bindings/dma/altr,msgdma.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--
2.31.0.rc2

Comments

Stefan Roese June 18, 2021, 5:54 a.m. UTC | #1
On 17.06.21 21:52, Olivier Dautricourt wrote:
> Response port is not required in some configuration of the IP core.

> 

> Signed-off-by: Olivier Dautricourt <olivier.dautricourt@orolia.com>


Reviewed-by: Stefan Roese <sr@denx.de>


Thanks,
Stefan

> ---

>   Documentation/devicetree/bindings/dma/altr,msgdma.yaml | 4 +++-

>   1 file changed, 3 insertions(+), 1 deletion(-)

> 

> diff --git a/Documentation/devicetree/bindings/dma/altr,msgdma.yaml b/Documentation/devicetree/bindings/dma/altr,msgdma.yaml

> index a4f9fe23dcd9..b193ee2db4a7 100644

> --- a/Documentation/devicetree/bindings/dma/altr,msgdma.yaml

> +++ b/Documentation/devicetree/bindings/dma/altr,msgdma.yaml

> @@ -24,13 +24,15 @@ properties:

>       items:

>         - description: Control and Status Register Slave Port

>         - description: Descriptor Slave Port

> -      - description: Response Slave Port

> +      - description: Response Slave Port (Optional)

> +    minItems: 2

> 

>     reg-names:

>       items:

>         - const: csr

>         - const: desc

>         - const: resp

> +    minItems: 2

> 

>     interrupts:

>       maxItems: 1

> --

> 2.31.0.rc2

> 



Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
Rob Herring (Arm) June 24, 2021, 9:34 p.m. UTC | #2
On Thu, 17 Jun 2021 21:52:32 +0200, Olivier Dautricourt wrote:
> Response port is not required in some configuration of the IP core.

> 

> Signed-off-by: Olivier Dautricourt <olivier.dautricourt@orolia.com>

> ---

>  Documentation/devicetree/bindings/dma/altr,msgdma.yaml | 4 +++-

>  1 file changed, 3 insertions(+), 1 deletion(-)

> 


Reviewed-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/dma/altr,msgdma.yaml b/Documentation/devicetree/bindings/dma/altr,msgdma.yaml
index a4f9fe23dcd9..b193ee2db4a7 100644
--- a/Documentation/devicetree/bindings/dma/altr,msgdma.yaml
+++ b/Documentation/devicetree/bindings/dma/altr,msgdma.yaml
@@ -24,13 +24,15 @@  properties:
     items:
       - description: Control and Status Register Slave Port
       - description: Descriptor Slave Port
-      - description: Response Slave Port
+      - description: Response Slave Port (Optional)
+    minItems: 2

   reg-names:
     items:
       - const: csr
       - const: desc
       - const: resp
+    minItems: 2

   interrupts:
     maxItems: 1