From mboxrd@z Thu Jan 1 00:00:00 1970 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> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hello, On Sun, 20 May 2018 19:17:04 +0300, Andy Shevchenko wrote: > >> Though, I completely dislike "rdy" name of GPIO. Where is it documente= d? =20 > > > > No documentation files for Amstrad Delta nor for its NAND driver specif= ically > > 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 discu= ssed. =20 >=20 > > To be honest, I don't like it much either, but I'm just using it instea= d of > > inventing something new. =20 >=20 > 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=C3=A8l