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 6DE9DC3064D for ; Fri, 28 Jun 2024 16:05:06 +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=0MvgHP9fzoVayfnRezd1YuCbJK7a1W4elBLt7LTDecs=; b=DDtn4S6xgHOGFdU52cj/UosE7v 0yxxsfzGxt0WBQs78ibjHXM21RlYiOoGCkBjS7EeIt+94Mvp6vVaBqTfIRe83G4TwsVyqk4JUspK8 7iPwiQiE7ZczkiJcCzt5+KXjcJEr6bqLML95wVQtRP3LPHWgXcrvlnXbsAY5zjDkAuGb9fiIw9m/h dExe3S1Vh04GbTF4ynbg2zx1yI1OIrsX7hfp0dxp6hmszB3YvZYIM1I/8GZy1r2Cdi0uu8Gk+Czr2 QjLwOYUY0CAd1Md9Z/wy3f8EwbFr4OI5TK/djmJWqa9CV+oF36du1rPRgoCNmnEOtVVXLtwmUV6HO SWIT2EWA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sNE5j-0000000EI4B-36PI; Fri, 28 Jun 2024 16:04:51 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sNE5Z-0000000EI2K-3oQ7 for linux-arm-kernel@lists.infradead.org; Fri, 28 Jun 2024 16:04:43 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id E3AB0CE2E22; Fri, 28 Jun 2024 16:04:39 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 96D7CC116B1; Fri, 28 Jun 2024 16:04:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719590679; bh=oTXfR2N6eN/rBWJlM/QQkHwnHvpBAkB6fBJKmFWgmyk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=TXGHVV84R2UNCEu9g298LoeAx0ZyOEsBfQK44L0ec56H4f3VQwhojE+81Rnwy+TcM I5AjHgSndtYSBRNJQ6eVyUB8t3tCOT+iVlvVxoszKdjMcfTVVoCQQFdT/0CmcFaJBC CsQcOvngcssNeMhHRU4M7ho7VYtt115QGJR6zUEyNKoc3I0CQBRMNcV/NU1rfbIYPd 11LIlC34+YcAO7Lork3rWefTA4utV4aLhP7kgdVDzv6iIqONVJf994omfi67deCeSB RytUpxaEetfcST35QVs9DQNLFl8scGjVrE/JtYVNVspVUrhNn998yVet/oDyZUtPh8 Ur9qQgERo7pQA== Date: Fri, 28 Jun 2024 10:04:36 -0600 From: Rob Herring To: Ayush Singh Cc: Mark Brown , Vaishnav M A , 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 2/7] dt-bindings: mikrobus: Add mikrobus board base Message-ID: <20240628160436.GA3143032-robh@kernel.org> References: <20240627-mikrobus-scratch-spi-v5-0-9e6c148bf5f0@beagleboard.org> <20240627-mikrobus-scratch-spi-v5-2-9e6c148bf5f0@beagleboard.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240627-mikrobus-scratch-spi-v5-2-9e6c148bf5f0@beagleboard.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240628_090442_325063_76360C6D X-CRM114-Status: GOOD ( 17.59 ) 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 On Thu, Jun 27, 2024 at 09:56:12PM +0530, Ayush Singh wrote: > Base dt bindings for mikrobus addon boards. Contains properties that are > part of all types of boards (SPI, I2C, etc). > > Each pin in mikroBUS connector can either be used for it's original > purpose (UART, I2C, SPI, etc) or as a normal GPIO. Introducing > `pinctrl-apply` allows selecting the pin configuration by name. This seems pointless. If a board uses UART, then uart_default has to be supported. Why does the board need to list it? > > Note: Some mikrobus-connectors might not support all valid pinctrl. > > Signed-off-by: Ayush Singh > --- > .../devicetree/bindings/mikrobus/mikrobus-board.yaml | 20 ++++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 21 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mikrobus/mikrobus-board.yaml b/Documentation/devicetree/bindings/mikrobus/mikrobus-board.yaml > new file mode 100644 > index 000000000000..42e2219c596f > --- /dev/null > +++ b/Documentation/devicetree/bindings/mikrobus/mikrobus-board.yaml > @@ -0,0 +1,20 @@ > +# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mikrobus/mikrobus-board.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: mikroBUS add-on board properties > + > +maintainers: > + - Ayush Singh > + > +properties: > + pinctrl-apply: Missing a description. > + minItems: 1 > + maxItems: 9 > + items: > + enum: [default, pwm_default, pwm_gpio, uart_default, uart_gpio, i2c_default, i2c_gpio, spi_default, > + spi_gpio] > + > +additionalProperties: false > diff --git a/MAINTAINERS b/MAINTAINERS > index 8e4115e93aeb..14eba18832d5 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -15113,6 +15113,7 @@ M: Ayush Singh > M: Vaishnav M A > S: Maintained > F: Documentation/devicetree/bindings/connector/mikrobus-connector.yaml > +F: Documentation/devicetree/bindings/mikrobus/mikrobus-board.yaml > > MIKROTIK CRS3XX 98DX3236 BOARD SUPPORT > M: Luka Kovacic > > -- > 2.45.2 >