Received: by 2002:a05:7412:e794:b0:fa:551:50a7 with SMTP id o20csp1089150rdd; Wed, 10 Jan 2024 08:17:59 -0800 (PST) X-Google-Smtp-Source: AGHT+IGHJNCHTQSO47wK0QCjbgwzX+It4aPAdSyaxLp+AEVf0sPT5WxZRkxrD6zkqt3nbHj/aRbg X-Received: by 2002:a17:90b:2b44:b0:28d:bec5:a5be with SMTP id rr4-20020a17090b2b4400b0028dbec5a5bemr593153pjb.68.1704903479079; Wed, 10 Jan 2024 08:17:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704903479; cv=none; d=google.com; s=arc-20160816; b=MLKsGnibLIgFV+mbMHEXXNb+86AlF3TWHC/otCU9hrinbT/sSjaC7ZyCVPRHtKbucL I23TcoAg7M+aB5r4R5kZ5Emx31C7v17+ramINBdZFnUoqAeV4eVEfsvp/vMk6Oxv7AHk IO7EGt0P7GA5JsDo9JnW5Cswkwvn0A6njlVKliIan1ARRqepEJkWhBTPwBnkBnzpgYoI VN6MXhH8KR4T07tdMvOJPqsap32ljp61Z9CV/xwg+v/zys0vcBcA4tBKoZA0AxDie/gR rJ+dMBQhb+fomIQJOkJq5oak0Cj2WhaujYZciz9ofMPimQM4AOMI84r3Pk0F/JF5UyJ8 GsGA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=9wopxSr8beJOyW1Bdq8uThg6V6aSkZOo5nPwaQjPRgQ=; fh=NxRtoii2bBREIQy1/dr8QbeMuJPvTXRps4fA0TZSwns=; b=pViC4DowKWLr4MG3tEq3zMuKlXw2nKaAb5ItcYlwHetHMORMvE11mBwtLNil9UZa16 sYBXbY0emiNePqCBK0c1OvpwpP8RrFvR9Tq0dJkXKak/56iLJCyVT0PkJ+AC3HgFcGTp NUoXT64WzfTKUis8SAxqJB9H/DUXwnyr6fxgdfhZ9uzEt3yjzEBvpFSdRWD5dB4HLtlM mub6tu52ruXrNSOsqX+1FFjwLO7JX+EX6dnE2ZhUGXMJgFsZPQ7B2LkaOAfxZ+iSn1Aj FvNNPbhFPKg0l8m+hgP+hBPGPNd5UE4W3+I2JdbWzDUEpwQbUIm0KEtElUT4Oeo0t2Hd BtsA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=MvsM2yLM; spf=pass (google.com: domain of linux-kernel+bounces-22482-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-22482-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id y14-20020a17090a154e00b0028c0de933bcsi1680432pja.99.2024.01.10.08.17.58 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Jan 2024 08:17:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-22482-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=MvsM2yLM; spf=pass (google.com: domain of linux-kernel+bounces-22482-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-22482-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 sv.mirrors.kernel.org (Postfix) with ESMTPS id BADC128332E for ; Wed, 10 Jan 2024 16:17:58 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 3B3534CB3F; Wed, 10 Jan 2024 16:17:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="MvsM2yLM" 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 5F76C4BAA3; Wed, 10 Jan 2024 16:17:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BD2A0C433C7; Wed, 10 Jan 2024 16:17:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1704903469; bh=pbmSsy1/d6xhr1NG03LlnQKyRKDqNIQsX4bsmOvDy4I=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=MvsM2yLMyutDa9ogOPS0SwOZ4ivFss6rzuKCVYgE4yuHWSWQl2Vc7l/lmRuk4U3JU 3+VNA2r0i95Xy3BB4utIttOd27N1l02xdoJh5GTCGKTTUMjFGnRsSp7zH7yqP+UMS8 kr1gB0+XZBVMxZ4Kbgg7/4To3M55wGyHMiA52n81Y0nSBjsgM3ayhrtaGdHH1xd8yo hIK46hbKH98MM0KzyUZGT8qFDmw3lQfwhj8l/MPiv0ebRr23kExR7rvkU8roGpoH9D Xg+sLJx8feKllrMUTCebxd6gIQhuiCrXdWN3H3ndzRQQQCSh3ikR8zhHFL/tYSuYpC XAaMNkf2F0QzA== Date: Wed, 10 Jan 2024 16:17:45 +0000 From: Conor Dooley To: Dumitru Ceclan Cc: Lars-Peter Clausen , Jonathan Cameron , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Ceclan Dumitru Subject: Re: [PATCH v2 1/4] dt-bindings: iio: hmc425a: add conditional GPIO array size constraints Message-ID: <20240110-unfitting-squander-b1d71c185bb7@spud> References: <20240110153757.5754-1-mitrutzceclan@gmail.com> <20240110153757.5754-2-mitrutzceclan@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ocmaXkT3FSa5q5AJ" Content-Disposition: inline In-Reply-To: <20240110153757.5754-2-mitrutzceclan@gmail.com> --ocmaXkT3FSa5q5AJ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jan 10, 2024 at 05:37:09PM +0200, Dumitru Ceclan wrote: > ADRF5740 and HMC540S have a 4 bit parallel interface. > Update ctr-gpios description and min/maxItems values depending on the > matched compatible to correctly reflect the hardware properties. >=20 > Signed-off-by: Dumitru Ceclan Seems like you need a Fixes: tag, since the original binding was wrong? > --- > .../bindings/iio/amplifiers/adi,hmc425a.yaml | 33 +++++++++++++++++-- > 1 file changed, 30 insertions(+), 3 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a= =2Eyaml b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml > index 67de9d4e3a1d..a434cb8ddcc9 100644 > --- a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml > +++ b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml > @@ -33,11 +33,38 @@ properties: > =20 > ctrl-gpios: > description: > - Must contain an array of 6 GPIO specifiers, referring to the GPIO = pins > - connected to the control pins V1-V6. > - minItems: 6 > + Must contain an array of GPIO specifiers, referring to the GPIO pi= ns > + connected to the control pins. > + ADRF5740 - 4 GPIO connected to D2-D5 > + HMC540S - 4 GPIO connected to V1-V4 > + HMC425A - 6 GPIO connected to V1-V6 > + minItems: 1 > maxItems: 6 > =20 > +allOf: > + - if: > + properties: > + compatible: > + contains: > + const: adi,hmc425a > + then: > + properties: > + ctrl-gpios: > + minItems: 6 > + maxItems: 6 This one should not be needed, it's already set by constraints on the property above. Acked-by: Conor Dooley > + - if: > + properties: > + compatible: > + contains: > + anyOf: > + - const: adi,adrf5740 > + - const: adi,hmc540s > + then: > + properties: > + ctrl-gpios: > + minItems: 4 > + maxItems: 4 I'd say this should be an else, but that clearly would just be churn since your next patch adds something with a different set of constraints. Acked-by: Conor Dooley Cheers, Conor. > + > required: > - compatible > - ctrl-gpios > --=20 > 2.42.0 >=20 --ocmaXkT3FSa5q5AJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZZ7DKQAKCRB4tDGHoIJi 0j1+AQDpkvkoiKiOe2NBsll+Y6LgOQPhgPUftyg19N0ErtAPtQD/eOIioUVTYqnh quZX75EybwW5nU18g2mZW9A+dBsUzw0= =fYKx -----END PGP SIGNATURE----- --ocmaXkT3FSa5q5AJ--