* [PATCH] RFC: Do not enable the v1 uAPI by default
@ 2023-11-04 22:54 Linus Walleij
2023-11-05 1:12 ` Kent Gibson
2023-11-06 10:15 ` Andy Shevchenko
0 siblings, 2 replies; 5+ messages in thread
From: Linus Walleij @ 2023-11-04 22:54 UTC (permalink / raw)
To: Bartosz Golaszewski, Andy Shevchenko, Kent Gibson
Cc: linux-gpio, linux-kernel, Linus Walleij
It's been two years since we introduced the v2 uAPI and
the major consumer libgpiod is at v2.1.
What about discouraging the old uAPI?
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
drivers/gpio/Kconfig | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index 913948876c93..65c9914ee3fa 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -85,7 +85,7 @@ config GPIO_CDEV
config GPIO_CDEV_V1
bool "Support GPIO ABI Version 1"
- default y
+ default n
depends on GPIO_CDEV
help
Say Y here to support version 1 of the GPIO CDEV ABI.
@@ -93,8 +93,6 @@ config GPIO_CDEV_V1
This ABI version is deprecated.
Please use the latest ABI for new developments.
- If unsure, say Y.
-
config GPIO_GENERIC
depends on HAS_IOMEM # Only for IOMEM drivers
tristate
---
base-commit: e392ea4d4d00880bf94550151b1ace4f88a4b17a
change-id: 20231103-no-y-uapi1-default-4204e86c04a3
Best regards,
--
Linus Walleij <linus.walleij@linaro.org>
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] RFC: Do not enable the v1 uAPI by default
2023-11-04 22:54 [PATCH] RFC: Do not enable the v1 uAPI by default Linus Walleij
@ 2023-11-05 1:12 ` Kent Gibson
2023-11-05 15:39 ` Linus Walleij
2023-11-06 10:15 ` Andy Shevchenko
1 sibling, 1 reply; 5+ messages in thread
From: Kent Gibson @ 2023-11-05 1:12 UTC (permalink / raw)
To: Linus Walleij
Cc: Bartosz Golaszewski, Andy Shevchenko, linux-gpio, linux-kernel
On Sat, Nov 04, 2023 at 11:54:40PM +0100, Linus Walleij wrote:
> It's been two years since we introduced the v2 uAPI and
> the major consumer libgpiod is at v2.1.
>
Believe it or not, it is nearly three years. But libgpiod support for
it, added in v2.x, is less than one year old, and migrating from libgpiod
v1 to v2 is non-trivial as their APIs are very different.
So I would not be surprised to find that the major consumer of the uAPI
remains users of libgpiod v1.x - which requires the old uAPI.
> What about discouraging the old uAPI?
>
If you want to provide the end user with two years to migrate, and given
that libgpiod is the major consumer, you might want to hold off for
another year.
OTOH, if distros/users want to continue including/using libgpiod v1 they
can always re-enable GPIO_CDEV_V1, so I'm not completely against the idea
- just be aware that it may be more disruptive than you might expect.
Cheers,
Kent.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RFC: Do not enable the v1 uAPI by default
2023-11-05 1:12 ` Kent Gibson
@ 2023-11-05 15:39 ` Linus Walleij
2023-11-06 8:20 ` Bartosz Golaszewski
0 siblings, 1 reply; 5+ messages in thread
From: Linus Walleij @ 2023-11-05 15:39 UTC (permalink / raw)
To: Kent Gibson
Cc: Bartosz Golaszewski, Andy Shevchenko, linux-gpio, linux-kernel
On Sun, Nov 5, 2023 at 2:12 AM Kent Gibson <warthog618@gmail.com> wrote:
> If you want to provide the end user with two years to migrate, and given
> that libgpiod is the major consumer, you might want to hold off for
> another year.
OK then, just testing the waters.
Probably we need to encourage libgpiod 2.x a bit.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RFC: Do not enable the v1 uAPI by default
2023-11-05 15:39 ` Linus Walleij
@ 2023-11-06 8:20 ` Bartosz Golaszewski
0 siblings, 0 replies; 5+ messages in thread
From: Bartosz Golaszewski @ 2023-11-06 8:20 UTC (permalink / raw)
To: Linus Walleij, Phil Howard
Cc: Kent Gibson, Andy Shevchenko, linux-gpio, linux-kernel
On Sun, Nov 5, 2023 at 4:39 PM Linus Walleij <linus.walleij@linaro.org> wrote:
>
> On Sun, Nov 5, 2023 at 2:12 AM Kent Gibson <warthog618@gmail.com> wrote:
>
> > If you want to provide the end user with two years to migrate, and given
> > that libgpiod is the major consumer, you might want to hold off for
> > another year.
>
> OK then, just testing the waters.
>
> Probably we need to encourage libgpiod 2.x a bit.
>
Thanks to Phil Howard's work on python bindings, they should soon be
more widely adopted. They are now the first thing that pops up in
google when searching for "libgpiod python" and now have a proper
homepage in pypi. Running 'pip3 install gpiod' will now work on
systems with no libgpiod installed too as it's vendored with libgpiod
v2.1 sources. I also plan to finally get around to using breathe and
exhale to produce proper sphinx docs. So hopefully we'll start getting
traction soon.
Bart
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RFC: Do not enable the v1 uAPI by default
2023-11-04 22:54 [PATCH] RFC: Do not enable the v1 uAPI by default Linus Walleij
2023-11-05 1:12 ` Kent Gibson
@ 2023-11-06 10:15 ` Andy Shevchenko
1 sibling, 0 replies; 5+ messages in thread
From: Andy Shevchenko @ 2023-11-06 10:15 UTC (permalink / raw)
To: Linus Walleij; +Cc: Bartosz Golaszewski, Kent Gibson, linux-gpio, linux-kernel
On Sat, Nov 04, 2023 at 11:54:40PM +0100, Linus Walleij wrote:
> It's been two years since we introduced the v2 uAPI and
> the major consumer libgpiod is at v2.1.
> What about discouraging the old uAPI?
...
> - default y
> + default n
'n' _is_ the default 'default', just drop a line.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-11-06 10:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-04 22:54 [PATCH] RFC: Do not enable the v1 uAPI by default Linus Walleij
2023-11-05 1:12 ` Kent Gibson
2023-11-05 15:39 ` Linus Walleij
2023-11-06 8:20 ` Bartosz Golaszewski
2023-11-06 10:15 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox