Re: [PATCH v3 2/4] ASoC: dt-bindings: Add Airoha AN7581 AFE with WM8960 Codec schema

[Krzysztof Kozlowski]

On Thu, May 28, 2026 at 07:48:30PM +0200, Christian Marangi wrote:
> Add YAML schema for Airoha AN7581 AFE with the specific WM8960 i2c Codec.

DT schema. There is no YAML schema.

Also, looks like sound card, not AFE... but previous patch said it is
a sound card (while it was AFE), so completely confusing...

> 
> This gives example on how to define and connect the AFE driver with the
> WM9860 for full functionality.
> 
> Signed-off-by: Christian Marangi <ansuelsmth@xxxxxxxxx>
> ---
>  .../bindings/sound/airoha,an7581-wm8960.yaml  | 71 +++++++++++++++++++
>  1 file changed, 71 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/airoha,an7581-wm8960.yaml
> 
> diff --git a/Documentation/devicetree/bindings/sound/airoha,an7581-wm8960.yaml b/Documentation/devicetree/bindings/sound/airoha,an7581-wm8960.yaml
> new file mode 100644
> index 000000000000..b637c294657f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/sound/airoha,an7581-wm8960.yaml
> @@ -0,0 +1,71 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/sound/airoha,an7581-wm8960.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Airoha AN7581 sound card with WM8960 codec
> +
> +maintainers:
> +  - Christian Marangi <ansuelsmth@xxxxxxxxx>
> +
> +allOf:
> +  - $ref: sound-card-common.yaml#
> +
> +properties:
> +  compatible:
> +    const: airoha,an7581-wm8960-sound
> +
> +  platform:
> +    type: object
> +

additionalProperties go here. Looks at other Mediatek sound cards.

> +    properties:
> +      sound-dai:
> +        items:
> +          - description: The phandle of AN7581 platform.
> +
> +    required:
> +      - sound-dai
> +
> +    additionalProperties: false
> +
> +  codec:
> +    type: object
> +
> +    properties:
> +      sound-dai:
> +        items:
> +          - description: The phandle of WM8960 i2c codec.
> +
> +    required:
> +      - sound-dai
> +
> +    additionalProperties: false
> +
> +unevaluatedProperties: false

This goes after "required:".


> +
> +required:
> +  - compatible

Best regards,
Krzysztof