Unverified Commit e4cf7805 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Mark Brown
Browse files

ASoC: dt-bindings: qcom,lpass-rx-macro: narrow clocks per variants 



Currently the Qualcomm RX macro codec binding allows two different clock
setups - with (for ADSP) and without macro/dcodec entries (for ADSP
bypassed).  With more devices coming soon, this will keep growing, thus
rework the clocks/clock-names to be specific for each binding.

Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20230313075445.17160-1-krzysztof.kozlowski@linaro.org


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent ffaf886e

Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml

+44 −14
Original line number Diff line number Diff line
@@ -9,9 +9,6 @@ title: LPASS(Low Power Audio Subsystem) RX Macro audio codec 
maintainers:

  - Srinivas Kandagatla <srinivas.kandagatla@linaro.org>



allOf:

  - $ref: dai-common.yaml#



properties:

  compatible:

    enum:
@@ -33,17 +30,8 @@ properties: 
    maxItems: 5



  clock-names:

    oneOf:

      - items:   #for ADSP based platforms

          - const: mclk

          - const: npl

          - const: macro

          - const: dcodec

          - const: fsgen

      - items:   #for ADSP bypass based platforms

          - const: mclk

          - const: npl

          - const: fsgen

    minItems: 3

    maxItems: 5



  clock-output-names:

    maxItems: 1
@@ -61,6 +49,48 @@ required: 
  - reg

  - "#sound-dai-cells"



allOf:

  - $ref: dai-common.yaml#

  - if:

      properties:

        compatible:

          enum:

            - qcom,sc7280-lpass-rx-macro

    then:

      properties:

        clock-names:

          oneOf:

            - items:   #for ADSP based platforms

                - const: mclk

                - const: npl

                - const: macro

                - const: dcodec

                - const: fsgen

            - items:   #for ADSP bypass based platforms

                - const: mclk

                - const: npl

                - const: fsgen



  - if:

      properties:

        compatible:

          enum:

            - qcom,sc8280xp-lpass-rx-macro

            - qcom,sm8250-lpass-rx-macro

            - qcom,sm8450-lpass-rx-macro

    then:

      properties:

        clocks:

          minItems: 5

          maxItems: 5

        clock-names:

          items:

            - const: mclk

            - const: npl

            - const: macro

            - const: dcodec

            - const: fsgen



unevaluatedProperties: false



examples: