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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 15653C5478C for ; Tue, 27 Feb 2024 12:38:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=oiGA0xawbp+n96wjHk1tekjbVnW2SSK3DZbPlhYpy+s=; b=apRs6vJAeIcLbp8JdNh9K/BQ55 RYOoPYYaNPOVfTUBu6KnlmqfiwoyYRrSbjz8aSS0BIIMazN9AYCCHhDE96BoijBUFKkr5a/tHrQMj xfvA+E4iSqrs6vium2eJf9Ky/LDel3AzpeL00Tm8lmGB2iaTdfuTlScuylzRo/9hFnJWxTyb4hA/U PmfkczjTTSEi5lAkpvU3AUPSHw8NTkDSAw1dZWxxF/3IIJW1e/zmj+g8JyOWaGan525MSUxmmdJqH 0VW/5YgT3VQsyHK/nWonVE+Y6s9DCiShhb1vltYePpmDXkYgBbsvKzh9ptvwffCzg82O4k1NCEeon yGmOGqrw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rewjO-00000005DfR-0ik8; Tue, 27 Feb 2024 12:38:46 +0000 Received: from madrid.collaboradmins.com ([46.235.227.194]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rewjK-00000005De2-42pM; Tue, 27 Feb 2024 12:38:44 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1709037521; bh=wIFETi/MZvXW9/FAsIozT825+vybcsyNan5jzXcLPQc=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=evMyTP2kJ2qJhKVWh8x49Jwu/aes3ZGEKemzrZ61lFaUHq/IycOxxot4qKk41kzxl cqrVwpwI2NQI8YcQOgzdaf/bXWHrbZjaU1J64If8Ew92UnrwNxcp/T8UsMnbar6tMb 1yT36QOG2Ht1R2By9rJ3rSWkkqsaLdTPVC2VTXqOZ+l+PbPg723HKrlFHzIvcjjtMg TE+w7uKNuh0laf1rPtuaRrPg9XIBZD19vQxaBPoxzvi9cPLcias50byRUGVTT2c6gt stWeZQARKK5MlueSNvTX44aEHsI3GAuCRJpH4qXv3dcqhIMpFJT0AywZDSBymXtvG+ xRKNlrETnAaAw== Received: from [100.113.186.2] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kholk11) by madrid.collaboradmins.com (Postfix) with ESMTPSA id D93BE37813B6; Tue, 27 Feb 2024 12:38:39 +0000 (UTC) Message-ID: Date: Tue, 27 Feb 2024 13:38:39 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 02/18] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card document Content-Language: en-US To: Alexandre Mergnat , Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , Lee Jones , Flora Fu , Jaroslav Kysela , Takashi Iwai , Sumit Semwal , =?UTF-8?Q?Christian_K=C3=B6nig?= , Catalin Marinas , Will Deacon Cc: linux-sound@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org References: <20240226-audio-i350-v1-0-4fa1cea1667f@baylibre.com> <20240226-audio-i350-v1-2-4fa1cea1667f@baylibre.com> <92b9e9ac-6265-4611-888d-ba74bb871be5@baylibre.com> From: AngeloGioacchino Del Regno In-Reply-To: <92b9e9ac-6265-4611-888d-ba74bb871be5@baylibre.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240227_043843_216778_C897A7F0 X-CRM114-Status: GOOD ( 21.40 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 27/02/24 11:23, Alexandre Mergnat ha scritto: > > > On 26/02/2024 16:30, AngeloGioacchino Del Regno wrote: >> Il 26/02/24 15:01, Alexandre Mergnat ha scritto: >>> Add soundcard bindings for the MT8365 SoC with the MT6357 audio codec. >>> >>> Signed-off-by: Alexandre Mergnat >>> --- >>>   .../bindings/sound/mediatek,mt8365-mt6357.yaml     | 127 +++++++++++++++++++++ >>>   1 file changed, 127 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/sound/mediatek,mt8365-mt6357.yaml >>> b/Documentation/devicetree/bindings/sound/mediatek,mt8365-mt6357.yaml >>> new file mode 100644 >>> index 000000000000..f469611ec6b6 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/sound/mediatek,mt8365-mt6357.yaml >>> @@ -0,0 +1,127 @@ >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/sound/mediatek,mt8365-mt6357.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: Mediatek MT8365 sound card with MT6357 sound codec. >>> + >>> +maintainers: >>> +  - Alexandre Mergnat >>> + >>> +description: >>> +  This binding describes the MT8365 sound card. >>> + >>> +properties: >>> +  compatible: >>> +    const: mediatek,mt8365-mt6357 >>> + >>> +  mediatek,hp-pull-down: >>> +    description: >>> +      Earphone driver positive output stage short to the >>> +      audio reference ground. >>> +      Default value is false. >>> +    type: boolean >>> + >>> +  mediatek,micbias0-microvolt: >>> +    description: | >> >> description: Selects MIC Bias 0 output voltage >> >>> +      Selects MIC Bias 0 output voltage. >>> +      [1.7v, 1.8v, 1.9v, 2.0v, 2.1v, 2.5v, 2.6v, 2.7v] >>> +    enum: [0, 1, 2, 3, 4, 5, 6, 7] >> >> No, you don't say 0 1 2 3 4 to a property that says "microvolt", that's simply >> wrong. >> >> mediatek,micbias0-microvolt = <2100000>; >> >> ...so you want a binding that says >> enum: [ 1700000, 1800000, this, that, 2700000] >> > > Is it correct if I put "description: Selects MIC Bias 0 output voltage index" ? > No, it's still not correct. You have to pass microvolt values. The driver will then transform the microvolt values to the index and subsequently write the index value to the hardware registers. The bindings shall be generic, in the sense that they shall not express hardware register values... and this is especially true when we have a value that *does* actually have means to be expressed in common units. Besides, in the cases in which there's no common units involved, the values most probably won't be suited for devicetree//bindings, so those would be hardcoded in the driver as platform data. This is not the case, so, please keep this property but specify microvolts in the bindings (and obviously in devicetree). >>> + >>> +  mediatek,micbias1-microvolt: >>> +    description: | >>> +      Selects MIC Bias 1 output voltage. >>> +      [1.7v, 1.8v, 1.9v, 2.0v, 2.1v, 2.5v, 2.6v, 2.7v] >>> +    enum: [0, 1, 2, 3, 4, 5, 6, 7] >> >> same here. >> >>> + >>> +  mediatek,platform: >>> +    $ref: /schemas/types.yaml#/definitions/phandle >>> +    description: The phandle of MT8365 ASoC platform. >>> + >>> +  pinctrl-names: >>> +    minItems: 1 >>> +    items: >>> +      - const: aud_default >>> +      - const: aud_dmic >>> +      - const: aud_miso_off >>> +      - const: aud_miso_on >>> +      - const: aud_mosi_off >>> +      - const: aud_mosi_on >>> + >>> +  vaud28-supply: >>> +    description: >>> +      2.8 volt supply for the audio codec >>> + >>> +patternProperties: >>> +  "^dai-link-[0-9]+$": >>> +    type: object >>> +    description: >>> +      Container for dai-link level properties and CODEC sub-nodes. >>> + >>> +    properties: >>> +      codec: >>> +        type: object >>> +        description: Holds subnode which indicates codec dai. >>> + >>> +        properties: >>> +          sound-dai: >>> +            maxItems: 1 >>> +            description: phandle of the codec DAI >>> + >>> +        additionalProperties: false >>> + >>> +      link-name: >>> +        description: >>> +          This property corresponds to the name of the BE dai-link to which >>> +          we are going to update parameters in this node. >>> +        items: >>> +          const: 2ND I2S BE >>> + >>> +      sound-dai: >>> +        maxItems: 1 >>> +        description: phandle of the CPU DAI >>> + >>> +    additionalProperties: false >>> + >>> +    required: >>> +      - link-name >>> +      - sound-dai >>> + >>> +additionalProperties: false >>> + >>> +required: >>> +  - compatible >>> +  - mediatek,platform >>> +  - pinctrl-names >>> +  - vaud28-supply >>> + >>> +examples: >>> +  - | >>> +    sound { >>> +        compatible = "mediatek,mt8365-mt6357"; >>> +        mediatek,platform = <&afe>; >> >> Please: >> >> https://docs.kernel.org/devicetree/bindings/dts-coding-style.html > > Is it about the wrong pinctrl-names tab alignment ? > Also, 2ND I2S BE => 2ND_I2S_BE ? > Otherwise, I don't get it sorry. > ...as Krzysztof already clarified, won't repeat :-P Cheers! >> >> Regards, >> Angelo >> >>> +        pinctrl-names = "aud_default", >>> +            "aud_dmic", >>> +            "aud_miso_off", >>> +            "aud_miso_on", >>> +            "aud_mosi_off", >>> +            "aud_mosi_on"; >>> +        pinctrl-0 = <&aud_default_pins>; >>> +        pinctrl-1 = <&aud_dmic_pins>; >>> +        pinctrl-2 = <&aud_miso_off_pins>; >>> +        pinctrl-3 = <&aud_miso_on_pins>; >>> +        pinctrl-4 = <&aud_mosi_off_pins>; >>> +        pinctrl-5 = <&aud_mosi_on_pins>; >>> +        vaud28-supply = <&mt6357_vaud28_reg>; >>> + >>> +        /* hdmi interface */ >>> +        dai-link-0 { >>> +            sound-dai = <&afe>; >>> +            link-name = "2ND I2S BE"; >>> + >>> +            codec { >>> +                sound-dai = <&it66121hdmitx>; >>> +            }; >>> +        }; >>> +    }; >>> >> >