From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sebastian Andrzej Siewior Subject: Re: [PATCH 3/3] ARM: dts: socfpga: add gpio pieces Date: Sat, 22 Mar 2014 12:28:17 +0100 Message-ID: <532D73D1.7020408@linutronix.de> References: <1395345324-18299-1-git-send-email-bigeasy@linutronix.de> <1395345324-18299-4-git-send-email-bigeasy@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from www.linutronix.de ([62.245.132.108]:54912 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750878AbaCVL2U (ORCPT ); Sat, 22 Mar 2014 07:28:20 -0400 In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: delicious quinoa Cc: Alan Tull , Linus Walleij , Alexandre Courbot , "linux-gpio@vger.kernel.org" , linux-kernel , Dinh Nguyen , "devicetree@vger.kernel.org" On 03/21/2014 11:15 PM, delicious quinoa wrote: > snps,nr-gpios = <27>; > > As noted on other thread, gpio2 is 27 wide, despite what the > documentation says. When I made that change and remove your other two > patches the gpios worked for me on a cyclone5 devkit board. > > So if you fix the "#gpio-cells = <2>;" for all 3 gpios and fix > "snps,nr-gpios = <27>;" for gpio2, then this one patch looks good to > me. With these changes, Okay will do. And I also add a note that the reference manual is wrong here. > > Acked-by: Alan Tull > > Alan Tull > aka > delicous quinoa Sebastian