Received: by 2002:a05:7208:9594:b0:7e:5202:c8b4 with SMTP id gs20csp492382rbb; Sat, 24 Feb 2024 09:07:42 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWoXzfxh2RfGPFVMK0XYbBjK+oHykKqYqWEFqo3g+t19rr+ZVIiyO/jWv2cLGr5ptnlqcBY6MYbhbCi3iRJVv9MoUVLl5Cdi6G6DTaT+w== X-Google-Smtp-Source: AGHT+IEnWoMYA8EACjd1sg/BhyWep3Ykxzidms1T7INUq3tXDOoiV77HZVqyvZ4v19iXwRhdTG+k X-Received: by 2002:a17:902:d4cb:b0:1dc:8c77:3381 with SMTP id o11-20020a170902d4cb00b001dc8c773381mr1038346plg.14.1708794461860; Sat, 24 Feb 2024 09:07:41 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708794461; cv=pass; d=google.com; s=arc-20160816; b=c6wR6mnsvFdRw2N6fJuysBwyJR+O4sc2SUKuJIufEJVKnTbJtpGiQdgraol3CotKso eTdTpCNA2rDMm2NEYA0IOZ15veXHfq8HvBeYir7LHolJCGl8XZIneeK0vA+ROLf7bxsd Ke3uxkJMKHJHmZfkQdCq+MnwYi9POWOHkj4/f0HPUnhb7qv4XvD175QScH2jhbehAJM6 CI6YK53TyNfxFTqgFPxCLvtQAAuC9cBE0J8UPB52OMRGTyodExwxUNyjiWOmmIIhcT55 981Xp1w+6Jh66234WmCW2cH59hjcanHGQcINwCbGBJnNlPlRkooOU38O1l0KWsaoa11N wg3Q== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :subject:cc:to:from:date:dkim-signature; bh=7Rfnd/GqfFT1WFKSiAuC+CdYmPoFRTQhG8W81yIj/JA=; fh=KbVjG2Kk3/DgP/UxKx75ufC6RDwUOasLccdOIlag9YY=; b=abce9lMbHCryYlw8zy5x3FYB6VStBFFjaZ1adZcIlxOlWCWOyYgP0KyMXO81tmdU1V VNdnUwQz8drR+my+tM0JyqVohor7wX5X3myEZHnU5O67cW1U0ouiIf1YXnoIhZkIwjwo im6DDe1KGtxhTTKaV793xrXzM7AZ0bnQctM/oSEjmnmZQf1oNrNHbY/yxagegIMiiY5N dC2fDU1EGPSlYLAKkwmoNdjQ/fUd4sm9kgmPo5OHx0CE9A0VSEM13f1k7kjNRJv9j6LT A7LBczfFGxDOGn2AT/PVhtCmxy8ibyflkCyg2bwzNSpIync06cxpATPKYXxTCfnrJOw1 7+KQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=JkpaewT3; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-79753-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-79753-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id ma4-20020a170903094400b001dc64999b6dsi1057411plb.48.2024.02.24.09.07.41 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 24 Feb 2024 09:07:41 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-79753-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=JkpaewT3; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-79753-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-79753-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 05526B21A61 for ; Sat, 24 Feb 2024 17:07:36 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 42DB5DDCB; Sat, 24 Feb 2024 17:07:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="JkpaewT3" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5ED341EB34; Sat, 24 Feb 2024 17:07:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708794445; cv=none; b=QxuJMrsavDoYK44ZePvnaJ2/vsvVwb9Wxd3ySVnECQwCaxQe+mJeSpPawE3koipvp7UACpP8ZTh5af97HNCWFehDKB07B7JF4Q/Wicji7+lcBLhQKNogS8N5puhN12tUsgnVJMLrJVWd7g5P7f9Xscjt8/l/jtHdYw7LxdGLLUM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708794445; c=relaxed/simple; bh=eeYrfwWrvmfry0lGfIaaBnx0Xj+7Z5pHDan2aTVa8+I=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=qPKPFySMYMXYrdcB69Bxd3jALU0K+p7HVcGaziAUsAGyxLDm0vgXzCdRs/o0uF9Nf3Ia8YKEqPBwhfKjfs8Ufl2Un/BBxa418m3C08ht8TuatrLLgaj5aH5+zHu5ukdXgPIwyJ/NMKtDW4i7qT9AwjIyGkRyy2qg0sRGr8R7Cns= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=JkpaewT3; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7AB6AC433C7; Sat, 24 Feb 2024 17:07:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1708794445; bh=eeYrfwWrvmfry0lGfIaaBnx0Xj+7Z5pHDan2aTVa8+I=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=JkpaewT3OUJ2LYRrRKYiGZ62iq7pxMqURkDolTc8o17I5DbxGHgwjDdelaWrH8vl/ kuV9Wo0qgEbkhCqha8x9VLGMIVSIc4Khw4EbtBIA22UpAC/yFONVr2VZsASAfnpu0I dcXPBkoB1dMMYQyM0MX5MSrkauZ6O15TEN8VWm+7ZLzvZfzF528+bsMD8wP/gw//m2 ywxBIDwnexfQjxp9Lj++wdsHvHC1itGK99Nf2Lc49E9/qFhS/RtirkaV1kJX8Lm5qr a9LDQNN8mYIPytlJ8erMO26DLQpgzXAF5hTSzXiC1HPz4JP9v7P+24wKg2pcEbe5wz XAQYY05pGaXng== Date: Sat, 24 Feb 2024 17:07:07 +0000 From: Jonathan Cameron To: Ceclan Dumitru Cc: David Lechner , linus.walleij@linaro.org, brgl@bgdev.pl, andy@kernel.org, linux-gpio@vger.kernel.org, Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Walle , Andy Shevchenko , Arnd Bergmann , ChiaEn Wu , Niklas Schnelle , Leonard =?UTF-8?B?R8O2aHJz?= , Mike Looijmans , Haibo Chen , Hugo Villeneuve , Ceclan Dumitru , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v13 1/3] dt-bindings: adc: add AD7173 Message-ID: <20240224170707.4ffcec19@jic23-huawei> In-Reply-To: <0fa819d1-2a0f-4243-8e5f-a098528278c3@gmail.com> References: <20240220094344.17556-1-mitrutzceclan@gmail.com> <0fa819d1-2a0f-4243-8e5f-a098528278c3@gmail.com> X-Mailer: Claws Mail 4.2.0 (GTK 3.24.41; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Wed, 21 Feb 2024 10:29:30 +0200 Ceclan Dumitru wrote: > On 2/20/24 22:54, David Lechner wrote: > > On Tue, Feb 20, 2024 at 3:43=E2=80=AFAM Dumitru Ceclan wrote: =20 >=20 > ... >=20 > >> + clocks: > >> + maxItems: 1 > >> + description: | =20 > >=20 > > Don't need `|` here. > > =20 > The description contains ": ". Without '|' yaml syntax considers the > whole string before ':' as another attribute >=20 > >> + Optional external clock source. Can include one clock source: e= xternal > >> + clock or external crystal. > >> + =20 >=20 > ... >=20 > >> + > >> + diff-channels: > >> + items: > >> + minimum: 0 > >> + maximum: 31 > >> + =20 > >=20 > > Are we missing `bipolar: true` here? (since we have > > unevaluatedProperties: false) > > =20 >=20 > No, since we are referencing the adc schema "$ref: adc.yaml" > Which contains: > """ > bipolar: >=20 > $ref: /schemas/types.yaml#/definitions/flag >=20 > description: If provided, the channel is to be used in bipolar mode. > """ >=20 >=20 > ... >=20 > >> + > >> +required: > >> + - compatible > >> + - reg =20 > >=20 > > Aren't the various power supplies supposed to be required? > >=20 > > - avdd-supply > > - avdd2-supply > > - iovdd-supply =20 >=20 > From my point of view, if someone uses a single supply (avdd =3D=3D avdd2= =3D=3D > iovdd), and uses only the internal reference then the supplies should > not necessarily be required. Convention is that anything that represent a voltage on a pin that is needed for operation should be required. Key here is the difference from optional supplies where the driver does something different. vref is a good example of this. The ones above are always needed I think. Obviously they may all say the same thing if they are connected externally.