* Re: [libgpiod v2][PATCH] treewide: implement v2 API
[not found] <20220602112656.31261-1-brgl@bgdev.pl>
@ 2022-06-02 11:52 ` Kent Gibson
2022-06-02 13:34 ` Bartosz Golaszewski
0 siblings, 1 reply; 2+ messages in thread
From: Kent Gibson @ 2022-06-02 11:52 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Linus Walleij, Andy Shevchenko, Darrien, Viresh Kumar, Jiri Benc,
Joel Savitz, Alexander Stein, David Kozub, Jan Kundrát,
Michael Beach, Jack Winch, Geert Uytterhoeven, linux-gpio,
Ben Hutchings
On Thu, Jun 02, 2022 at 01:26:56PM +0200, Bartosz Golaszewski wrote:
> This is the main changeset implementing the new libgpiod API for the core
> C library as well as C++ and Python bindings. It's actually the effect of
> squashing all the patches we've had in the next/libgpiod-v2.0 branch (and
> that have been developed and reviewed over the course of several months)
> so that the master branch can remain bisectable.
>
> Big thanks to Kent Gibson for his reviews.
>
> Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> [Ben: fix format specifier for uint64_t]
> Signed-off-by: Ben Hutchings <ben.hutchings@mind.be>
> [Viresh: Fix ioctl number for gpiod_line_request_reconfigure_lines()]
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> [Kent: splitting of chip_info out of chip, documentation tweaks and some renaming]
> Signed-off-by: Kent Gibson <warthog618@gmail.com>
> ---
> There were no reviews so far for the python bindings but I'm generally
> happy with them and think that the contents of next/libgpiod-2.0 branch
> are mature enough to get them into master and continue the work incrementally
> there. So this is the amalgamation of all the work done so far on libgpiod v2
> that I'd like to apply soon.
>
My apologies for not getting back to you earlier wrt the python
bindings.
When I try them I get:
$ python3
Python 3.9.2 (default, Feb 28 2021, 17:03:44)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import gpiod
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "exception.c", line 1, in <module>
NameError: __build_class__ not found
>>>
I may be doing something dopey, but building v1.6.3 on the same
platform (a clean Debian 11.3 install with a v5.18 kernel with gpiosim
and gpio-mockup) works fine.
Haven't had a chance to look into it further...
Any ideas?
Cheers,
Kent.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [libgpiod v2][PATCH] treewide: implement v2 API
2022-06-02 11:52 ` [libgpiod v2][PATCH] treewide: implement v2 API Kent Gibson
@ 2022-06-02 13:34 ` Bartosz Golaszewski
0 siblings, 0 replies; 2+ messages in thread
From: Bartosz Golaszewski @ 2022-06-02 13:34 UTC (permalink / raw)
To: Kent Gibson
Cc: Linus Walleij, Andy Shevchenko, Darrien, Viresh Kumar, Jiri Benc,
Joel Savitz, Alexander Stein, David Kozub, Jan Kundrát,
Michael Beach, Jack Winch, Geert Uytterhoeven,
open list:GPIO SUBSYSTEM, Ben Hutchings
On Thu, Jun 2, 2022 at 1:52 PM Kent Gibson <warthog618@gmail.com> wrote:
>
> On Thu, Jun 02, 2022 at 01:26:56PM +0200, Bartosz Golaszewski wrote:
> > This is the main changeset implementing the new libgpiod API for the core
> > C library as well as C++ and Python bindings. It's actually the effect of
> > squashing all the patches we've had in the next/libgpiod-v2.0 branch (and
> > that have been developed and reviewed over the course of several months)
> > so that the master branch can remain bisectable.
> >
> > Big thanks to Kent Gibson for his reviews.
> >
> > Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> > [Ben: fix format specifier for uint64_t]
> > Signed-off-by: Ben Hutchings <ben.hutchings@mind.be>
> > [Viresh: Fix ioctl number for gpiod_line_request_reconfigure_lines()]
> > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> > [Kent: splitting of chip_info out of chip, documentation tweaks and some renaming]
> > Signed-off-by: Kent Gibson <warthog618@gmail.com>
> > ---
> > There were no reviews so far for the python bindings but I'm generally
> > happy with them and think that the contents of next/libgpiod-2.0 branch
> > are mature enough to get them into master and continue the work incrementally
> > there. So this is the amalgamation of all the work done so far on libgpiod v2
> > that I'd like to apply soon.
> >
>
> My apologies for not getting back to you earlier wrt the python
> bindings.
> When I try them I get:
>
> $ python3
> Python 3.9.2 (default, Feb 28 2021, 17:03:44)
> [GCC 10.2.1 20210110] on linux
> Type "help", "copyright", "credits" or "license" for more information.
> >>> import gpiod
> Traceback (most recent call last):
> File "<stdin>", line 1, in <module>
> File "exception.c", line 1, in <module>
> NameError: __build_class__ not found
> >>>
>
> I may be doing something dopey, but building v1.6.3 on the same
> platform (a clean Debian 11.3 install with a v5.18 kernel with gpiosim
> and gpio-mockup) works fine.
> Haven't had a chance to look into it further...
>
> Any ideas?
I ran Debian 11.3 with docker and can confirm this issue. I don't see
it on Ubuntu 22.04. Interesting. I'll look into it.
Bart
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-06-02 13:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20220602112656.31261-1-brgl@bgdev.pl>
2022-06-02 11:52 ` [libgpiod v2][PATCH] treewide: implement v2 API Kent Gibson
2022-06-02 13:34 ` Bartosz Golaszewski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.