Re: [PATCH v3 1/9] ASoC: dt-bindings: qcom,qaif-cpu: Add binding

[Manuel Ebner <manuelebnerli@mailbox.org>]

Hi

On Mon, 2026-08-24 at 12:06 +0530, Harendra Gautam wrote:
> QAIF routes audio between system memory, external PCM/TDM/MI2S serial
> interfaces and the internal Bolero/WCD codec. Platform Devicetree files
> need a binding to describe the controller resources, DAI cells and per-
> interface AIF configuration, and named constants to avoid raw numeric IDs
> in sound-dai references and child-node reg values.
> 
> Describe the MMIO region, interrupt, clocks, DMA IOMMU mapping and
> aif-interface child nodes required by the controller.
> 
> Signed-off-by: Harendra Gautam <harendra.gautam@oss.qualcomm.com>
> ---
>  .../bindings/sound/qcom,qaif-cpu.yaml         | 307 ++++++++++++++++++
>  include/dt-bindings/sound/qcom,qaif.h         |  97 ++++++
>  2 files changed, 404 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/qcom,qaif-cpu.yaml
>  create mode 100644 include/dt-bindings/sound/qcom,qaif.h
> 
> diff --git a/Documentation/devicetree/bindings/sound/qcom,qaif-cpu.yaml b/Documentation/devicetree/bindings/sound/qcom,qaif-cpu.yaml
> new file mode 100644
> index 000000000000..134356114733
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/sound/qcom,qaif-cpu.yaml
> @@ -0,0 +1,307 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/sound/qcom,qaif-cpu.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#

Both of the links redirect to https://www.devicetree.org/

> 
> ...
> 
> diff --git a/include/dt-bindings/sound/qcom,qaif.h b/include/dt-bindings/sound/qcom,qaif.h
> new file mode 100644
> index 000000000000..24706efd07d1
> --- /dev/null
> +++ b/include/dt-bindings/sound/qcom,qaif.h
> @@ -0,0 +1,97 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + *
> + * DAI IDs for the Qualcomm Audio Interface (QAIF) controller.
> + * QAIF_MI2S_AIF* and QAIF_TDM_AIF* values are used in sound-dai
> + * references and as the reg value of aif@N child nodes.
> + * QAIF_CDC_DMA_* values are for sound-dai references only and must
> + * not be used as aif@N child node reg values.
> + */
> +#ifndef __DT_QCOM_QAIF_H
> +#define __DT_QCOM_QAIF_H
> +
> +/*
> + * MI2S DAI IDs -- one per physical AIF port in MI2S mode.
> + * Each port supports up to 2 channels (stereo I2S) over a single
> + * data lane sharing a bit clock and frame sync.
> + */
> +#define QAIF_MI2S_AIF0		200
> +#define QAIF_MI2S_AIF1		201
> +#define QAIF_MI2S_AIF2		202
> +#define QAIF_MI2S_AIF3		203
> +#define QAIF_MI2S_AIF4		204
> +#define QAIF_MI2S_AIF5		205
> +#define QAIF_MI2S_AIF6		206
> +#define QAIF_MI2S_AIF7		207
> +#define QAIF_MI2S_AIF8		208
> +#define QAIF_MI2S_AIF9		209
> +#define QAIF_MI2S_AIF10		210
> +#define QAIF_MI2S_AIF11		211
> +#define QAIF_MI2S_AIF12		212
> +
> +/*
> + * TDM DAI IDs -- one per physical AIF port in TDM mode.

Is this sentence complete?

> + * Each port supports up to 8 channels over up to 8 independent data
> + * lanes sharing a single bit clock and frame sync.
> + */
> 
> ...

Thanks
 Manuel