From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8C3E4C54EE9 for ; Wed, 7 Sep 2022 17:59:03 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id C669D1682; Wed, 7 Sep 2022 19:58:10 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz C669D1682 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1662573540; bh=cm7A7ug8aaFFojJwO39wmSq6lnarpOxZFUxtfHynspI=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=ur4fHvedt3wvxEnrd7U9IrciVryKieDFRq0k9LzOe5S4l8iA6/3qtYkvKzojp3f3y We14rnNPal7hxh3IGAogyg2WC0DReAQtmdthRRLiihCEUlvmyMODerLSGcoPXm2f3c E+X1n1e/53xQbmJ7ek0lbRpjZr/4f75eKyTLWJfw= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 65E6CF80249; Wed, 7 Sep 2022 19:58:10 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id A228DF8028D; Wed, 7 Sep 2022 19:58:08 +0200 (CEST) Received: from mail-oa1-f53.google.com (mail-oa1-f53.google.com [209.85.160.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 830A1F80118 for ; Wed, 7 Sep 2022 19:58:02 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 830A1F80118 Received: by mail-oa1-f53.google.com with SMTP id 586e51a60fabf-1279948d93dso18136650fac.10 for ; Wed, 07 Sep 2022 10:58:02 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date; bh=UdQDfJDRdgGrPGjr0rQoSFWlcm9yHV7FNhW1rklMkE0=; b=EV9PYHd/J5l+Ni6rLi16rVAH5TjIBQzg9wda6sGki8er0mSG/qUAAOayUt/DxD/3oI vtoEQzQaVU8TS1moUvLEnhkReJaiTKHwg5mmJoSuCFQVVgtNz6chCHaYNMHlwOE5eUUx Soc5aZkC99MYORsViZxbS4CQFL050o23gGf0dV865BB4/nGc865l7LcaCh8qS5SFNzq1 jJApNuFBUPsGcoR+yZTsoQcY3/xm1gAqZqZXVYKY87Ca691xccx1Y82H83vgPaKdisOz M3fKLVQw4Rd7RcRRx7s1wyH6aJ/8KnWitEwy/ni+fSpMfQ5a5ITZBLTTu8rE/CXOblha RJpg== X-Gm-Message-State: ACgBeo2hc1g73U0LZsj56xUaTknry846UkfUldKd4oi/ayqqzj2HH0Te MhPa0z++gwv0IbXhcHSjKA== X-Google-Smtp-Source: AA6agR6xKrGz+BcPw2Zi9fBkKG87AYCGD492SkFstQc2YuZbmPjB1jUx7jBedulSPqAHkVLIY16gbQ== X-Received: by 2002:a05:6808:e90:b0:345:49f2:a112 with SMTP id k16-20020a0568080e9000b0034549f2a112mr12928078oil.7.1662573480405; Wed, 07 Sep 2022 10:58:00 -0700 (PDT) Received: from robh.at.kernel.org (66-90-144-107.dyn.grandenetworks.net. [66.90.144.107]) by smtp.gmail.com with ESMTPSA id l3-20020a056870f14300b0012752d3212fsm4943445oac.53.2022.09.07.10.57.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Sep 2022 10:58:00 -0700 (PDT) Received: (nullmailer pid 3798414 invoked by uid 1000); Wed, 07 Sep 2022 17:57:59 -0000 Date: Wed, 7 Sep 2022 12:57:59 -0500 From: Rob Herring To: Krzysztof Kozlowski Subject: Re: [PATCH v2 09/14] ASoC: dt-bindings:: qcom,q6asm: convert to dtschema Message-ID: <20220907175759.GA3791463-robh@kernel.org> References: <20220907101556.37394-1-krzysztof.kozlowski@linaro.org> <20220907101556.37394-10-krzysztof.kozlowski@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220907101556.37394-10-krzysztof.kozlowski@linaro.org> Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, linux-arm-msm@vger.kernel.org, Bjorn Andersson , Konrad Dybcio , Liam Girdwood , linux-kernel@vger.kernel.org, Andy Gross , Srinivas Kandagatla , Mark Brown , Krzysztof Kozlowski X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Wed, Sep 07, 2022 at 12:15:51PM +0200, Krzysztof Kozlowski wrote: > Convert Qualcomm Audio Stream Manager (Q6ASM) bindings to DT schema. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Expected warning because the qcom,apr.yaml is being fixed in next commit: > > Documentation/devicetree/bindings/sound/qcom,q6asm.example.dtb: apr: service@7: 'dais' does not match any of the regexes: '^.*@[0-9a-f]+$', 'pinctrl-[0-9]+' > From schema: /home/krzk/dev/linux/linux/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml > > Changes since v1: > 1. New patch. > --- > .../devicetree/bindings/sound/qcom,q6asm.txt | 70 ----------- > .../devicetree/bindings/sound/qcom,q6asm.yaml | 112 ++++++++++++++++++ > 2 files changed, 112 insertions(+), 70 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/sound/qcom,q6asm.txt > create mode 100644 Documentation/devicetree/bindings/sound/qcom,q6asm.yaml Should be qcom,q6asm-dais.yaml or you need to add qcom,q6asm compatible? > > diff --git a/Documentation/devicetree/bindings/sound/qcom,q6asm.txt b/Documentation/devicetree/bindings/sound/qcom,q6asm.txt > deleted file mode 100644 > index 0d0075125243..000000000000 > --- a/Documentation/devicetree/bindings/sound/qcom,q6asm.txt > +++ /dev/null > @@ -1,70 +0,0 @@ > -Qualcomm Audio Stream Manager (Q6ASM) binding > - > -Q6ASM is one of the APR audio service on Q6DSP. > -Please refer to qcom,apr.txt for details of the common apr service bindings > -used by the apr service device. > - > -- but must contain the following property: > - > -- compatible: > - Usage: required > - Value type: > - Definition: must be "qcom,q6asm-v.". > - Or "qcom,q6asm" where the version number can be queried > - from DSP. > - example "qcom,q6asm-v2.0" Where is this compatible handled now? > - > -= ASM DAIs (Digital Audio Interface) > -"dais" subnode of the ASM node represents dai specific configuration > - > -- compatible: > - Usage: required > - Value type: > - Definition: must be "qcom,q6asm-dais". > - > -- #sound-dai-cells > - Usage: required > - Value type: > - Definition: Must be 1 > - > -== ASM DAI is subnode of "dais" and represent a dai, it includes board specific > -configuration of each dai. Must contain the following properties. > - > -- reg > - Usage: required > - Value type: > - Definition: Must be dai id > - > -- direction: > - Usage: Required for Compress offload dais > - Value type: > - Definition: Specifies the direction of the dai stream > - Q6ASM_DAI_TX_RX (0) for both tx and rx > - Q6ASM_DAI_TX (1) for only tx (Capture/Encode) > - Q6ASM_DAI_RX (2) for only rx (Playback/Decode) > - > -- is-compress-dai: > - Usage: Required for Compress offload dais > - Value type: > - Definition: present for Compress offload dais > - > - > -= EXAMPLE > -#include > - > -apr-service@7 { > - compatible = "qcom,q6asm"; > - reg = ; > - q6asmdai: dais { > - compatible = "qcom,q6asm-dais"; > - #address-cells = <1>; > - #size-cells = <0>; > - #sound-dai-cells = <1>; > - > - dai@0 { > - reg = <0>; > - direction = ; > - is-compress-dai; > - }; > - }; > -}; > diff --git a/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml b/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml > new file mode 100644 > index 000000000000..505bd64a0717 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml > @@ -0,0 +1,112 @@ > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sound/qcom,q6asm.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Audio Stream Manager (Q6ASM) > + > +maintainers: > + - Krzysztof Kozlowski > + - Srinivas Kandagatla > + > +description: > + Q6ASM is one of the APR audio services on Q6DSP. Each of its subnodes > + represent a dai with board specific configuration. > + > +properties: > + compatible: > + enum: > + - qcom,q6asm-dais > + > + iommus: > + maxItems: 1 > + > + '#sound-dai-cells': > + const: 1 > + > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > +patternProperties: > + '^dai@[0-9]+$': > + type: object > + description: > + Q6ASM Digital Audio Interface > + > + properties: > + reg: > + maxItems: 1 > + > + direction: > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [0, 1, 2] > + description: | > + The direction of the dai stream:: > + - Q6ASM_DAI_TX_RX (0) for both tx and rx > + - Q6ASM_DAI_TX (1) for only tx (Capture/Encode) > + - Q6ASM_DAI_RX (2) for only rx (Playback/Decode) > + > + is-compress-dai: > + type: boolean > + description: > + Compress offload dai. > + > + dependencies: > + is-compress-dai: ['direction'] > + > + required: > + - reg > + > + additionalProperties: false > + > +required: > + - compatible > + - "#sound-dai-cells" > + - "#address-cells" > + - "#size-cells" > + > +additionalProperties: false > + > +examples: > + - | > + #include > + #include > + > + apr { > + compatible = "qcom,apr-v2"; > + qcom,domain = ; > + #address-cells = <1>; > + #size-cells = <0>; > + > + service@7 { > + compatible = "qcom,q6asm"; > + reg = ; > + qcom,protection-domain = "avs/audio", "msm/adsp/audio_pd"; > + > + dais { > + compatible = "qcom,q6asm-dais"; > + iommus = <&apps_smmu 0x1821 0x0>; > + #address-cells = <1>; > + #size-cells = <0>; > + #sound-dai-cells = <1>; > + > + dai@0 { > + reg = <0>; > + }; > + > + dai@1 { > + reg = <1>; > + }; > + > + dai@2 { > + reg = <2>; > + is-compress-dai; > + direction = <1>; > + }; > + }; > + }; > + }; > -- > 2.34.1 > > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9CD6BC38145 for ; Wed, 7 Sep 2022 17:58:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230107AbiIGR6N (ORCPT ); Wed, 7 Sep 2022 13:58:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42768 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230116AbiIGR6C (ORCPT ); Wed, 7 Sep 2022 13:58:02 -0400 Received: from mail-oa1-f42.google.com (mail-oa1-f42.google.com [209.85.160.42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3C222AA4DA; Wed, 7 Sep 2022 10:58:01 -0700 (PDT) Received: by mail-oa1-f42.google.com with SMTP id 586e51a60fabf-11e9a7135easo37949859fac.6; Wed, 07 Sep 2022 10:58:01 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date; bh=UdQDfJDRdgGrPGjr0rQoSFWlcm9yHV7FNhW1rklMkE0=; b=zs+rUhahmEXicw/Ubu+iuq+okCF//247nzn5p+pSTJql9nx7pDcUL88ZCJeovy8z0F HtoKp6vObUuRpP1TezsaZyo0ddAi0Yt8pm5wLBKrMMGIDEGmt0NLHZn8BndW2CgSFSnl S6KxVgd6PEB4i6H28D0T65FxzruJIti+tmRyAmV32+GnsEzzXESiceZvjSJpZCOai8xE OQYCZUaNJaLrRvEbXn+XeAAd6++n2pMnc7eNlwWBBzXBfZ/B1Es+mravJ0IWIV0xRpLb WpyBYa6/hFQm10teO7pxR4Fr4VK41/+qXi03WwtjpJojUFTmu1gtU9VWudo6Wyt+Hp6X gyfw== X-Gm-Message-State: ACgBeo1eR1eligW33qMttYoQZwa9DK2JehVEPmNKsrQwgvXP2McwD8xz fR1FrQQ9MOUS1RyUtvKzxw== X-Google-Smtp-Source: AA6agR6xKrGz+BcPw2Zi9fBkKG87AYCGD492SkFstQc2YuZbmPjB1jUx7jBedulSPqAHkVLIY16gbQ== X-Received: by 2002:a05:6808:e90:b0:345:49f2:a112 with SMTP id k16-20020a0568080e9000b0034549f2a112mr12928078oil.7.1662573480405; Wed, 07 Sep 2022 10:58:00 -0700 (PDT) Received: from robh.at.kernel.org (66-90-144-107.dyn.grandenetworks.net. [66.90.144.107]) by smtp.gmail.com with ESMTPSA id l3-20020a056870f14300b0012752d3212fsm4943445oac.53.2022.09.07.10.57.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Sep 2022 10:58:00 -0700 (PDT) Received: (nullmailer pid 3798414 invoked by uid 1000); Wed, 07 Sep 2022 17:57:59 -0000 Date: Wed, 7 Sep 2022 12:57:59 -0500 From: Rob Herring To: Krzysztof Kozlowski Cc: Andy Gross , Bjorn Andersson , Konrad Dybcio , Krzysztof Kozlowski , Liam Girdwood , Mark Brown , Srinivas Kandagatla , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH v2 09/14] ASoC: dt-bindings:: qcom,q6asm: convert to dtschema Message-ID: <20220907175759.GA3791463-robh@kernel.org> References: <20220907101556.37394-1-krzysztof.kozlowski@linaro.org> <20220907101556.37394-10-krzysztof.kozlowski@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220907101556.37394-10-krzysztof.kozlowski@linaro.org> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Wed, Sep 07, 2022 at 12:15:51PM +0200, Krzysztof Kozlowski wrote: > Convert Qualcomm Audio Stream Manager (Q6ASM) bindings to DT schema. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Expected warning because the qcom,apr.yaml is being fixed in next commit: > > Documentation/devicetree/bindings/sound/qcom,q6asm.example.dtb: apr: service@7: 'dais' does not match any of the regexes: '^.*@[0-9a-f]+$', 'pinctrl-[0-9]+' > From schema: /home/krzk/dev/linux/linux/Documentation/devicetree/bindings/soc/qcom/qcom,apr.yaml > > Changes since v1: > 1. New patch. > --- > .../devicetree/bindings/sound/qcom,q6asm.txt | 70 ----------- > .../devicetree/bindings/sound/qcom,q6asm.yaml | 112 ++++++++++++++++++ > 2 files changed, 112 insertions(+), 70 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/sound/qcom,q6asm.txt > create mode 100644 Documentation/devicetree/bindings/sound/qcom,q6asm.yaml Should be qcom,q6asm-dais.yaml or you need to add qcom,q6asm compatible? > > diff --git a/Documentation/devicetree/bindings/sound/qcom,q6asm.txt b/Documentation/devicetree/bindings/sound/qcom,q6asm.txt > deleted file mode 100644 > index 0d0075125243..000000000000 > --- a/Documentation/devicetree/bindings/sound/qcom,q6asm.txt > +++ /dev/null > @@ -1,70 +0,0 @@ > -Qualcomm Audio Stream Manager (Q6ASM) binding > - > -Q6ASM is one of the APR audio service on Q6DSP. > -Please refer to qcom,apr.txt for details of the common apr service bindings > -used by the apr service device. > - > -- but must contain the following property: > - > -- compatible: > - Usage: required > - Value type: > - Definition: must be "qcom,q6asm-v.". > - Or "qcom,q6asm" where the version number can be queried > - from DSP. > - example "qcom,q6asm-v2.0" Where is this compatible handled now? > - > -= ASM DAIs (Digital Audio Interface) > -"dais" subnode of the ASM node represents dai specific configuration > - > -- compatible: > - Usage: required > - Value type: > - Definition: must be "qcom,q6asm-dais". > - > -- #sound-dai-cells > - Usage: required > - Value type: > - Definition: Must be 1 > - > -== ASM DAI is subnode of "dais" and represent a dai, it includes board specific > -configuration of each dai. Must contain the following properties. > - > -- reg > - Usage: required > - Value type: > - Definition: Must be dai id > - > -- direction: > - Usage: Required for Compress offload dais > - Value type: > - Definition: Specifies the direction of the dai stream > - Q6ASM_DAI_TX_RX (0) for both tx and rx > - Q6ASM_DAI_TX (1) for only tx (Capture/Encode) > - Q6ASM_DAI_RX (2) for only rx (Playback/Decode) > - > -- is-compress-dai: > - Usage: Required for Compress offload dais > - Value type: > - Definition: present for Compress offload dais > - > - > -= EXAMPLE > -#include > - > -apr-service@7 { > - compatible = "qcom,q6asm"; > - reg = ; > - q6asmdai: dais { > - compatible = "qcom,q6asm-dais"; > - #address-cells = <1>; > - #size-cells = <0>; > - #sound-dai-cells = <1>; > - > - dai@0 { > - reg = <0>; > - direction = ; > - is-compress-dai; > - }; > - }; > -}; > diff --git a/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml b/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml > new file mode 100644 > index 000000000000..505bd64a0717 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/qcom,q6asm.yaml > @@ -0,0 +1,112 @@ > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sound/qcom,q6asm.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Audio Stream Manager (Q6ASM) > + > +maintainers: > + - Krzysztof Kozlowski > + - Srinivas Kandagatla > + > +description: > + Q6ASM is one of the APR audio services on Q6DSP. Each of its subnodes > + represent a dai with board specific configuration. > + > +properties: > + compatible: > + enum: > + - qcom,q6asm-dais > + > + iommus: > + maxItems: 1 > + > + '#sound-dai-cells': > + const: 1 > + > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > +patternProperties: > + '^dai@[0-9]+$': > + type: object > + description: > + Q6ASM Digital Audio Interface > + > + properties: > + reg: > + maxItems: 1 > + > + direction: > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [0, 1, 2] > + description: | > + The direction of the dai stream:: > + - Q6ASM_DAI_TX_RX (0) for both tx and rx > + - Q6ASM_DAI_TX (1) for only tx (Capture/Encode) > + - Q6ASM_DAI_RX (2) for only rx (Playback/Decode) > + > + is-compress-dai: > + type: boolean > + description: > + Compress offload dai. > + > + dependencies: > + is-compress-dai: ['direction'] > + > + required: > + - reg > + > + additionalProperties: false > + > +required: > + - compatible > + - "#sound-dai-cells" > + - "#address-cells" > + - "#size-cells" > + > +additionalProperties: false > + > +examples: > + - | > + #include > + #include > + > + apr { > + compatible = "qcom,apr-v2"; > + qcom,domain = ; > + #address-cells = <1>; > + #size-cells = <0>; > + > + service@7 { > + compatible = "qcom,q6asm"; > + reg = ; > + qcom,protection-domain = "avs/audio", "msm/adsp/audio_pd"; > + > + dais { > + compatible = "qcom,q6asm-dais"; > + iommus = <&apps_smmu 0x1821 0x0>; > + #address-cells = <1>; > + #size-cells = <0>; > + #sound-dai-cells = <1>; > + > + dai@0 { > + reg = <0>; > + }; > + > + dai@1 { > + reg = <1>; > + }; > + > + dai@2 { > + reg = <2>; > + is-compress-dai; > + direction = <1>; > + }; > + }; > + }; > + }; > -- > 2.34.1 > >