Re: [PATCH] dt-bindings: dmaengine: zynqmp_dma: add xlnx,bus-width required property

[Peter Korsgaard]

>>>>> "Radhey" == Radhey Shyam Pandey <radhey.shyam.pandey@xxxxxxx> writes:

 > xlnx,bus-width is a required property. In yaml conversion somehow
 > it got missed out. Bring it back and mention it in required list.
 > Also add Harini and myself to the maintainer list.

 > Fixes: 5a04982df8da ("dt-bindings: dmaengine: zynqmp_dma: convert to yaml")
 > Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@xxxxxxx>
 > ---
 >  .../devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml    | 3 +++
 >  1 file changed, 3 insertions(+)

 > diff --git a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml
 > index c0a1408b12ec..a10019d3a650 100644
 > --- a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml
 > +++ b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml
 > @@ -13,6 +13,8 @@ description: |
 
 >  maintainers:
 >    - Michael Tretter <m.tretter@xxxxxxxxxxxxxx>
 > +  - Harini Katakam <harini.katakam@xxxxxxx>
 > +  - Radhey Shyam Pandey <radhey.shyam.pandey@xxxxxxx>

That should ideally be in a separate patch.


 >  allOf:
 >    - $ref: "../dma-controller.yaml#"
 > @@ -65,6 +67,7 @@ required:
 >    - interrupts
 >    - clocks
 >    - clock-names
 > +  - xlnx,bus-width

Reviewed-by: Peter Korsgaard <peter@xxxxxxxxxxxxx>

-- 
Bye, Peter Korsgaard