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>

gpiod.h: remove GPIOD_ARRAY_SIZE()

This macro is only used by gpioinfo and is not relevant for the GPIO
API. Remove it from gpiod.h and define it directly in gpioinfo.c.

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

core: don't set the error number if zalloc() fails

zalloc() is implemented locally and already sets the error number on
failure.

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

core: release all lines in gpiod_chip_close()

This fixes a memory leak if the user forgets to explicitly release
requested lines.

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

core: dynamically allocate memory for GPIO line request data

For bulk requests, the lines will share the struct gpiohandle_request
object. Instead of it being a part of struct gpiod_line, allocate its
memory dynamically and only store the pointer in the line context
structure.

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

core: rename two fields of struct gpiod_line

Drop the 'l' prefix - it's clear the fields are part of a GPIO line
structure.

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

core: ioctl() code shrink

Wrap GPIO ioctl() calls in a routine setting the libgpiod errno in
case of an error.

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

error: add internal wrappers for setting libgpiod errno

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

documentation: add Doxyfile

Add doxygen configuration and a new target to Makefile.am to generate
documentation. Also: update gitignore.

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

libgpiod: use a per-thread variable for storing errors

The kernel-like trick with embedding error numbers in pointers can't
be guaranteed to work on all architectures. Use a per-thread variable
instead and add corresponding routines to access it.

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

build: define _GNU_SOURCE for .c files

This is needed for asprintf().

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

libgpiod: add a simple README

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

gitignore: ignore files generated by autotools

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

build: add autotools configuration

This allows to build the library and programs using autotools.

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

libgpiod: initial commit

This adds four basic tools and implements core functionalities.

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