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 763DEC77B73 for ; Tue, 2 May 2023 06:35:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kO7A9IS+TFCY0EkTSJb8x6ItQhCXCViCb2eo9nRzeIk=; b=NH87laHagJdrG4 yy+MVsQXb1VkQ3mwvLpfkaIgJiZsPCrNyZXWfbYrNwnCfihrJAqPPKyCnCOD7KMd7oYc7Ge/coJgF CcBqxNLpcAijaxbeSnRrVBlZt5XXYMcuKmI16imo8uXPrXsgc6quQeKrh7AIngIxRIcZfbBQhaplb uSO0asIa6m/7oLPRInXc+w77ZsNoE5jA6zROrYF/Faw9FQXnqaNVeInN5x1QCcDTTeslxH5ey14wI fVd4AQmtNl3dVPTftlr40OQykkChJdFT6IObLalSJYyGqcTmCUrTYzEESv6Jxjz5yeC0tpLLRT2/O GAQtfEfCaLjClf46seEw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1ptjai-000LkX-0H; Tue, 02 May 2023 06:34:24 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1ptjae-000LjF-02 for linux-arm-kernel@lists.infradead.org; Tue, 02 May 2023 06:34:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1683009260; x=1714545260; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=/BR4pOk50AuJACYyIx65LcARG4wlTe4CEGYiKtvQu1c=; b=Xd5oQ0wleDanRkiENKHZ9snxWKrBa2odw5zyV+3p7apeNxLttXea2uqv BUssG9IkbsdnDzUy+7mFm2zd2w1V1JNa8w2FTXZNzx5qHj9YqYdzFRVPl SBdiXCmCF4V5uvhc26CH9ni8QkmHjt/BgS2+fV3uXjaGZw7kK+Fb7DjrX 98zOTATH3o8Qsnglki8durbHbj/PLgx6dZ3p9H7TlHpHiTG3/JLWfn/f2 dft282l5imMTw0v305P8LnzsAj+elQ3MZrGCgK7ze9NwOkxeQxIVd129O V5Fzd0OPpkqq8I8raipa5Jlo0bO9dp0bojZPSiQ1WOPmA+GB9L+jCPXkb g==; X-IronPort-AV: E=Sophos;i="5.99,243,1677538800"; d="scan'208";a="30654676" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 02 May 2023 08:34:15 +0200 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Tue, 02 May 2023 08:34:15 +0200 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Tue, 02 May 2023 08:34:15 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1683009255; x=1714545255; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=/BR4pOk50AuJACYyIx65LcARG4wlTe4CEGYiKtvQu1c=; b=SxY6tBHht9vZGLH1ksclaWP2SUOF9cZ2S/1RgaJnn1knhAadmhVGExoR qYNEikJIPPW1Y56u9fIx6x7MJIH7Z9P2yKBr98XZmrT3CpwOO/OZAuk+8 uu1MkdJJgspCg0vkMH8HpjzY/9W689MwhwweoLlqsEZklreXwwlf2mwbb 76rB3UAxYh5ZQ02cSVsQsSkh0M7qk6wrYMK26o+cU6Nc7zE7Uq02R3WYD Zr6Rqu5PCaAWSXrPNg92tMosLfblWR4VQKJQ2TTcFncWPgiNiZwMBRATr 5Haqg9ooA7qWMq94zlNX1w5n5eXzfGj2/xCv9ktOGic+5Tayu/rMKE6WA w==; X-IronPort-AV: E=Sophos;i="5.99,243,1677538800"; d="scan'208";a="30654675" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 02 May 2023 08:34:14 +0200 Received: from steina-w.localnet (unknown [10.123.53.21]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id ACCAA280056; Tue, 2 May 2023 08:34:14 +0200 (CEST) From: Alexander Stein To: Jonathan Cameron Cc: Cai Huoqing , Haibo Chen , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Shawn Guo , Sascha Hauer , Fabio Estevam , Pengutronix Kernel Team , NXP Linux Team , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Marco Felsch Subject: Re: [PATCH v2 1/1] dt-bindings: iio: imx8qxp-adc: add missing vref-supply Date: Tue, 02 May 2023 08:34:12 +0200 Message-ID: <5671528.DvuYhMxLoT@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20230430181558.7614cbfc@jic23-huawei> References: <20230424092312.61746-1-alexander.stein@ew.tq-group.com> <20230430181558.7614cbfc@jic23-huawei> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230501_233420_445709_00F1DFB5 X-CRM114-Status: GOOD ( 26.72 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Sonntag, 30. April 2023, 19:15:58 CEST schrieb Jonathan Cameron: > On Mon, 24 Apr 2023 11:23:12 +0200 > = > Alexander Stein wrote: > > Although this property is used right now for IIO_CHAN_INFO_SCALE, > > this ADC has two internal reference voltages, which the driver currently > > doesn't make use of. > > = > > Fixes: db73419d8c06 ("dt-bindings: iio: adc: Add binding documentation = for > > NXP IMX8QXP ADC") Signed-off-by: Alexander Stein > > > = > Interesting that we are talking her only about vrefh, what about vrefp? > I guess the assumption is that will be wired to 0V? What's vrefp? The only other signal related to ADC is vrefl. > The first reference I found didn't seem to imply that was necessarily the > case. https://www.mouser.com/pdfDocs/IMX8QXPAEC.pdf Assuming you are talking about vrefh and vrefl, how is this a problem? Even= if = vrefl is not 0V, the voltage difference is what the ADC uses as reference = voltage, so this should be supplied in the DT then. Best regards Alexander > Jonathan > = > > --- > > Thanks for your inputs. This improved descritpion should make it clear > > that this property is only about the external reference voltage, not > > about the optional, internal voltages. > > = > > Changes in v2: > > * Improved commit message subject as suggested > > * Add hint about feature flag regarding multiple, internal, reference > > = > > voltages > > = > > .../devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml | 7 +++++++ > > 1 file changed, 7 insertions(+) > > = > > diff --git > > a/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml > > b/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml index > > 63369ba388e4..878e67054d7c 100644 > > --- a/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml > > +++ b/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml > > = > > @@ -39,6 +39,12 @@ properties: > > power-domains: > > maxItems: 1 > > = > > + vref-supply: > > + description: | > > + External ADC reference voltage supply on VREFH pad. If VERID[MVI] > > is > > + set, there are additional, internal reference voltages selectabl= e. > > + VREFH1 is always from VREFH pad. > > + > > = > > "#io-channel-cells": > > const: 1 > > = > > @@ -72,6 +78,7 @@ examples: > > assigned-clocks =3D <&clk IMX_SC_R_ADC_0>; > > assigned-clock-rates =3D <24000000>; > > power-domains =3D <&pd IMX_SC_R_ADC_0>; > > = > > + vref-supply =3D <®_1v8>; > > = > > #io-channel-cells =3D <1>; > > = > > }; > > = > > }; -- = TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/ _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel