Messages in this thread | | | Date | Tue, 19 Nov 2013 10:24:22 +0100 | Subject | Re: [PATCH v1.2] gpiolib: append SFI helpers for GPIO API | From | Linus Walleij <> |
| |
On Wed, Jun 5, 2013 at 3:58 PM, Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> To support some (legacy) firmwares and platforms let's make life easier for > their customers. > > This patch extracts SFI GPIO API from arch/x86/platform/mrst/mrst.c. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
So since this patch was ACKed the world has changed a bit and now I want new changes (or maybe I was tired and not paying enough attention at the time).
(...) > +int sfi_get_gpio_by_name(const char *name) > +{ > + struct sfi_gpio_table_entry *pentry = sfi_gpio_table; > + int i; > + > + if (!pentry) > + return -EINVAL; > + > + for (i = 0; i < sfi_gpio_num_entry; i++, pentry++) { > + if (!strncmp(name, pentry->pin_name, SFI_NAME_LEN)) > + return pentry->pin_no; > + } > + > + return -ENODEV; > +} > +EXPORT_SYMBOL_GPL(sfi_get_gpio_by_name);
Last merge window we merged the GPIO descriptor API and this is now the recommended way to handle GPIOs and it is also deployed into the ACPI and DT implementations.
So I'd like the signature of this function changed to return a GPIO descriptor rather than an int so we don't stockpile more stuff to refactor.
i.e.: struct gpio_desc *sfi_get_gpio_by_name(const char *name);
--- /dev/null > +++ b/include/linux/sfi_gpio.h
Maybe that header could move to <linux/gpio/sfi.h> instead. Alexandre what do you think?
Yours, Linus Walleij
| |