Re: [PATCH v5 3/5] dt-bindings: dma: arm,dma-350: Document optional clocks and resets

[Jelly Jia]

Hi Frank,

> Suppose you should add front compatible string and fall back to arm,dma-350
>
> and use front compatible at if branch to restrict clocks and resets.

Thanks. I've updated the binding locally as follows:

  compatible:
    oneOf:
      - const: arm,dma-350
      - items:
          - const: cix,sky1-dma350
          - const: arm,dma-350

and added this condition under allOf:

  - if:
      properties:
        compatible:
          contains:
            const: cix,sky1-dma350
    then:
      required:
        - clocks

This requires a clock for Sky1, while resets remain optional. Both
resources remain optional for the generic compatible alone.

Both FCH and AUDSS DMA nodes now use the CIX compatible with the generic
fallback. The driver's optional clock/reset helpers remain unchanged.

I've tested the updated tree on Sky1 and will include these changes
in the next revision.

Best regards,
Jelly