Skip to content

bindings: cxx: fix potential glitch in gpiosetcxx

Bartosz Golaszewski linux-gpio 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2bindings: cxx: provide request_builder::set_output_values()bindings/cxx/gpiodcxx/request-builder.hpp bindings/cxx/request-builder.cpp+16 -00 replies
2/2bindings: cxx: examples: fix potential glitch in gpiosetcxxbindings/cxx/examples/gpiosetcxx.cpp+1 -20 replies

Reviews trailers per patch

None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchKent Gibson
SeriesR-b
1/2
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [libgpiod][PATCH 0/2] bindings: cxx: fix potential glitch in gpiosetcxx

From: Bartosz Golaszewski <[email protected]>

Kent,

This is an alternative approach to the problem you raised with your gpioset patches. It adds a new helper method that allows to set all output values at request-time using the line_config's set_output_values() method.

Please take a look and let me know what you think.

Bart

Bartosz Golaszewski (2):
bindings: cxx: provide request_builder::set_output_values()
bindings: cxx: examples: fix potential glitch in gpiosetcxx

 bindings/cxx/examples/gpiosetcxx.cpp      | 3 +--
 bindings/cxx/gpiodcxx/request-builder.hpp | 8 ++++++++
 bindings/cxx/request-builder.cpp          | 8 ++++++++
 3 files changed, 17 insertions(+), 2 deletions(-)
Signature · 1 line
2.39.2