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 0293CECAAA1 for ; Mon, 24 Oct 2022 17:54:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229696AbiJXRyJ (ORCPT ); Mon, 24 Oct 2022 13:54:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55038 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233770AbiJXRxq (ORCPT ); Mon, 24 Oct 2022 13:53:46 -0400 Received: from mail-qk1-f175.google.com (mail-qk1-f175.google.com [209.85.222.175]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 37D6793794 for ; Mon, 24 Oct 2022 09:32:20 -0700 (PDT) Received: by mail-qk1-f175.google.com with SMTP id f8so6396476qkg.3 for ; Mon, 24 Oct 2022 09:32:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=lGJrVQPc2u8BmXJHhbTLdghe4GhptaKKpCGXInaxYBc=; b=mN9JEcfA/k4mMvL/hZaY6pEx5NHRiDoUSSXyXc+7ktUksysoXmwb7A6j3c0J4sBPi4 royZIXe/7FbOVh+Zgynooe8CQ6KGcnTMryluWGp/CC7+Jo2d71hU3E9h0GVnOqB/73pM 01WzsLuqJqxdiWhX4TwvucbzFKYn/XKnzc1zj5xd5yNc+6bfcutHdo/ecPMD3+lnY9kE ntJp9Unx/qmeWLKaVGGdjj5fqj21OBcVJTOmLNpHZbzvmC+++I2G8TVFA6fA8YOhKjHO vJHvmuMw/P/QZeMQ9cwuHJlk4u26ZZ03c0wwxvS4tI5T6mauuDgA2sEB+KXEOybpwQ3r 71Ew== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=lGJrVQPc2u8BmXJHhbTLdghe4GhptaKKpCGXInaxYBc=; b=NjOYXiVQcSXNQEwdYHhY5HEux03ZC4w6mpNEsgniyhUiUHH5wY607KCncipB5cqrBz S92ScGZ1nuq93eykd0syxxO1Vyru2CxaZj33DQs0vcRZknW4sycnnTiI0M55PRukpiEO uZSRUGSp8uL2+Ome458fY20w+eHef+/WZXQx2Ej9PwxUllsI0dlNyQhJAM/ohJbUuKiO JYu7+oGaVxOdlFgIgdNnD6Y8LavgUOhEjoXsFvQLABauARFd4wJl1dWsjqQehjaFEpsh pUToq3e9wjTwTuj8Ts4FVVXgnsHwJo76SHYpgmWIibri4F8WKXEqcgWxZ4flbSb2QO6h nsSA== X-Gm-Message-State: ACrzQf2PdTJILHB8ja8pQFYNt/cFRmYt3AjsK1QGq8fPSAVSWcxVX2jM LjHgZ3otlck89mXTtkHHy8cHfg== X-Google-Smtp-Source: AMsMyM6nNKEGA5WXpAqyAoIBFJfkWQ0wIqWX2Rnnj9/gvhCBUBxmQ8Wb0wGieNxJDAU17FPd6rMxeQ== X-Received: by 2002:a05:620a:858:b0:6ea:8a17:19fd with SMTP id u24-20020a05620a085800b006ea8a1719fdmr23874112qku.397.1666628955532; Mon, 24 Oct 2022 09:29:15 -0700 (PDT) Received: from [192.168.1.8] ([64.57.193.93]) by smtp.gmail.com with ESMTPSA id h7-20020a05620a284700b006ea7f9d8644sm201040qkp.96.2022.10.24.09.29.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 24 Oct 2022 09:29:14 -0700 (PDT) Message-ID: <1f56ff1d-4b59-97f9-60d0-12197100ae4a@linaro.org> Date: Mon, 24 Oct 2022 12:29:12 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.0 Subject: Re: [PATCH 3/3] ASoC: dt-bindings: rt5682: Add AVDD, MICVDD and VBAT supplies Content-Language: en-US To: =?UTF-8?B?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= Cc: Mark Brown , AngeloGioacchino Del Regno , kernel@collabora.com, Krzysztof Kozlowski , Liam Girdwood , Rob Herring , alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20221021190908.1502026-1-nfraprado@collabora.com> <20221021190908.1502026-4-nfraprado@collabora.com> <20221024160029.cfpyp2cptwig27jd@notapiano> From: Krzysztof Kozlowski In-Reply-To: <20221024160029.cfpyp2cptwig27jd@notapiano> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 24/10/2022 12:00, Nícolas F. R. A. Prado wrote: > On Sat, Oct 22, 2022 at 12:41:01PM -0400, Krzysztof Kozlowski wrote: >> On 21/10/2022 15:09, Nícolas F. R. A. Prado wrote: >>> The rt5682 codec can have three supplies: AVDD, MICVDD and VBAT. Add >>> properties for them. >>> >>> Signed-off-by: Nícolas F. R. A. Prado >>> --- >>> >>> Documentation/devicetree/bindings/sound/rt5682.txt | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/sound/rt5682.txt b/Documentation/devicetree/bindings/sound/rt5682.txt >>> index c5f2b8febcee..5ccf4eaf12a9 100644 >>> --- a/Documentation/devicetree/bindings/sound/rt5682.txt >>> +++ b/Documentation/devicetree/bindings/sound/rt5682.txt >>> @@ -48,6 +48,12 @@ Optional properties: >>> >>> - #sound-dai-cells: Should be set to '<0>'. >>> >>> +- AVDD-supply: phandle to the regulator supplying AVDD >>> + >>> +- MICVDD-supply: phandle to the regulator supplying MICVDD >>> + >>> +- VBAT-supply: phandle to the regulator supplying VBAT >> >> Lowercase. > > Actually looks like there's already a DT using these properties before the > binding was added: > > arm64/boot/dts/qcom/sc7180-trogdor.dtsi > > In this case should we keep them uppercase for compatibility or carry on with > the name changes and also update the DT? (the driver also uses uppercase names) Driver seems to use them as well, then uppercase is fine. But you need to describe it in commit msg, that you document existing usage, not adding new properties. Best regards, Krzysztof