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 0EFC6C2BBCA for ; Tue, 25 Jun 2024 16:35:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Type:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To :Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=8YGpp4KqcK927fmutvzo+C6+viGUqZ4wb+9r93852Tk=; b=OJMwoZP+t7ENPSJUE0xTkb+0Oq UeQCI4Aqxc30qe+giGyL38etXTIV/SqTV/3CD/OSALqBE3Tw0GTLV32wcP9Uhv8l8dEArdU/HTobe a47EnaU9f/a9wZlXCGkChIPzE6eQ13EGz4/r1lI1n6fK1Kzz1ra+PrSYORXBS0GZmbLcTDgN676Zj 7pD1kkoifOK69xW1m3bF2ahQb6/k/cKaMpNUN4dn18NiTQ63WykkhsPBO/UwZxLQdG1kYcUm+1MdF 9Euc28dx5P8QDj4KmF6Hb1n6NnzdUItbvxaoT2xjTZPAIQxA3uh+z3kcaQzhSLCqKZNYCYXsZQpdl lUnvc1Sg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sM98B-00000003iTk-0pdk; Tue, 25 Jun 2024 16:34:55 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sM982-00000003iPo-0m29 for linux-arm-kernel@lists.infradead.org; Tue, 25 Jun 2024 16:34:48 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 9E8CC6143F; Tue, 25 Jun 2024 16:34:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5DAA0C32786; Tue, 25 Jun 2024 16:34:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719333285; bh=q4fE+7eaIIr13+voG5rrTACoPpuwo5LGe0aCY955GSA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nUL8q5UOFj08sDJ0TVVFPZqKLQykvjRO1wzx2LmAXZy2e5iieEra0ofAVrnlI4tk7 O/iO+bAsEGB3Hbr6Fx4PbWO8FIOt0R5lmC7LCN0qyBl6Ah5xLzpg61tdThOcFx8xdG ASzQTC/r/1w9FLZ4kQrbdYWBTRtfHsOMhK3RO8s4YQJt6pbQY19Y+FbZjPuGI3Z0ls Eoefo4mv3g0ZgVVhjhAyDtGmnTJR8bxYPqjjrE0hIKrQhgTFBADUehQitO7I0lEwHj y/r/tjIcsscq+yV7GBoIn6/KdLhrbLfaVlpZBEQAtH9i6QV4ySVUXZZlRU8QW999W8 +QBK4SW6jRkYA== Date: Tue, 25 Jun 2024 17:34:40 +0100 From: Conor Dooley To: Manikandan Muralidharan Subject: Re: [PATCH v2 4/5] dt-bindings: gpio: convert Atmel GPIO to json-schema Message-ID: <20240625-clerical-manicure-ebc8a1d1b16b@spud> References: <20240625043525.279711-1-manikandan.m@microchip.com> <20240625043525.279711-5-manikandan.m@microchip.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="or5R3vZ5mvg49RKg" Content-Disposition: inline In-Reply-To: <20240625043525.279711-5-manikandan.m@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240625_093446_372278_D860F530 X-CRM114-Status: GOOD ( 21.56 ) 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: , Cc: robh@kernel.org, conor+dt@kernel.org, arnd@arndb.de, alexandre.belloni@bootlin.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linus.walleij@linaro.org, claudiu.beznea@tuxon.dev, linux-gpio@vger.kernel.org, durai.manickamkr@microchip.com, krzk+dt@kernel.org, brgl@bgdev.pl, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --or5R3vZ5mvg49RKg Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jun 25, 2024 at 10:05:24AM +0530, Manikandan Muralidharan wrote: > Convert the Atmel GPIO controller binding document to DT schema format > using json-schema. > The compatible string "microchip,sam9x7-gpio" is added as well. >=20 > Signed-off-by: Manikandan Muralidharan > --- > changes in v2: > - Fix bot errors with 'make dt_binding_check', missed to add > "atmel,at91rm9200-gpio" as separate compatible for devices that uses it > - Remove label from example > - Add default entry for #gpio-lines property > - Add new compatible string details in commit message > --- > .../bindings/gpio/atmel,at91rm9200-gpio.yaml | 81 +++++++++++++++++++ > .../devicetree/bindings/gpio/gpio_atmel.txt | 31 ------- > 2 files changed, 81 insertions(+), 31 deletions(-) > create mode 100644 Documentation/devicetree/bindings/gpio/atmel,at91rm92= 00-gpio.yaml > delete mode 100644 Documentation/devicetree/bindings/gpio/gpio_atmel.txt >=20 > diff --git a/Documentation/devicetree/bindings/gpio/atmel,at91rm9200-gpio= =2Eyaml b/Documentation/devicetree/bindings/gpio/atmel,at91rm9200-gpio.yaml > new file mode 100644 > index 000000000000..3dd70933ed8e > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpio/atmel,at91rm9200-gpio.yaml > @@ -0,0 +1,81 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/gpio/atmel,at91rm9200-gpio.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Microchip GPIO controller (PIO) > + > +maintainers: > + - Manikandan Muralidharan > + > +properties: > + compatible: > + oneOf: > + - items: > + - enum: > + - atmel,at91sam9x5-gpio > + - microchip,sam9x60-gpio > + - const: atmel,at91rm9200-gpio > + - items: > + - enum: > + - microchip,sam9x7-gpio > + - const: microchip,sam9x60-gpio > + - const: atmel,at91rm9200-gpio It's worth pointing out that this is required, because the driver implements a different set of ops for the sam9x60. There's not just more of them, they're different too. Are the sam9x60 and at91rm9200 are actually compatible, or is the fallback here some mistake that originated in the dts? --or5R3vZ5mvg49RKg Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZnrxoAAKCRB4tDGHoIJi 0ql8AP0cwdCvyag4UWMffdbYolpWJoRGv4r3yRn8ULhaM4CovQEAwn+TazCP90z8 dPd+Hevm548epiRVEwTDWuc8T/sw6gU= =U0hx -----END PGP SIGNATURE----- --or5R3vZ5mvg49RKg--