From: Bartosz Golaszewski <brgl@bgdev.pl>
To: Kent Gibson <warthog618@gmail.com>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Jack Winch <sunt.un.morcov@gmail.com>,
Helmut Grohne <helmut.grohne@intenta.de>,
Linus Walleij <linus.walleij@linaro.org>
Cc: linux-gpio@vger.kernel.org,
Bartosz Golaszewski <bgolaszewski@baylibre.com>
Subject: [libgpiod][PATCH 00/14] treewide: start shaving off cruft for v2.0
Date: Thu, 10 Dec 2020 14:23:01 +0100 [thread overview]
Message-ID: <20201210132315.5785-1-brgl@bgdev.pl> (raw)
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
The following series removes a lot of interfaces that were deemed overkill
in libgpiod and the removal of which was suggested to me before proceeding
with the new API.
This leaves a couple holes in the library but we'll follow them up with
more improvements all over the tree. We'll create a new object called
gpiod_request for dealing with line requests of arbitrary size. We'll
probably remove the the bulk objects from bindings and eventually we'll
switch to using the v2 kernel uAPI.
Andy - a note for you: I know you're always very thorough in your reviews
but in this case let's consider this series preparing a construction zone
for the new API. Please don't nitpick too much. :)
Bartosz Golaszewski (14):
bindings: cxx: check for error from gpiod_line_bulk_new()
build: drop the message about tests having been built successfully
core: export gpiod_is_gpiochip_device()
bulk: drop the limit on the max number of lines
core: drop line iterators
treewide: kill opening chips by label
API: move gpiod_line_get_chip() to line attributes section
core: kill gpiod_line_close_chip()
core: kill gpiod_line_get()
treewide: kill global line lookup
treewide: kill find_lines()
core: rework gpiod_chip_find_line()
build: add a configure switch for building examples
core: kill chip iterators
bindings/cxx/Makefile.am | 8 +-
bindings/cxx/chip.cpp | 47 +--
bindings/cxx/examples/Makefile.am | 4 +-
bindings/cxx/examples/gpiodetectcxx.cpp | 13 +-
bindings/cxx/examples/gpiofindcxx.cpp | 18 +-
bindings/cxx/examples/gpioinfocxx.cpp | 43 ++-
bindings/cxx/gpiod.hpp | 147 +-------
bindings/cxx/iter.cpp | 95 +----
bindings/cxx/line.cpp | 15 -
bindings/cxx/line_bulk.cpp | 24 +-
bindings/cxx/tests/tests-chip.cpp | 62 ++--
bindings/cxx/tests/tests-iter.cpp | 37 --
bindings/cxx/tests/tests-line.cpp | 19 -
bindings/python/Makefile.am | 10 +-
bindings/python/examples/gpiodetect.py | 12 +-
bindings/python/examples/gpiofind.py | 15 +-
bindings/python/examples/gpioinfo.py | 33 +-
bindings/python/gpiodmodule.c | 473 +++++++-----------------
bindings/python/tests/gpiod_py_test.py | 100 +----
configure.ac | 12 +
include/gpiod.h | 258 ++-----------
lib/Makefile.am | 2 +-
lib/core.c | 12 +-
lib/helpers.c | 176 +++------
lib/iter.c | 171 ---------
tests/Makefile.am | 13 -
tests/gpiod-test.h | 4 -
tests/tests-bulk.c | 9 -
tests/tests-chip.c | 97 +----
tests/tests-iter.c | 123 ------
tests/tests-line.c | 52 ---
tools/gpiodetect.c | 29 +-
tools/gpiofind.c | 34 +-
tools/gpioinfo.c | 41 +-
tools/gpiomon.c | 5 +-
tools/tools-common.c | 15 +
tools/tools-common.h | 3 +
37 files changed, 557 insertions(+), 1674 deletions(-)
delete mode 100644 lib/iter.c
delete mode 100644 tests/tests-iter.c
--
2.29.1
next reply other threads:[~2020-12-10 13:24 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-10 13:23 Bartosz Golaszewski [this message]
2020-12-10 13:23 ` [libgpiod][PATCH 01/14] bindings: cxx: check for error from gpiod_line_bulk_new() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 02/14] build: drop the message about tests having been built successfully Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 03/14] core: export gpiod_is_gpiochip_device() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 04/14] bulk: drop the limit on the max number of lines Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 05/14] core: drop line iterators Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 06/14] treewide: kill opening chips by label Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 07/14] API: move gpiod_line_get_chip() to line attributes section Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 08/14] core: kill gpiod_line_close_chip() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 09/14] core: kill gpiod_line_get() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 10/14] treewide: kill global line lookup Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 11/14] treewide: kill find_lines() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 12/14] core: rework gpiod_chip_find_line() Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 13/14] build: add a configure switch for building examples Bartosz Golaszewski
2020-12-10 13:23 ` [libgpiod][PATCH 14/14] core: kill chip iterators Bartosz Golaszewski
2020-12-10 13:56 ` [libgpiod][PATCH 00/14] treewide: start shaving off cruft for v2.0 Andy Shevchenko
2020-12-11 8:38 ` Bartosz Golaszewski
2020-12-11 14:31 ` Andy Shevchenko
2020-12-11 14:33 ` Bartosz Golaszewski
2020-12-11 14:58 ` Andy Shevchenko
2020-12-11 15:06 ` Bartosz Golaszewski
2020-12-14 15:02 ` Bartosz Golaszewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201210132315.5785-1-brgl@bgdev.pl \
--to=brgl@bgdev.pl \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bgolaszewski@baylibre.com \
--cc=geert@linux-m68k.org \
--cc=helmut.grohne@intenta.de \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=sunt.un.morcov@gmail.com \
--cc=warthog618@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).