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>

TODO: update

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

core: move a call to memset() in gpiod_line_event_wait_bulk()

Check for error conditions first.

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

gpioinfo: tweak the output format

Add an additional space before the active state for better
readability.

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

core: fix gpiod_line_event_request()

Correctly set the last error if the line is busy.

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

core: extend verify_line_bulk()

Make verify_line_bulk() set the last error number (according to the
error condition) and verify if any line is busy too.

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

core: new error number

Add an error indicating that given line is currently in use.

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

core: implement gpiod_line_is_free()

This routine allows the user to check if neither ownership nor any
event notifications were requested for this line.

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

gpioinfo: improve output format

Improve the way the information is formatted. The code is still pretty
ugly though.

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

gpioinfo: code cleanup

Use tools-common & getopt. Split the code into separate routines.

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

core: naming convention tweaks

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

core: new error code

Add an error code indicating that an event operation was requested on
a line for which no events were configured.

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

core: new error number

Add an error number indicating that a bulk operation was requested
on a set of lines not belonging to the same gpiochip.

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

core: tweak the "line not requested" error code

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

core: merge the reserved and event_configured fields

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

core: create accessors for up_to_date field of struct gpiod_line

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

gpiofind: fix segfault

Should have been argv[0] instead of argv[1].

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

core: fix segfault in gpiod_line_find_by_name()

We need to check whether gpiod_line_name() didn't return NULL.

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

gpiofind: remove stray newline

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

tools-common: use gpiod_last_strerror()

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

core: implement gpiod_last_strerror()

This is the same as gpiod_strerror() except that it automatically
uses the last error number by calling gpiod_errno().

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

TODO: update

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

gpiod.h: documentation updates

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

gpiod.h: make the include guard more unique

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

configure: check for ppoll()

We're using ppoll() for events - check if we have it before trying
to compile.

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

core: use strerror_r() instead of strerror()

Use a thread-local buffer for the error message.

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

tools-common: add the UNUSED attribute and use it in gpioset

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

TODO: update

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

gpioset: code shrink

Make gpioset use getopt, tools-common and use the new simple API for
setting values.

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

gpioget: fix help message

Also: add a space for readability.

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

core: implement gpiod_simple_set_value()

Implement a counterpart for gpiod_simple_get_value().

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

core: move gpiod_line_find_by_name()

Move this routine to where other line-related functions live.

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

gpioget: remove a FIXME

The user can't pass a negative number as argument anyway, as it's
considered an unrecognized option by getopt.

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

tools: return from main() rather than exit()ing

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

gpioget: move printing the help message to a separate function

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

gpiofind: add options to the help text

Also: move displaying the help message to a separate function.

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

gpiodetect: add options to the help text

Also: move displaying the help message to a separate function.

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

gpioget: use tools-common

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

core: extend gpiod_simple_get_value()

Add the active state argument.

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

global: fix active state naming

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

gpiodetect: handle unrecognized options

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

gpiofind: user tools-common

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

global: include gpiod.h locally from tools and library code

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

tools-common: add gcc attributes

Add macros for two gcc attributes and use them in die()
and die_perror().

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

tools-common: new source file

Introduce a .c file containing code that is common across all GPIO
tools but not really related to the libgpiod functionality. This file
is statically linked with every executable.

While we're at it - make gpiodetect use it right away.

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

TODO: update

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

gpiod.h: don't include stdint.h

We can move this include to core.c.

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

gpiofind: new tool

Add a simple program for finding the gpiochip and offset of a GPIO
line by name.

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

TODO: update

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

event: modify the API

In order to make it possible for users to poll the event file
descriptor, we need to split the logic of waiting for events to occur
and reading them into separate routines.

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

core: remove gpiod_chip_get_fd()

With current functionality offered by the library, there's no need for
the user to directly access the gpiochip file descriptor.

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

core: move struct definitions to the top of core.c

We will need to access the chip data from the line-related code.

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

gpiod.h: shuffle some declarations

Move the opaque structs declarations to the top of the file - they are
ignored by doxygen anyway and that way we can move
gpiod_line_get_chip() to the line defgroup.

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

gpiod.h: doxygen documentation - part II

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

TODO: update

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

core: naming convention tweaks

Change naming of some routines and variables dealing with line
requests. This makes their purpose more obvious.

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

gpiod.h: document the API - part I

As of now, the public API should stay more or less similar to the
current version. Add the first batch of doxygen documentation for
exported symbols, routines and macros.

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

gpiomon: fix Makefile.am

Add gpiomon to the list of programs to build.

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

gpiomon: new tool

Implement a simple GPIO event monitor.

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

gpioset: release the line before exiting

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

core: fix gpiod_simple_get_value()

We should set direction to INPUT.

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

core: fix gpiod_line_update()

Don't zero the line offset.

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

core: implement the event API

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

build: install headers too

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

core: fix leftovers from flag shuffling

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

core: implement gpiod_line_find_by_name()

Implement a routine for finding lines by name.

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

core: tweak line iterators

Some line iterator changes to keep them similar to chip iterators.

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

core: simplify iterating over line bulk objects

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

core: rename direction defines

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

core: remove redundant memset()

The request structure is already zeroed by zalloc().

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