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 CCBD8C30658 for ; Fri, 28 Jun 2024 16:48:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc: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=oSed/PvaOJ+aacYgk2KYRlvnhVeqh1GjEiIEr4HRc7g=; b=HUUQSSoKlQ8yWoxa07Wb5+KDU8 Sk+HF7g7f54XwBcLVuNUDvpntwBFaZuz2aNr9xNY6ETDU39wcL/occ+A7ABVSxNebTov/GTatSOcZ YqfcvZKNUB3V8Zfy52j5V+quhqvjbVuCZmHZYr9BsuvDOIk/STdDp6nT2Oz7Tf/sfnX0blVPkDV/n AHGnojd3B1Xkmb3LqJF0nEX5+yCcvvGbH7XDjepdB/CpXe9IVVoUbUXtXG+s6JcIScXVMwtcDUak/ 2q3kzoq9AxmwPRvEh8TbYqbwZ3023lkUZQdb6/aq8+nESRL56EeMJctp8ggEDsZ3mHRDlSy4WSDos g5JeXfbA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sNEmB-0000000EOyJ-1FHY; Fri, 28 Jun 2024 16:48:43 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sNEm1-0000000EOxD-1Dxh for linux-arm-kernel@lists.infradead.org; Fri, 28 Jun 2024 16:48:34 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 86B8F62213; Fri, 28 Jun 2024 16:48:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4CADDC116B1; Fri, 28 Jun 2024 16:48:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719593312; bh=xPTahYQJelAr+V5E11fJx0UgUKke4AGftBFZip9Tksg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FBIG95UM2bQkGjm2FtAz7IEzUjyfJDIV3TSKc6ZjvqMuCacQ1xi2PZwCMW2GQUs5U 5yvMQu6Z2aRdAm3Y0j4GvCX0/vHU9Dyf3UVzMesJe4pzbcFbe30IufPxKCaPlQVKBy DybzGI0pilibQLVCvg6CcdOJNBsWpbjn/bVKRznUSIbHqGog15KzVsSXVp7belElSf A1WNhLne0185tQBoFJoPVej7lWzh4mZqJIlCI7GeYFCCuMh8LN6Cb494fVNIV9WKJD njpKEj1SMqxCjONfgu1kV/PvpmOWJaKfnDqqGjQOLRLvuaUXRTemupatmjRn5eWxvF pwgtbV7H8RZUQ== Date: Fri, 28 Jun 2024 17:48:26 +0100 From: Conor Dooley To: Ayush Singh Cc: Mark Brown , Vaishnav M A , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Michael Walle , Andrew Lunn , jkridner@beagleboard.org, robertcnelson@beagleboard.org, linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v5 3/7] dt-bindings: mikrobus: Add mikrobus-spi binding Message-ID: <20240628-cake-ocean-25363137b1ab@spud> References: <20240627-mikrobus-scratch-spi-v5-0-9e6c148bf5f0@beagleboard.org> <20240627-mikrobus-scratch-spi-v5-3-9e6c148bf5f0@beagleboard.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="q7m1m69FJSotxIJN" Content-Disposition: inline In-Reply-To: <20240627-mikrobus-scratch-spi-v5-3-9e6c148bf5f0@beagleboard.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240628_094833_488197_EBE6CE4D X-CRM114-Status: GOOD ( 26.16 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --q7m1m69FJSotxIJN Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jun 27, 2024 at 09:56:13PM +0530, Ayush Singh wrote: > Add bindings for MikroBUS boards using SPI interface. >=20 > Almost all of the properties that are valid for SPI devices can be used > except reg. Since the goal is to allow use of the same MikroBUS board > across different connectors, config needs to be independent of the actual > SPI controller in mikroBUS port(s), it is not possible to define the > chipselect by number in advance. Thus, `spi-cs-apply` property is used to > specify the chipselect(s) by name. >=20 > Another important fact is that while there is a CS pin in the mikroBUS > connector, some boards (eg SPI Extend Click) use additional pins as > chipselect. Thus we need a way to specify the CS pin(s) in terms of > mikcrobus-connector which can then handle bindings the actual CS pin(s). >=20 > Link: https://www.mikroe.com/spi-extend-click SPI Extend Click >=20 > Signed-off-by: Ayush Singh > --- > .../devicetree/bindings/mikrobus/mikrobus-spi.yaml | 37 ++++++++++++++++= ++++++ > MAINTAINERS | 1 + > 2 files changed, 38 insertions(+) >=20 > diff --git a/Documentation/devicetree/bindings/mikrobus/mikrobus-spi.yaml= b/Documentation/devicetree/bindings/mikrobus/mikrobus-spi.yaml > new file mode 100644 > index 000000000000..35ca2cce3b03 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mikrobus/mikrobus-spi.yaml > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mikrobus/mikrobus-spi.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: mikroBUS add-on board using SPI > + > +maintainers: > + - Ayush Singh > + > +allOf: > + - $ref: /schemas/mikrobus/mikrobus-board.yaml# > + > +properties: > + compatible: > + const: mikrobus-spi > + > + spi-cs-apply: > + minItems: 1 > + maxItems: 12 > + items: > + enum: [default, pwm, int, rx, tx, scl, sda, an, rst, sck, cipo, co= pi] Property descriptions belong in the property, not in the commit message. > + > +required: > + - compatible > + > +additionalProperties: false > + > +examples: > + - | > + thermo-click { > + compatible =3D "maxim,max31855k", "mikrobus,spi"; I am really not keen on what this implies, as I think Rob and I have already mentioned, the connector should handle the "mapping" and the regular SPI/I2C/whatever bindings for the SPI devices themselves should be usable. Also you clearly didn't test this binding - please test them. Thanks, Conor. > + spi-max-frequency =3D <1000000>; > + pinctrl-apply =3D "default", "spi_default"; > + spi-cs-apply =3D "default"; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index 14eba18832d5..88f2b3adc824 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -15114,6 +15114,7 @@ M: Vaishnav M A > S: Maintained > F: Documentation/devicetree/bindings/connector/mikrobus-connector.yaml > F: Documentation/devicetree/bindings/mikrobus/mikrobus-board.yaml > +F: Documentation/devicetree/bindings/mikrobus/mikrobus-spi.yaml > =20 > MIKROTIK CRS3XX 98DX3236 BOARD SUPPORT > M: Luka Kovacic >=20 > --=20 > 2.45.2 >=20 --q7m1m69FJSotxIJN Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZn7pWgAKCRB4tDGHoIJi 0t+BAP9xCq4ptTm0LV6thbxqhsSKb00MXWPQyiah0LuqX6j+BAEA45D3y7cLzK8q UE7G5invTOoL958xU4V3zI+AeAlTXAo= =S7NC -----END PGP SIGNATURE----- --q7m1m69FJSotxIJN--