event: rename event requests routines

The event request functionality is now merged with the regular
requests. Change the naming pattern for event requests and move the
prototypes under the input and output requests.

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

core: merge event requests with regular line requests

Requesting the lines both for reading/setting values as well as for
watching events will now be done using the gpiod_line_request_*()
family of functions.

This is the first part of changes required to fully convert the API.

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

chip: lazily allocate lines

Only allocate the array of line pointers is they're actually needed.

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

core: don't duplicate chip-related data structures

Introduce an internal structure - line_chip_ctx - used to store the
chip-related information. This structure is shared among all line
objects exposed by a single chip.

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

simple-api: reposition the prototypes in the public header

Move the 'multiple' variants below the single value ones.

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

tests: line: new test case

Verify that a consumer string over 31 characters long is correctly
cropped.

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

line: remove an unnecessary check

We have changed the way line objects are allocated. This check is no
longer needed.

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

chip: use a temporary variable in gpiod_chip_close()

The code looks better if we use a temporary pointer to gpiod_line.

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

core: separate the chip and line logic

Lazily allocate memory for line objects in chip.c. This eliminates the
need for chip routines to know the size of a line object.

Line objects now contain the chip file descriptor as well, so that we
don't need to access the chip object in any way from line.c.

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

chip: release lines in bulk on close()

When closing the chip use a bulk operation to release all lines.

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

line: remove stray semicolons

These got introduced accidentally when adding chip_get_fd().

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

chip: rename is_unsigned_int()

Use a shorter name for this simple routine.

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

line: coding style fix

Correctly align a broken line.

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

core: tweak some comments

Minor changes to the file header comments in the core library.

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

core: move all line-related code to line.c and remove core.c

This is the final stage of splitting the core library code into
separate files.

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

core: move chip definitions to a separate file

Move all chip-related code to chip.c and create temporary glue-code
header for chip.c and core.c.

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

core: kill gpiod_line_event_release()

We can (and already do in gpiod_chip_close()) easily figure out if
a GPIO line was configured for events or values, so there's no need
to have a separate release function for events.

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

core: move gpiod_version_string() to misc.c

Create a separate .c file for misc definitions.

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

core: move iterator definitions to a separate file

Move all iterator-related code to iter.c.

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

simple API: un-inline simple API routines

Try to avoid inline functions in the public header wherever possible.

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

core: move the simple API definitions into a separate file

This is the first step in splitting up the core library code.

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

core: kill nsec_to_timespec()

This function is tiny and only used in one place.

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

core: kill zalloc() and use calloc() where applicable

Remove zalloc() and use malloc() + memset() in the code. When
allocating an array of elements, use calloc(). This is done in
preparation for splitting up the core library code into separate
files.

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

core: kill gpio_ioctl()

We used this wrapper around ioctl() to set the libgpiod-specific error
number if the ioctl() call failed.

We no longer use custom error numbers, so remove this function.

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

core: kill custom error handling

The libgpiod-specific error codes are not really necessary and can be
easily replaced with regular libc error numbers defined in errno.h.

Remove all routines and definitions dealing with error numbers and
switch to using errnos in the library code.

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

tests: fix checking the kernel version

Previous check was incorrect and would break at major version change.

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

libgpiod: start 1.0 development cycle

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

libgpiod v0.3

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

NEWS: updates for v0.3

Add release notes for libgpiod v0.3.

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

tests: gpiomon: new test case

Verify that gpiomon bails out if the same line offset is given more
than once.

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

tests: gpiomon: new test case for watching multiple lines

Make sure gpiomon behaves correctly when the requested line offsets
are not in order.

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

gpiomon: implement monitoring multiple lines at the same time

Switch from the simple API to low-level libgpiod routines and
implement watching multiple lines at the same time.

Update tests and README.

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

core: event: use correct flags when requesting events

We must use the GPIOEVENT_REQUEST_* flags, not GPIOEVENT_EVENT_* since
the latter are only meaningful when receiving event info.

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

gpioset: improve error messages

Say more than just the name of the failed function.

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

README: improve readability of examples

We're using the same character for comments and the command prompt.
That's not very readable. Use '$' as prompt.

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

README: add an example of gpiomon custom output formats

In the examples section: show how to specify a custom, easily parsable
output in gpiomon.

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

tests: gpiomon: new test case

Verify that we actually ignore events we're not watching.

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

README: update the building section

Commit 6876721f7407 ("build: use a more standard autogen.sh") made
the supplied autogen execute ./configure unless the NOCONFIGURE
environment variable is set. Reflect that in the README file.

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

gpiomon: simplify custom formats

Instead of replacing the format specifiers in a separate buffer, just
make use of the output stream buffering.

Also: update the gpiomon tests with some corner cases.

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

tests: don't allocate the subprocess output buffer everytime

It's more efficient to just allocate it at testing suite's startup and
reuse it for all child processes.

While we're at it: verify that the output of subprocesses consists
of ASCII characters only.

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

tests: gpiomon: use a different unknown custom format specifier

We previously used a character that is not interpreted by
printf()-like functions due to a bug in the testing framework.

This is now fixed, so just use %x so that we self-test the test suite
at the same time.

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

tests: fix reading the output of subprocesses

We're currently using vasprintf() internally when reading the output
streams of terminated subprocesses. This routine interprets the buffer
as a printf() format string, so substrings like '%x' get converted
unintentionally.

Instead allocate a big enough buffer and just read the output without
any modifications.

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

gpiomon: fix parsing of custom output formats

The '%%e' string should be printed as '%e' - the same as for printf()
format specifiers - but gpiomon prints '%%e' in this case. Fix it.

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

configure: tests: require versions of libkmod & libudev known to work

When building tests: require libkmod and libudev versions to be at
least 18 & 215 respectively. These versions are shipped with debian
jessie and are known to work.

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

configure: check for regex functions when building tests

Check if regexec() is available if we need to build the test suite.

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

tests: add a FIXME comment

Reading output from child processes should be fixed at some point to
ignore printf() format specifiers.

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

gpiomon: implement custom output formats

Add a new argument allowing the user to specify the format of the
output string printed on GPIO line events.

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

tests: fix subprocess cleanup when dying

We need to kill the tool subprocess when bailing out during tests
everytime, not only when an error during test_tool_wait() occurs.

Also: we need to wait() for killed processes too as otherwise we may
end up calling delete_module() before the process actually stops using
gpio-mockup giving us a nasty error message from libkmod and leaving
the module in memory.

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

tests: fix the error message when dying during test's execution

If die() or die_perr() is called during a test case, print an
additional newline character so that the error message is displayed
below the test description.

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

NEWS: updates for v0.2.1

Add release notes for libgpiod v0.2.1.

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

tests: gpiomon: more test cases

Add various test cases for gpiomon. Testing invalid input, active-low
mode, termination on signals etc.

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

gpiomon: don't ignore superfluous arguments

We don't currently support watching more than one line at a time. Bail
out if the user gives us more than two non-option arguments.

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

gpiomon: capitalize 'GPIO' in error message

Stay consistent with other error strings.

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

tests: gpiomon: check the whole output string

Add the 'event:' prefix to the regex pattern when checking the output
of gpiomon.

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

tests: gpiomon: new test cases

Add test cases verifying the --silent mode and receiving
multiple events.

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

gpiomon: fix the displayed name for falling edge events

Remove the unnecessary underscore.

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

tests: don't hardcode the chip names

Use test_chip_name() to get the name of a chip. Don't hardcode it as
it will make the tests not work if any gpiochips other than the ones
created by gpio-mockup already exist in the system.

This also fixes a couple hidden bugs, where we requested inexistent
chips in tools' command-line arguments.

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

tests: gpiomon: test a single rising edge event

Add a simple test case checking if gpiomon can receive a single rising
edge event.

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

tests: gpioget: fix potential crash

We're writing outside the values array. Fix the index.

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

tests: chip: fix potential segfault

The automatically cleaned up string is not initialized, but if
asprintf() failed, its contents would be undefined which could lead
to a segmentation fault.

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

tests: iter: fix potential segfault

All automatically cleaned up variables must be initialized. Otherwise
we risk calling free() on an uninitialized pointer if an assertion
fails before we assign it a value.

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

tests: gpioinfo: improve test descriptions

Explain more precisely what we're actually testing.

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

tests: gpioinfo: test incorrect input

Check if gpioinfo behaves correctly if the chip for which we want the
info dumped doesn't exist.

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

tests: gpioinfo: test cases for printing info about specific chips

Add test cases checking if gpioinfo correctly dumps info if we specify
only certain chips over command-line and omits the ones we don't
request.

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

tests: new assertions

Add two new string-related assertions for tests.

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

tools: capitalize 'GPIO' in error messages

Be consistent in all error messages in gpioget & gpioset.

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

tests: gpioset: test cases for invalid arguments

Add various test cases checking the behavior of gpioset when given
invalid arguments.

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

tests: gpioset: test the --sec argument

Make sure gpioset understands the --sec argument together
with --mode=time.

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

tests: gpioset: test the --active-low flag

Duplicate the basic exit mode test but use the --active-low flag and
verify that the values are inverted.

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

tests: gpioset: test the 'time' mode

Order gpioset to wait for 100 milliseconds before exiting, then sleep
for 200 milliseconds before checking if it exited correctly.

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

tests: gpioset: test case for the 'signal' mode

Run gpioset on a set of lines, give it a couple milliseconds to
execute, then send it SIGTERM and check that it exits correctly.

Repeat for SIGINT.

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

tests: improve sigprocmask()-related error messages

When sigprocmask() fails for how=SIG_BLOCK|SIG_UNBLOCK, say explicitly
we were unable to block/unblock SIGCHLD.

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

tests: unblock SIGCHLD after closing the signalfd

Restore the state from before running the subprocess.

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

tests: gpioset: check the 'exit' mode explicitly

Duplicate the previous 'exit' mode test but pass the mode argument
explicitly to gpioset.

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

tests: read siginfo after receiving SIGCHLD

If we don't actually read the signal info, the signals pile up after
every child process exits even if we close the sigfd.

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

tests: gpioset: test the 'wait' mode

Verify the behavior of gpioset in the 'wait' mode. Send a newline
to gpioset's standard input and check if it correctly exits.

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

tests: provide test_tool_stdin_write()

Implement a routine allowing the test code to write a formatted string
to the child process' standard input.

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

tests: gpioget: test the --active-low flag

Read all lines, change their values and re-read but this time pass
the --active-low flag to gpioget.

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

tests: add a simple test case for gpioset

Verify the 'exit' mode of gpioset.

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

tests: check reading specific lines with gpioget

Add a test case where we only read a couple GPIO line values at
specified offsets.

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

tests: new test case for gpioget

Verify that gpioget correctly bails out if the user requested to read
more GPIO lines than the number exposed by the chip.

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

tests: use calloc() where applicable

Introduce xcalloc() and use it whenever we're allocating an array
of elements.

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

tests: new test cases for gpioget

Verify the correct behavior of gpioget on invalid arguments.

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

tests: add a test case for gpioget

Add a simple test case for gpioget: read all lines at their initial
state, change the values of some of them and re-read.

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

tests: rename test_gpiotool_run()

Use the same prefix for all functions for tools testing.

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

tests: new test case for gpiodetect

Verify that we bail out if the user passes any non-option argument
to gpiodetect.

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

tests: verify the behavior of gpiofind in case of invalid arguments

Make sure we bail out with the correct error message when the user
doesn't give us exactly one GPIO line name.

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

gpiofind: tweak the error message on invalid arguments

We print the same error message in case of too many or not enough
arguments. Make the message appropriate for both cases.

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

tests: fix a potential segfault

A string passed to test_free_str() must be allocated by malloc() or
NULL, so we need to initialize ptrn.

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

tests: fix a memory leak

We don't free the string allocated by asprintf() in a test case for
gpioinfo. Fix it.

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

tests: fix a test case for gpiofind

We must not hardcode the chip names. Use test_chip_name() instead.

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

tests: add a missing assertion

asprintf() can fail. Check its return value.

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

tests: improve the test names

Make the names of the gpiotools test cases more informative.

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

tests: new test case for gpioinfo

Check if we correctly report exported lines.

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

tests: tweak test output

Tweak the log message header a bit: instead of having a fixed width
of the contents between the brackets in the header, fix the length of
the whole header to 8 characters.

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

tests: add test cases for gpiofind

Verify the correct behavior of gpiofind in two cases: when it can find
a GPIO line by name and when the requested line doesn't exist.

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

tests: improve the version string test

Use the recently introduced regex matching function to verify that
the version string has a correct format.

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

tests: don't request gpio-mockup chips for the version string test

Change the line sizes list into an empty array.

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

tests: add a test case for gpioinfo

Add a simple test for gpioinfo.

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

tests: add regex handling

Add a routine allowing to check if a string contains a substring
matching against given regex pattern.

Also: add a corresponding assertion.

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