From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752470AbcCGJm4 (ORCPT ); Mon, 7 Mar 2016 04:42:56 -0500 Received: from down.free-electrons.com ([37.187.137.238]:33757 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751938AbcCGJmu (ORCPT ); Mon, 7 Mar 2016 04:42:50 -0500 Date: Mon, 7 Mar 2016 10:38:55 +0100 From: Boris Brezillon To: Roger Quadros Cc: , , devicetree@vger.kernel.org, nsekhar@ti.com, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, ezequiel@vanguardiasur.com.ar, javier@dowhile0.org, linux-omap@vger.kernel.org, dwmw2@infradead.org, fcooper@ti.com Subject: Re: [PATCH v5 13/26] memory: omap-gpmc: Support general purpose input for WAITPINs Message-ID: <20160307103855.57720146@bbrezillon> In-Reply-To: <20160307103440.4b9d7a55@bbrezillon> References: <1455916548-3441-1-git-send-email-rogerq@ti.com> <1455916548-3441-14-git-send-email-rogerq@ti.com> <20160307103440.4b9d7a55@bbrezillon> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.27; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 7 Mar 2016 10:34:40 +0100 Boris Brezillon wrote: > Hi Roger, > > On Fri, 19 Feb 2016 23:15:35 +0200 > Roger Quadros wrote: > > > OMAPs can have 2 to 4 WAITPINs that can be used as general purpose > > input if not used for memory wait state insertion. > > > > The first user will be the OMAP NAND chip to get the NAND > > read/busy status using gpiolib. > > Just a comment on this approach. Why do you need to exposed native R/B > pins as GPIOs? I mean, other NAND controllers are supporting R/B > detection using dedicated logic, and they do not exposed those pins a > plain GPIOs. Have you considered adding another property (rb-native ?) Just had a look at the sunxi-nand binding, and we chose "allwinner,rb" for this native RB logic. So "ti,rb" would be the equivalent for the gpmc driver. -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com