Re: [PATCH 7/9] ASoC: dt-bindings: meson: convert axg spdif input to schema

On 02/02/2023 19:36, Jerome Brunet wrote:

Convert the DT binding documentation for the Amlogic axg spdif input to schema.

Signed-off-by: Jerome Brunet jbrunet@baylibre.com

.../bindings/sound/amlogic,axg-spdifin.txt | 27 ------ .../bindings/sound/amlogic,axg-spdifin.yaml | 90 +++++++++++++++++++ 2 files changed, 90 insertions(+), 27 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.txt create mode 100644 Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.yaml

diff --git a/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.txt b/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.txt deleted file mode 100644 index df92a4ecf288..000000000000 --- a/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.txt +++ /dev/null @@ -1,27 +0,0 @@ -* Amlogic Audio SPDIF Input

-Required properties: -- compatible: 'amlogic,axg-spdifin' or

•      'amlogic,g12a-spdifin' or
  

•      'amlogic,sm1-spdifin'
  

-- interrupts: interrupt specifier for the spdif input. -- clocks: list of clock phandle, one for each entry clock-names. -- clock-names: should contain the following:

• • "pclk" : peripheral clock.
• • "refclk" : spdif input reference clock

-- #sound-dai-cells: must be 0.

-Optional property: -- resets: phandle to the dedicated reset line of the spdif input.

-Example on the A113 SoC:

-spdifin: audio-controller@400 {

• compatible = "amlogic,axg-spdifin";
• reg = <0x0 0x400 0x0 0x30>;
• #sound-dai-cells = <0>;
• interrupts = <GIC_SPI 87 IRQ_TYPE_EDGE_RISING>;
• clocks = <&clkc_audio AUD_CLKID_SPDIFIN>,

•  <&clkc_audio AUD_CLKID_SPDIFIN_CLK>;
  

• clock-names = "pclk", "refclk";

-}; diff --git a/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.yaml b/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.yaml new file mode 100644 index 000000000000..4f7e4c21acc1 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/amlogic,axg-spdifin.yaml @@ -0,0 +1,90 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/amlogic,axg-spdifin.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: Amlogic Audio AXG SPDIF Input

+maintainers:

• • Jerome Brunet jbrunet@baylibre.com

+allOf:

• • $ref: dai-common.yaml#

+properties:

• $nodename:
• pattern: "^audio-controller@.*"
• compatible:
• oneOf:

•  - items:
  

All usual comments apply.

•      - const:
  

•          amlogic,axg-spdifin
  

•  - items:
  

•      - enum:
  

•          - amlogic,g12a-spdifin
  

•          - amlogic,sm1-spdifin
  

•      - const:
  

•          amlogic,axg-spdifin
  

All usual comments apply.

• reg:
• maxItems: 1
• "#sound-dai-cells":
• const: 0
• clocks:
• items:

•  - description: Peripheral clock
  

•  - description: SPDIF input reference clock
  

• clock-names:
• items:

•  - const: pclk
  

•  - const: refclk
  

• interrupts:
• maxItems: 1

+required:

• • compatible
• • reg
• • "#sound-dai-cells"
• • clocks
• • clock-names
• • interrupts

+if:

• properties:
• compatible:

•  contains:
  

•    enum:
  

•      - amlogic,g12a-spdifin
  

•      - amlogic,sm1-spdifin
  

+then:

• properties:
• resets:

•  items:
  

•    - description: dedicated device reset line
  

• required:
• • resets

+unevaluatedProperties: false

+examples:

• • |
• #include <dt-bindings/clock/axg-audio-clkc.h>
• #include <dt-bindings/interrupt-controller/irq.h>
• #include <dt-bindings/interrupt-controller/arm-gic.h>
• spdifin: audio-controller@400 {

• compatible = "amlogic,axg-spdifin";
  

Fix indentation.

Best regards, Krzysztof