Subject: Re: [PATCH v2] dt-bindings: sound: nvidia,tegra-audio: Convert
 multiple txt bindings to yaml
To:     David Heidelberg <david@ixit.cz>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Mark Brown <broonie@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Jonathan Hunter <jonathanh@nvidia.com>
Cc:     ~okias/devicetree@lists.sr.ht, alsa-devel@alsa-project.org,
        devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20211211224946.79875-1-david@ixit.cz>
From:   Dmitry Osipenko <digetx@gmail.com>
Message-ID: <a84213cb-272a-f1b2-338f-ed8ed0bf133d@gmail.com>
Date:   Thu, 16 Dec 2021 05:52:12 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
MIME-Version: 1.0
In-Reply-To: <20211211224946.79875-1-david@ixit.cz>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Precedence: bulk

12.12.2021 01:49, David Heidelberg пишет:
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    sound {
> +        compatible = "nvidia,tegra-audio-rt5677-ryu",
> +                     "nvidia,tegra-audio-rt5677";
> +        nvidia,model = "NVIDIA Tegra Ryu";
> +
> +        nvidia,audio-routing =
> +                "Headphone", "LOUT2",
> +                "Headphone", "LOUT1",
> +                "Headset Mic", "MICBIAS1",
> +                "IN1P", "Headset Mic",
> +                "IN1N", "Headset Mic",
> +                "DMIC L1", "Internal Mic 1",
> +                "DMIC R1", "Internal Mic 1",
> +                "DMIC L2", "Internal Mic 2",
> +                "DMIC R2", "Internal Mic 2",
> +                "Speaker", "PDM1L",
> +                "Speaker", "PDM1R";
> +
> +        nvidia,i2s-controller = <&tegra_i2s1>;
> +        nvidia,audio-codec = <&rt5677>;
> +
> +        nvidia,hp-det-gpios = <&gpio 143 0>;
> +        nvidia,mic-present-gpios = <&gpio 132 1>;
> +        nvidia,hp-en-gpios = <&rt5677 1 0>;
> +        nvidia,dmic-clk-en-gpios = <&rt5677 2 1>;

I spotted that nvidia,dmic-clk-en-gpios is undocumented, but DTs and
binding are passing the validation. We will make another patch to fix it.

Rob, could you please tell whether this is because unevaluatedProperties
doesn't work yet or we're missing something?