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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0A38FE7D0A4 for ; Thu, 21 Sep 2023 18:52:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230396AbjIUSwd (ORCPT ); Thu, 21 Sep 2023 14:52:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59324 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230347AbjIUSwW (ORCPT ); Thu, 21 Sep 2023 14:52:22 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 33B66ECA63 for ; Thu, 21 Sep 2023 11:44:14 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BAB9FC433C8; Thu, 21 Sep 2023 18:44:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695321853; bh=8Ky5HU9QsWtwK565KZVC73jkJBv6z9hSJThMalkX6wk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cc9HJs80jcQb27Yue8ri7NXdOPVbGkkCWLuAEvqjUUuolg/M8IMK2u1XB2/H1GCrZ bxJ6SgG/uLmE6tkfF/Yyft2qwUaK9hy2cNijKxB3ofO5IsceHyG4nxmpeUphKjl5h+ 3aQNvJ1Vyu0Ta4W2Shgej9Ce99Q2ZOOGlvaaIUPMMKD1PaRd2xdXVN8tuqpxr/BQGR PV8Ur1oeHs+zlLc1ytu89RGDwyTNVzdONIQG40S1weEDOaTMedWVTvLoeSGZX9XDH8 6IEjm5/pOb4pJ4Djwanbh1PMhvQ3tDvytZFo6Wfz4XeWbzkVaVFy4xF9xGxZqZqg8+ EfrEtkJwKhETA== Received: (nullmailer pid 788465 invoked by uid 1000); Thu, 21 Sep 2023 18:44:11 -0000 Date: Thu, 21 Sep 2023 13:44:11 -0500 From: Rob Herring To: Linus Walleij Cc: Linus Walleij , Imre Kaloz , Krzysztof Halasa , Bartosz Golaszewski , Andy Shevchenko , Krzysztof Kozlowski , Conor Dooley , linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 1/2] gpio: Rewrite IXP4xx GPIO bindings in schema Message-ID: <20230921184411.GA784095-robh@kernel.org> References: <20230921-ixp4xx-gpio-clocks-v1-0-574942bf944a@linaro.org> <20230921-ixp4xx-gpio-clocks-v1-1-574942bf944a@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230921-ixp4xx-gpio-clocks-v1-1-574942bf944a@linaro.org> Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Thu, Sep 21, 2023 at 12:23:45AM +0200, Linus Walleij wrote: > This rewrites the IXP4xx GPIO bindings to use YAML schema, > and adds two new properties to enable fixed clock output on > pins 14 and 15. > > Signed-off-by: Linus Walleij > --- > .../devicetree/bindings/gpio/intel,ixp4xx-gpio.txt | 38 ------------ > .../bindings/gpio/intel,ixp4xx-gpio.yaml | 70 ++++++++++++++++++++++ > MAINTAINERS | 2 +- > 3 files changed, 71 insertions(+), 39 deletions(-) > > diff --git a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt > deleted file mode 100644 > index 8dc41ed99685..000000000000 > --- a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt > +++ /dev/null > @@ -1,38 +0,0 @@ > -Intel IXP4xx XScale Networking Processors GPIO > - > -This GPIO controller is found in the Intel IXP4xx processors. > -It supports 16 GPIO lines. > - > -The interrupt portions of the GPIO controller is hierarchical: > -the synchronous edge detector is part of the GPIO block, but the > -actual enabling/disabling of the interrupt line is done in the > -main IXP4xx interrupt controller which has a 1:1 mapping for > -the first 12 GPIO lines to 12 system interrupts. > - > -The remaining 4 GPIO lines can not be used for receiving > -interrupts. > - > -The interrupt parent of this GPIO controller must be the > -IXP4xx interrupt controller. > - > -Required properties: > - > -- compatible : Should be > - "intel,ixp4xx-gpio" > -- reg : Should contain registers location and length > -- gpio-controller : marks this as a GPIO controller > -- #gpio-cells : Should be 2, see gpio/gpio.txt > -- interrupt-controller : marks this as an interrupt controller > -- #interrupt-cells : a standard two-cell interrupt, see > - interrupt-controller/interrupts.txt > - > -Example: > - > -gpio0: gpio@c8004000 { > - compatible = "intel,ixp4xx-gpio"; > - reg = <0xc8004000 0x1000>; > - gpio-controller; > - #gpio-cells = <2>; > - interrupt-controller; > - #interrupt-cells = <2>; > -}; > diff --git a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml > new file mode 100644 > index 000000000000..bb1fc393bd8c > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml > @@ -0,0 +1,70 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/gpio/intel,ixp4xx-gpio.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Intel IXP4xx XScale Networking Processors GPIO Controller > + > +description: This GPIO controller is found in the Intel IXP4xx > + processors. It supports 16 GPIO lines. > + The interrupt portions of the GPIO controller is hierarchical. > + The synchronous edge detector is part of the GPIO block, but the > + actual enabling/disabling of the interrupt line is done in the > + main IXP4xx interrupt controller which has a 1-to-1 mapping for > + the first 12 GPIO lines to 12 system interrupts. > + The remaining 4 GPIO lines can not be used for receiving > + interrupts. > + The interrupt parent of this GPIO controller must be the > + IXP4xx interrupt controller. > + GPIO 14 and 15 can be used as clock outputs rather than GPIO, > + and this can be enabled by a special flag. Do you care about the formatting here? If so, you are missing a '|'. If not, you have strange line breaks. > + > +maintainers: > + - Linus Walleij > + > +properties: > + compatible: > + const: intel,ixp4xx-gpio > + > + reg: > + maxItems: 1 > + > + gpio-controller: true Blank line between DT properties > + "#gpio-cells": > + const: 2 > + > + interrupt-controller: true Blank line > + "#interrupt-cells": > + const: 2 Otherwise, with those nits fixed: Reviewed-by: Rob Herring 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 085ABE7D0A4 for ; Thu, 21 Sep 2023 18:44:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=LN923iqVhw7SMca41s0s5sMHdSM9um9dGtGjid1yNRM=; b=4OhWD+NfYWlrEF U+vZkt0Kt599t7QGf2kW1dp0PxybDgO1KSvu1Iy7nfaFpf4RVgXSzQRrIPjgIey6lanw+vpayn53i 3WcQXt6DnvyEIbJCAPDHiDPNXQLtM6NkCBCFt/8BaXdV4jWOH/2XfV43GG7zrR4taOhFXAWHofeeD +h2lyKUuVg57fJG8eTZkuAArTZ4tCqDATkMupNQfTRoX3/sJ+4ptVTvjfZ1fqxmdWxUTvVn2ZnwUT XE6PHjVFcnJN34K/lZ8HsZqu1Qs/8N2Y4j6jYAq7oJ/InKNahlpDB9fWUCdytwlr7TqiFgT6T3SKr RXEdlhAod25vJJNq9ciA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qjOew-006qkS-0w; Thu, 21 Sep 2023 18:44:18 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qjOet-006qjP-1L for linux-arm-kernel@lists.infradead.org; Thu, 21 Sep 2023 18:44:17 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 5F3F662011; Thu, 21 Sep 2023 18:44:14 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BAB9FC433C8; Thu, 21 Sep 2023 18:44:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695321853; bh=8Ky5HU9QsWtwK565KZVC73jkJBv6z9hSJThMalkX6wk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cc9HJs80jcQb27Yue8ri7NXdOPVbGkkCWLuAEvqjUUuolg/M8IMK2u1XB2/H1GCrZ bxJ6SgG/uLmE6tkfF/Yyft2qwUaK9hy2cNijKxB3ofO5IsceHyG4nxmpeUphKjl5h+ 3aQNvJ1Vyu0Ta4W2Shgej9Ce99Q2ZOOGlvaaIUPMMKD1PaRd2xdXVN8tuqpxr/BQGR PV8Ur1oeHs+zlLc1ytu89RGDwyTNVzdONIQG40S1weEDOaTMedWVTvLoeSGZX9XDH8 6IEjm5/pOb4pJ4Djwanbh1PMhvQ3tDvytZFo6Wfz4XeWbzkVaVFy4xF9xGxZqZqg8+ EfrEtkJwKhETA== Received: (nullmailer pid 788465 invoked by uid 1000); Thu, 21 Sep 2023 18:44:11 -0000 Date: Thu, 21 Sep 2023 13:44:11 -0500 From: Rob Herring To: Linus Walleij Cc: Linus Walleij , Imre Kaloz , Krzysztof Halasa , Bartosz Golaszewski , Andy Shevchenko , Krzysztof Kozlowski , Conor Dooley , linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 1/2] gpio: Rewrite IXP4xx GPIO bindings in schema Message-ID: <20230921184411.GA784095-robh@kernel.org> References: <20230921-ixp4xx-gpio-clocks-v1-0-574942bf944a@linaro.org> <20230921-ixp4xx-gpio-clocks-v1-1-574942bf944a@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230921-ixp4xx-gpio-clocks-v1-1-574942bf944a@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230921_114415_538068_0FD91328 X-CRM114-Status: GOOD ( 26.81 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Sep 21, 2023 at 12:23:45AM +0200, Linus Walleij wrote: > This rewrites the IXP4xx GPIO bindings to use YAML schema, > and adds two new properties to enable fixed clock output on > pins 14 and 15. > > Signed-off-by: Linus Walleij > --- > .../devicetree/bindings/gpio/intel,ixp4xx-gpio.txt | 38 ------------ > .../bindings/gpio/intel,ixp4xx-gpio.yaml | 70 ++++++++++++++++++++++ > MAINTAINERS | 2 +- > 3 files changed, 71 insertions(+), 39 deletions(-) > > diff --git a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt > deleted file mode 100644 > index 8dc41ed99685..000000000000 > --- a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.txt > +++ /dev/null > @@ -1,38 +0,0 @@ > -Intel IXP4xx XScale Networking Processors GPIO > - > -This GPIO controller is found in the Intel IXP4xx processors. > -It supports 16 GPIO lines. > - > -The interrupt portions of the GPIO controller is hierarchical: > -the synchronous edge detector is part of the GPIO block, but the > -actual enabling/disabling of the interrupt line is done in the > -main IXP4xx interrupt controller which has a 1:1 mapping for > -the first 12 GPIO lines to 12 system interrupts. > - > -The remaining 4 GPIO lines can not be used for receiving > -interrupts. > - > -The interrupt parent of this GPIO controller must be the > -IXP4xx interrupt controller. > - > -Required properties: > - > -- compatible : Should be > - "intel,ixp4xx-gpio" > -- reg : Should contain registers location and length > -- gpio-controller : marks this as a GPIO controller > -- #gpio-cells : Should be 2, see gpio/gpio.txt > -- interrupt-controller : marks this as an interrupt controller > -- #interrupt-cells : a standard two-cell interrupt, see > - interrupt-controller/interrupts.txt > - > -Example: > - > -gpio0: gpio@c8004000 { > - compatible = "intel,ixp4xx-gpio"; > - reg = <0xc8004000 0x1000>; > - gpio-controller; > - #gpio-cells = <2>; > - interrupt-controller; > - #interrupt-cells = <2>; > -}; > diff --git a/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml > new file mode 100644 > index 000000000000..bb1fc393bd8c > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpio/intel,ixp4xx-gpio.yaml > @@ -0,0 +1,70 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/gpio/intel,ixp4xx-gpio.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Intel IXP4xx XScale Networking Processors GPIO Controller > + > +description: This GPIO controller is found in the Intel IXP4xx > + processors. It supports 16 GPIO lines. > + The interrupt portions of the GPIO controller is hierarchical. > + The synchronous edge detector is part of the GPIO block, but the > + actual enabling/disabling of the interrupt line is done in the > + main IXP4xx interrupt controller which has a 1-to-1 mapping for > + the first 12 GPIO lines to 12 system interrupts. > + The remaining 4 GPIO lines can not be used for receiving > + interrupts. > + The interrupt parent of this GPIO controller must be the > + IXP4xx interrupt controller. > + GPIO 14 and 15 can be used as clock outputs rather than GPIO, > + and this can be enabled by a special flag. Do you care about the formatting here? If so, you are missing a '|'. If not, you have strange line breaks. > + > +maintainers: > + - Linus Walleij > + > +properties: > + compatible: > + const: intel,ixp4xx-gpio > + > + reg: > + maxItems: 1 > + > + gpio-controller: true Blank line between DT properties > + "#gpio-cells": > + const: 2 > + > + interrupt-controller: true Blank line > + "#interrupt-cells": > + const: 2 Otherwise, with those nits fixed: Reviewed-by: Rob Herring _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel