From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2 1/7] dt-binding: Add ngpios property to GPIO controller node Date: Fri, 20 Nov 2015 09:33:09 -0600 Message-ID: <20151120153309.GA4944@rob-hp-laptop> References: <1447905139-16002-1-git-send-email-pramodku@broadcom.com> <1447905139-16002-2-git-send-email-pramodku@broadcom.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1447905139-16002-2-git-send-email-pramodku@broadcom.com> Sender: linux-kernel-owner@vger.kernel.org To: Pramod Kumar Cc: Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Ray Jui , Scott Branden , Russell King , Linus Walleij , linux-gpio@vger.kernel.org, BCM Kernel Feedback , Jason Uy , Masahiro Yamada , Thomas Gleixner , Laurent Pinchart , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Jonas Gorski List-Id: devicetree@vger.kernel.org On Thu, Nov 19, 2015 at 09:22:13AM +0530, Pramod Kumar wrote: > Add ngpios property to the gpio controller's DT node so that controller > driver extracts total number of in-use gpio lines from DT and removes > dependency on driver. > > Signed-off-by: Pramod Kumar > Reviewed-by: Ray Jui > Reviewed-by: Scott Branden Acked-by: Rob Herring > --- > Documentation/devicetree/bindings/pinctrl/brcm,cygnus-gpio.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pinctrl/brcm,cygnus-gpio.txt b/Documentation/devicetree/bindings/pinctrl/brcm,cygnus-gpio.txt > index 16589fb6..8b1e5d1 100644 > --- a/Documentation/devicetree/bindings/pinctrl/brcm,cygnus-gpio.txt > +++ b/Documentation/devicetree/bindings/pinctrl/brcm,cygnus-gpio.txt > @@ -10,6 +10,9 @@ Required properties: > Define the base and range of the I/O address space that contains the Cygnus > GPIO/PINCONF controller registers > > +- ngpios: > + Total number of in-use slots in GPIO controller > + > - #gpio-cells: > Must be two. The first cell is the GPIO pin number (within the > controller's pin space) and the second cell is used for the following: > @@ -57,6 +60,7 @@ Example: > compatible = "brcm,cygnus-ccm-gpio"; > reg = <0x1800a000 0x50>, > <0x0301d164 0x20>; > + ngpios = <24>; > #gpio-cells = <2>; > gpio-controller; > interrupts = ; > @@ -78,6 +82,7 @@ Example: > gpio_asiu: gpio@180a5000 { > compatible = "brcm,cygnus-asiu-gpio"; > reg = <0x180a5000 0x668>; > + ngpios = <146>; > #gpio-cells = <2>; > gpio-controller; > interrupts = ; > -- > 1.9.1 >