RE: [PATCH 2/3] dt-bindings: dma: xilinx: Simplify dma-coherent property

[Katakam, Harini]

[AMD Official Use Only - AMD Internal Distribution Only]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Saturday, November 15, 2025 5:51 PM
> To: Jyri Sarha <jyri.sarha@xxxxxx>; Tomi Valkeinen
> <tomi.valkeinen@xxxxxxxxxxxxxxxx>; David Airlie <airlied@xxxxxxxxx>;
> Simona Vetter <simona@xxxxxxxx>; Maarten Lankhorst
> <maarten.lankhorst@xxxxxxxxxxxxxxx>; Maxime Ripard <mripard@xxxxxxxxxx>;
> Thomas Zimmermann <tzimmermann@xxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley
> <conor+dt@xxxxxxxxxx>; Vinod Koul <vkoul@xxxxxxxxxx>; Simek, Michal
> <michal.simek@xxxxxxx>; Ulf Hansson <ulf.hansson@xxxxxxxxxx>; Michael
> Tretter <m.tretter@xxxxxxxxxxxxxx>; Katakam, Harini
> <harini.katakam@xxxxxxx>; Pandey, Radhey Shyam
> <radhey.shyam.pandey@xxxxxxx>; dri-devel@xxxxxxxxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> dmaengine@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> mmc@xxxxxxxxxxxxxxx
> Cc: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Subject: [PATCH 2/3] dt-bindings: dma: xilinx: Simplify dma-coherent property
>
> Common boolean properties need to be only allowed in the binding (":true"),
> because their type is already defined by core DT schema.
> Simplify dma-coherent property to match common syntax.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>

Thanks,
Reviewed-by: Harini Katakam <harini.katakam@xxxxxxx>

Regards,
Harini

> ---
>  .../devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-1.0.yaml    | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> 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 b5399c65a731..2da86037ad79 100644
> --- a/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-
> 1.0.yaml
> +++ b/Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dma-
> 1.0.y
> +++ aml
> @@ -59,8 +59,7 @@ properties:
>    power-domains:
>      maxItems: 1
>
> -  dma-coherent:
> -    description: present if dma operations are coherent
> +  dma-coherent: true
>
>  required:
>    - "#dma-cells"
> --
> 2.48.1