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>

core: put the line request configuration into a structure

The base routine for line requesting should have all config parameters
packed in a single structure. Future request helpers will simplify
this.

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

core: remove different gpiod_line_request() versions

Until the request API is considered stable, remove all the request
helpers.

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

core: rename GPIOD_MAX_LINES

This define specifies the maximum number of lines that can be
requested in a single operation. Make it clear with the new name.

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

core: implement public line update routines

Line info is retrieved from the kernel for the first time
automatically when gpiod_chip_get_line() is called. Then the line
is updated automatically on request/release. If the ioctl fails in the
latter case, the up_to_date flag is set to false.

Expose an interface allowing users to check if a line is up-to-date
and manually update it if needed.

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

misc: add a TODO file

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

core: add input/output flavors of bulk request routines

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

core: merge the direction argument into flags for request routines

There's no reason to keep the direction as a separate argument if we
have polarity configured using the flags argument. We have separate
routines for input/output requests anyway.

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

build: add dependencies to Makefile.am

This fixes the build with multiple make jobs.

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

core: implement gpiod_line_polarity()

Instead of returning a boolean value from gpiod_line_is_active_low(),
introduce two separate values as is done for direction and implement
gpiod_line_polarity(). This makes the code less confusing.

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

gpiodetect: use the correct gpiod_chip_open() flavor

gpiod_chip_iter_next() should use gpiod_chip_open_by_name() internally
when opening chips by dentry->d_name.

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

core: code shrink

Reuse the bulk routines for single lines too. This unduplicates
a lot of code.

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

core: coding style fix

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

core: remove gpiod_simple_set_value()

This has been removed from gpiod.h and doesn't even work. Remove the
definition from core.c.

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

core: add bulk operations

Add versions of the request/release and get/set value operations for
bundles of GPIO lines.

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

core: update line info after a line request

Currently the line information isn't updated after it's been
requested. Re-read the line info after requesting it.

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

core: add libgpiod-specific error numbers

In order to better report error conditions to users, we need a set
of libgpiod-specific errnos. Extend the previous API to be able to
use both regular errno and GPIO-related error numbers together.

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

core: clamp GPIO values to either 1 or 0

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

gpioset: drop the call to gpiod_line_set_value()

This is not needed as we already pass the default value to
gpiod_line_request_dout();

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

core: fix a default value bug when requesting a GPIO line

The direction is stored in the 'direction' argument, not in flags.

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