diff mbox series

[v8,06/20] gpiolib: add build option for CDEV v1 ABI

Message ID 20200909102640.1657622-7-warthog618@gmail.com
State Superseded
Headers show
Series gpio: cdev: add uAPI v2 | expand

Commit Message

Kent Gibson Sept. 9, 2020, 10:26 a.m. UTC
Add a build option to allow the removal of the CDEV v1 ABI.

Suggested-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Signed-off-by: Kent Gibson <warthog618@gmail.com>
---

This patch is before the v2 implementation, and is non-functional until
that patch, as some parts of that patch would be written slightly
differently if removing v1 was not considered.
Adding this patch after that would necessitate revisiting the v2 changes,
so this ordering results in two simpler patches.

 drivers/gpio/Kconfig | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Andy Shevchenko Sept. 15, 2020, 10:15 a.m. UTC | #1
On Wed, Sep 9, 2020 at 1:33 PM Kent Gibson <warthog618@gmail.com> wrote:
>

> Add a build option to allow the removal of the CDEV v1 ABI.


Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>


One nit-pick below.

> Suggested-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>

> Signed-off-by: Kent Gibson <warthog618@gmail.com>

> ---

>

> This patch is before the v2 implementation, and is non-functional until

> that patch, as some parts of that patch would be written slightly

> differently if removing v1 was not considered.

> Adding this patch after that would necessitate revisiting the v2 changes,

> so this ordering results in two simpler patches.

>

>  drivers/gpio/Kconfig | 12 ++++++++++++

>  1 file changed, 12 insertions(+)

>

> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig

> index e4debd66d71f..ff9ca836026b 100644

> --- a/drivers/gpio/Kconfig

> +++ b/drivers/gpio/Kconfig

> @@ -82,6 +82,18 @@ config GPIO_CDEV

>

>           If unsure, say Y.

>

> +config GPIO_CDEV_V1

> +       bool "Support GPIO ABI Version 1"

> +       default y

> +       depends on GPIO_CDEV

> +       help

> +         Say Y here to support version 1 of the GPIO CDEV ABI.

> +


> +         This ABI version is deprecated and will be removed in the future.

> +         Please use the latest ABI for new developments.


"will be removed" is not what is going to happen in at least the next
few years. I think something less strong would be better.

> +

> +         If unsure, say Y.

> +

>  config GPIO_GENERIC

>         depends on HAS_IOMEM # Only for IOMEM drivers

>         tristate

> --

> 2.28.0

>



-- 
With Best Regards,
Andy Shevchenko
diff mbox series

Patch

diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index e4debd66d71f..ff9ca836026b 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -82,6 +82,18 @@  config GPIO_CDEV
 
 	  If unsure, say Y.
 
+config GPIO_CDEV_V1
+	bool "Support GPIO ABI Version 1"
+	default y
+	depends on GPIO_CDEV
+	help
+	  Say Y here to support version 1 of the GPIO CDEV ABI.
+
+	  This ABI version is deprecated and will be removed in the future.
+	  Please use the latest ABI for new developments.
+
+	  If unsure, say Y.
+
 config GPIO_GENERIC
 	depends on HAS_IOMEM # Only for IOMEM drivers
 	tristate