[PATCH v1 7/7] gpiolib: Get rid of not used anymore gpio_request_one()
Andy Shevchenko <[email protected]> Wed, 15 Jul 2026 20:46:47 +0200
| Newsgroups | org.kernel.vger.linux-mips,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-doc,org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media |
|---|---|
| Message-ID | <[email protected]> |
No more users. Signed-off-by: Andy Shevchenko <[email protected]> --- Documentation/driver-api/gpio/consumer.rst | 2 +- drivers/gpio/gpiolib-legacy.c | 31 ---------------------- include/linux/gpio/legacy.h | 8 ------ 3 files changed, 1 insertion(+), 40 deletions(-) diff --git a/Documentation/driver-api/gpio/consumer.rst b/Documentation/driver-api/gpio/consumer.rst index bb3366047fad..03edd9472f07 100644 --- a/Documentation/driver-api/gpio/consumer.rst +++ b/Documentation/driver-api/gpio/consumer.rst @@ -463,7 +463,7 @@ and vice-versa:: The GPIO number returned by desc_to_gpio() can safely be used as a parameter of the gpio\_*() functions for as long as the GPIO descriptor `desc` is not freed. All the same, a GPIO number passed to gpio_to_desc() must first be properly -acquired using e.g. gpio_request_one(), and the returned GPIO descriptor is only +acquired using e.g. gpio_request(), and the returned GPIO descriptor is only considered valid until that GPIO number is released using gpio_free(). Freeing a GPIO obtained by one API with the other API is forbidden and an diff --git a/drivers/gpio/gpiolib-legacy.c b/drivers/gpio/gpiolib-legacy.c index ef3f2ef30cf2..c2b04605b203 100644 --- a/drivers/gpio/gpiolib-legacy.c +++ b/drivers/gpio/gpiolib-legacy.c @@ -21,37 +21,6 @@ void gpio_free(unsigned gpio) } EXPORT_SYMBOL_GPL(gpio_free); -/** - * gpio_request_one - request a single GPIO with initial configuration - * @gpio: the GPIO number - * @flags: GPIO configuration as specified by GPIOF_* - * @label: a literal description string of this GPIO - * - * **DEPRECATED** This function is deprecated and must not be used in new code. - * - * Returns: - * 0 on success, or negative errno on failure. - */ -int gpio_request_one(unsigned gpio, unsigned long flags, const char *label) -{ - int err; - - err = gpio_request(gpio, label); - if (err) - return err; - - if (flags & GPIOF_IN) - err = gpio_direction_input(gpio); - else - err = gpio_direction_output(gpio, !!(flags & GPIOF_OUT_INIT_HIGH)); - - if (err) - gpio_free(gpio); - - return err; -} -EXPORT_SYMBOL_GPL(gpio_request_one); - /* * **DEPRECATED** This function is deprecated and must not be used in new code. */ diff --git a/include/linux/gpio/legacy.h b/include/linux/gpio/legacy.h index 557ef635935e..d4273162ed13 100644 --- a/include/linux/gpio/legacy.h +++ b/include/linux/gpio/legacy.h @@ -82,8 +82,6 @@ static inline int gpio_to_irq(unsigned gpio) return gpiod_to_irq(gpio_to_desc(gpio)); } -int gpio_request_one(unsigned gpio, unsigned long flags, const char *label); - int devm_gpio_request_one(struct device *dev, unsigned gpio, unsigned long flags, const char *label); @@ -104,12 +102,6 @@ static inline int gpio_request(unsigned gpio, const char *label) return -ENOSYS; } -static inline int gpio_request_one(unsigned gpio, - unsigned long flags, const char *label) -{ - return -ENOSYS; -} - static inline void gpio_free(unsigned gpio) { might_sleep(); -- 2.50.1