pon., 18 lis 2019 o 15:09 Kent Gibson <warthog618@xxxxxxxxx> napisał(a): > > On Mon, Nov 18, 2019 at 02:50:57PM +0100, Bartosz Golaszewski wrote: > > pt., 15 lis 2019 o 15:44 Kent Gibson <warthog618@xxxxxxxxx> napisał(a): > > > > > > This patch series adds support for changes to the GPIO uAPI that are on > > > track to be included in the v5.5 kernel. There are two components to the > > > uAPI changes - the addition of bias flags and a new SET_CONFIG ioctl. This > > > series adds support to the libgpiod API, and to both C++ and Python > > > bindings, for both of those components. > > > > > > The libgpiod tools are also updated, where appropriate, to support the bias > > > flags. > > > > > > There are a few additional patches that serve to restructure the code to > > > simplify subsequent patches, or to fix minor problems discovered during > > > development. These patches are generally adjacent to the main patch most > > > relevant to that patch. > > > > > > The series is based on the current libgpiod master@9ed02fc. > > > > > > Kent Gibson (19): > > > core: move request flag to handle flag conversion into a separate > > > function > > > API: add support for bias flags > > > core: fix misspelling of parameter > > > tests: add tests for bias flags > > > bindings: cxx: drop noexcept from direction and active_state > > > bindings: cxx: initialise bitset with integer instead of string > > > bindings: cxx: add support for bias flags > > > bindings: cxx: tests: add tests for bias flags > > > bindings: python: add support for bias flags > > > bindings: python: tests: add tests for bias flags > > > API: add support for SET_CONFIG > > > tests: add tests for SET_CONFIG > > > core: allow gpiod_line_set_value_bulk to accept null values > > > bindings: cxx: add support for SET_CONFIG > > > bindings: cxx: tests: add tests for SET_CONFIG methods > > > bindings: python: add support for SET_CONFIG > > > bindings: python: tests: add tests for SET_CONFIG methods > > > tools: add support for bias flags > > > treewide: change "correspond with" to "correspond to" > > > > > > bindings/cxx/gpiod.hpp | 85 +++- > > > bindings/cxx/line.cpp | 60 ++- > > > bindings/cxx/line_bulk.cpp | 95 ++++- > > > bindings/cxx/tests/tests-line.cpp | 215 ++++++++++ > > > bindings/python/gpiodmodule.c | 469 +++++++++++++++++++++- > > > bindings/python/tests/gpiod_py_test.py | 254 ++++++++++++ > > > include/gpiod.h | 303 +++++++++++++- > > > lib/core.c | 250 ++++++++++-- > > > lib/ctxless.c | 115 +++++- > > > tests/mockup/gpio-mockup.c | 2 +- > > > tests/tests-ctxless.c | 64 ++- > > > tests/tests-event.c | 120 ++++++ > > > tests/tests-line.c | 522 ++++++++++++++++++++++++- > > > tools/gpioget.c | 24 +- > > > tools/gpiomon.c | 28 +- > > > tools/gpioset.c | 26 +- > > > 16 files changed, 2561 insertions(+), 71 deletions(-) > > > > > > -- > > > 2.24.0 > > > > > > > Hi Kent, > > > > the series looks quite good. I addressed some obvious things. I'll > > review v2 more in detail, but honestly, I don't think there'll be a > > lot of issues. > > > > Would you mind if I applied patches 1, 3, 5, 6, 13 & 19 right away? > > They fix existing problems, so there's no need to carry them over to > > subsequent iterations of the series. > > > > I don't have a problem with that. > Done. You should be able to rebase on top of my master branch. Let's hope there are no conflicts - I had to fix up a couple details. Bart > Cheers, > Kent.