From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752678AbeETRZ2 convert rfc822-to-8bit (ORCPT ); Sun, 20 May 2018 13:25:28 -0400 Received: from mail.bootlin.com ([62.4.15.54]:59887 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751521AbeETRZ0 (ORCPT ); Sun, 20 May 2018 13:25:26 -0400 Date: Sun, 20 May 2018 19:25:23 +0200 From: Miquel Raynal To: Andy Shevchenko Cc: Janusz Krzysztofik , Aaro Koskinen , Tony Lindgren , Richard Weinberger , Mark Brown , Dmitry Torokhov , Liam Girdwood , Linux Kernel Mailing List , Peter Ujfalusi , Boris Brezillon , Tomi Valkeinen , "open list:MEMORY TECHNOLOGY..." , linux-arm Mailing List , Linux OMAP Mailing List , Jarkko Nikula Subject: Re: [PATCH 5/6] mtd: rawnand: ams-delta: use GPIO lookup table Message-ID: <20180520192523.32dde632@xps13> In-Reply-To: References: <20180518210954.29044-1-jmkrzyszt@gmail.com> <5456625.lDWjtgZygK@z50> <3505791.x6nWYTo8eb@z50> Organization: Bootlin X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On Sun, 20 May 2018 19:17:04 +0300, Andy Shevchenko wrote: > >> Though, I completely dislike "rdy" name of GPIO. Where is it documented? > > > > No documentation files for Amstrad Delta nor for its NAND driver specifically > > exist under Documentation/. However, there exist some for generic GPIO NAND > > driver where the pin name "rdy" is used explicitly: > > Documentation/driver-api/gpio/drivers-on-gpio.rst > > Documentation/devicetree/bindings/mtd/gpio-control-nand.txt > > You can find that mnemonic used across drivers/mtd/nand/, standalone or as a > > suffix, including the Amstrad Delta NAND driver before the change discussed. > > > To be honest, I don't like it much either, but I'm just using it instead of > > inventing something new. > > OK, that's what I was looking for. Since it's already in use and > documented, then it's fine for me. Do we actually have the possibility to rename this gpio? I guess no since it would break DT backward compatibility. Otherwise it would have been more descriptive to call it something like 'gpio-rb'. Anyway, if you find the time, documentation for this controller would be welcome! Thanks, Miquèl