gpioset: include <poll.h> instead of <sys/poll.h>

Just like we did in core.c: don't include <sys/poll.h> directly.

Follow the manual and include <poll.h>.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: use AC_FUNC_STRERROR_R to check the version of strerror_r()

Define _GNU_SOURCE globally in configure.ac and use the
AC_FUNC_STRERROR_R macro to figure out what version of strerror_r() is
provided. Some libraries (e.g. musl) only provide the POSIX version
even if we explicitly request GNU extensions.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: documentation updates

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: allow to read values of lines configured for events

The kernel allows us to read values of lines on which we're listening
for events, but libgpiod bails out if the user tries to do it.

Modify gpiod_line_get_value_bulk() to select the correct file
descriptor, depending on how the line object is configured and
allow reading values for both normally exported and monitored lines.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: include <poll.h> instead of <sys/poll.h>

<sys/poll.h> is an internal C library header, which shouldn't be
included directly. <poll.h> should be included instead, as advertised
in poll(2).

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
[Bartosz: tweaked the commit message]
Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiod.h: include missing <time.h> include

gpiod.h uses "struct timespec", but forgets to include the <time.h>
header which defines "struct timespec". This causes a build error with
the musl C library:

In file included from core.c:11:0:
../../include/gpiod.h:232:49: warning: ‘struct timespec’ declared inside parameter list
 typedef int (*gpiod_event_cb)(int, const struct timespec *, void *);

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
[Bartosz: tweaked the commit message]
Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: add helpers for printing common errors

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: tweak error messages

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: don't check stdint.h

We probably have it if AC_HEADER_STDC succeeded.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: don't check headers/functions unnecessarily

Only check functions & headers required by tools if we're actually
building them.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: check sys/poll.h availability

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: check ppoll() availability

We're using ppoll() for events - not poll(). Check the right syscall.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: fix the required version of kernel headers

The project actually needs kernel headers >= v4.8 to compile.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: fix shortops

The short option for --silent is missing in gpiomon. Add it.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: don't try to process docs if doxygen is not installed

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

libgpiod: start 0.2 development cycle

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

libgpiod v0.1

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: add examples

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: add the autoconf version needed to compile the project

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: add bullet points to the tool list

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: the introduction is not a code block

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: rename to README.md for nice github cover

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

README: update

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: constify function arguments that will not be modified

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: modify the output format

Drop unnecessary strings. Add offset info (if we ever wanted to allow
monitoring multiple lines at once while staying backwards compatible).

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: check if offset is not negative

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: name the iterator functions consistently

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: skip chips and lines on error in gpiod_line_find_by_name()

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: new options

Add new options to gpiomon:
  - allow to specify the number of events after which the program
    should exit
  - allow to silence the output
  - allow to only monitor certain types of events

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: arrange local arguments for better readability

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: check for daemon() and signalfd() functions

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: allow to specify the consumer in the simple event loop

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: allow to specify the consumer in simple set value routines

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: allow to specify the consumer in simple get value routines

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: tweak error messages

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: fix help text

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: new option

Add an option allowing the program to detach from the controlling
terminal after setting values. This is useful for pulled-down/up pins
that we want to keep exported.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: new mode of operation

Add a new mode: 'signal', which makes the program set GPIO values and
then wait until it receives SIGTERM or SIGINT.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: major rework

Allow setting multiple lines at once and add different modes
of operation.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

tools-common: make ARRAY_SIZE() a common macro

Move ARRAY_SIZE() out of gpioinfo.c and place it in tools-common.h.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: add a typedef for the set value callback

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: new error number

Introduce an error number indicating that the caller tried to request
too many lines at once. This is only relevant for the 'simple' API, as
the bulk operations are limited by the bulk buffer size.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioget: allow reading multiple lines at once

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: new set/get value helpers

Implement simple set/get value routines for multiple lines.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioinfo: readability tweaks

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

tools: tweak help text

Also: stop processing the command line after a first non-option is
encountered in all tools.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: don't generate class diagrams

This is only useful for C++. Maybe we'll restore it once we have C++
bindings done.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: document static inline functions

Configure doxygen to document static inline functions from gpiod.h
as well. While we're at it: add a PREDEFINED option to mark functions
in the header that are not part of the API.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: pass the EXTRA_VERSION to doxygen as well

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: fix Doxyfile

The header path was not updated in the Doxyfile after recent directory
structure changes.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

event: new event request helpers

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

libgpiod: start 0.1 development cycle

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: add EXTRA_VERSION to the version string

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: don't use the m4 directory

This fixes the buildroot build.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: create an organized directory structure

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

doc: doxygen: set the project version from Makefile

Drop the PROJECT_NUMBER option from the Doxyfile and pass the version
to doxygen via command-line when invoking it from Makefile.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: drop redundant version variables

Just use the variable defined by AC_INIT().

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: remove TODO from the repo - track features using github

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

configure: drop redundant AC_HEADER_STDC

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: drop the aclocal flags

We don't use the m4 local directory for now.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: update

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: add a configure option for tools

This allows to build the tools conditionally, depending
on the configuration.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: check if basename() is available

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioset: coding style fix

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: update

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

tools: add the --version argument

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

tools-common: add a function displaying the version info

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: add a function for accessing the library version string

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

build: add library versioning

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

make: readability tweaks

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

make: compile the library with debug symbols

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: rename line_bulk arguments to bulk where applicable

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiod.h: fix a typo

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: add a helper for requesting all types of events

Implement gpiod_line_event_request_all() which requests all types of
events on a single line with less arguments. While we're at it: use it
in gpiod_simple_event_loop().

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: code shrink in gpiod_simple_set_value()

Use one of the line request helpers and omit the set value operation,
since we can set the default value when requesting the line.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: code shrink in gpiod_simple_get_value()

Use one of the request helpers for smaller code.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiod.h: add line request helpers

Add inline functions to gpiod.h which allow to request lines with
less arguments.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

event: implement gpiod_line_event_read_fd()

Add a new function which allows to read GPIO event data directly from
a file descriptor instead of a gpiod_line object. This is useful if
the user is already directly polling the file descriptors for events.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiod.h: fix GPIOD_LINE_ITER_INITIALIZER()

Add a missing dot.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: un-inline gpiod_line_iter_next()

This function grew to the point where it should go into a .c file.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: simple events done

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: stop execution and exit cleanly on SIGINT or SIGTERM

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

event: tweak gpiod_simple_event_loop()

If the internal system call is interrupted by a signal - just act as
if the wait timed out.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpiomon: code shrink

Use tools-common like other tools. Convert the program to using the
simple event monitoring interface.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: implement simple event monitoring

Add a function and corresponding data structures for simple event
monitoring.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: code shrink

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

iterators: improve error handling of line iterators

As was done for chip iterators: extend the API to include functions
that allow to check if an error occured while iterating over lines
and use them in the foreach macro and in gpioinfo.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

gpioinfo: check for errors when iterating over gpiochips

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

iterators: extend chip iterator error handling

Add a function that allows to retrieve the name of the gpiochip that
we failed to open from an iterator routine. Use it in gpiodetect.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

iterators: improve error checking

Add new functions that allow to check if opening a gpiochip device
file from an iterator routine failed or if we're done iterating over
all gpiochips present on the system.

Use them in the relevant foreach macro and gpiodetect.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: check for empty name & label of a gpiochip

In case the kernel passes us an empty name or label field when reading
the chip info: return NULL from gpiod_chip_name() & gpiod_chip_label()
just like the line counterparts do.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: implement gpiod_chip_iter_free_noclose()

Implement a new version of gpiod_chip_iter_free() that doesn't close
the most recently opened gpiochip.

Use the new routine in gpiod_line_find_by_name().

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: update

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: rework line handle & event requests

In order to correctly handle bulk requests create a separate,
refcounted structure for handle data.

Put the handle and event fields of struct gpiod_line into a union as
they can't be shared.

Add relevant helpers and use them in appropriate places.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: rename the req field of struct gpiod_line to request

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: add helpers for the line state field

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

core: line status naming tweaks

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

event: rework gpiod_line_event_wait_bulk()

Instead of storing the index of the line - store the line handle
itself as the third parameter.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

TODO: update

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>

event: remove the line field from struct gpiod_line_event

This doesn't make sense - we know what line we're reading the event
from anyway.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>