Re: [PATCH v2 1/3] dt-bindings: lgm-dma: Added intel,dma-sw-desc property.

[Krzysztof Kozlowski]

On 08/08/2025 05:22, Zhu Yixin wrote:
> If the intel,dma-sw-desc property is present, it indicates that
> the DMA driver is responsible for managing the DMA engine, including
> handing the DMA descriptors.
> If this property is present, the system defaults to a mode where
> another hardware component takes control of the DMA engine.
> In this case, the hardware can generate and manage descriptors
> independently, without driver involvement.
> 
> This property only takes effect on HDMA(DMAV3x).
> 
> Signed-off-by: Zhu Yixin <yzhu@xxxxxxxxxxxxx>

Where is the changelog?

> ---
>  Documentation/devicetree/bindings/dma/intel,ldma.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/dma/intel,ldma.yaml b/Documentation/devicetree/bindings/dma/intel,ldma.yaml
> index d6bb553a2c6f..426168b8709e 100644
> --- a/Documentation/devicetree/bindings/dma/intel,ldma.yaml
> +++ b/Documentation/devicetree/bindings/dma/intel,ldma.yaml
> @@ -80,6 +80,12 @@ properties:
>        if it is disabled, the DMA RX will still support programmable fixed burst size of 2,4,8,16.
>        It only applies to RX DMA and memcopy DMA.
>  
> +  intel,dma-sw-desc:
> +    type: boolean
> +    description:
> +      Indicates that the DMA driver should operate in software-managed mode.
> +      If this property is not present, it implies that DMA descriptors are managed and generated by another hardware component that controls the DMA engine.

Nothing improved:
1. Still wrongly wrapped.
2. Still SW property.

I don't think you read the feedback. You need to describe hardware, not SW.

Best regards,
Krzysztof