From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.bootlin.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1g7kUM-0000R3-Ba for linux-mtd@lists.infradead.org; Wed, 03 Oct 2018 17:01:06 +0000 Date: Wed, 3 Oct 2018 18:58:15 +0200 From: Boris Brezillon To: Ricardo Ribalda Delgado Cc: Zhouyang Jia , LKML , Marek Vasut , linux-mtd@lists.infradead.org, Richard Weinberger , Brian Norris , David Woodhouse Subject: Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod Message-ID: <20181003185815.23b40832@bbrezillon> In-Reply-To: References: <20181001124351.31615-1-ricardo.ribalda@gmail.com> <20181001124351.31615-6-ricardo.ribalda@gmail.com> <20181003171114.51ab47b8@bbrezillon> <20181003171727.145e7e41@bbrezillon> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Ricardo, On Wed, 3 Oct 2018 17:56:03 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Wed, Oct 3, 2018 at 5:17 PM Boris Brezillon > wrote: > > > > On Wed, 3 Oct 2018 17:11:14 +0200 > > Boris Brezillon wrote: > > > > > Hi Ricardo, > > > > > > On Mon, 1 Oct 2018 14:43:49 +0200 > > > Ricardo Ribalda Delgado wrote: > > > > > > > @@ -248,14 +252,19 @@ static int gpio_flash_probe(struct platform_device *pdev) > > > > > > > > i = 0; > > > > do { > > > > - if (devm_gpio_request(&pdev->dev, state->gpio_addrs[i], > > > > - DRIVER_NAME)) { > > > > + unsigned int *gpio_id = (unsigned int *)gpios->start; > > > > + > > > > + if (devm_gpio_request_one(&pdev->dev, gpio_id[i], GPIOD_OUT_LOW, > > > > + DRIVER_NAME)) { > > > > dev_err(&pdev->dev, "failed to request gpio %d\n", > > > > - state->gpio_addrs[i]); > > > > + gpio_id[i]); > > > > return -EBUSY; > > > > } > > > > - gpio_direction_output(state->gpio_addrs[i], 0); > > > > - } while (++i < state->gpio_count); > > > > + > > > > + state->gpios->desc[i] = gpio_to_desc(gpio_id[i]); > > > > + if (!state->gpios->desc[i]) > > > > + return -EINVAL; > > > > + } while (++i < state->gpios->ndescs); > > > > > > Actually, I was thinking about using devm_gpiod_get_array() here and > > > defining a gpio lookup table in the board file registering the device. > > > This way, adding support for DT based parsing is transparent. > > > > It's actually easier than I thought since no one is registering such a > > device, so all you have to do is call devm_gpiod_get_array() and have a > > struct gpio_descs pointer in struct async_state. > > That is what I do in patch 8/8 for DT based parsing. Well, yes, except you do it differently for the !DT case, while I'm suggesting to use the same patch for both DT and !DT. > > I am am doing the gpio_to_desc and other to maintain compatibility > with old platform board files (in and out tree). I do care about keeping in-tree users functional (which is why I suggested to declare gpio lookup tables in the first place), but keeping out-of-tree code functional is on a best-effort basis. If it breaks because we have to rework an internal API then that's not our fault. Actually, that's one of the reason we push people to upstream their code => their maintenance burden is greatly reduced once the code has reached mainline. > > I think is important to maintain that compatibility, but you decide ;) And my decision is, keep the code as simple as possible even if it breaks out of tree users. Regards, Boris