This kills an ifdef that was due to die anyways. The only disadvantage
is that bare-* architectures can't be passed arguments to force
slave-only (while arch-gnu-linux can received it on the command line).
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This patch set removes duplicate code in ext-whiterabbit, using hooks
in proto-standard to call specific extension code.

There is still some duplication in wr-msg.c from msg.c), so I'll need
to touch stuff again, but this is good and stable enough to merge.

Unfortunately, the set of extension hooks is specifically tailored to
white rabbit, and we'll need to make it more orthogonal and solid as
soon as another extension must be supported (or as soon as we have
time).

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

The ifdef was used to print smaller messages if the daemon is known
to be slave only. Since this PPSI_SLAVE is going to die, remove the
ifdef and keep the more complete branch of the code.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Since timeInternal is seconds and nanoseconds, avoid copying both
and a temporary variable by just storing them to the target structure.

This is also meant to avoid a WR-specific hook in the next commit,
since timeInternal in WR is more fields than seconds and nanoseconds.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This removes the WR-specific state-initializing.c by introducing
the init() hook for the standard state-initializing.

Meanwhile, clean up error management a little in the init state.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This new "struct pp_ext_hooks" is used to include pointers to
extension-specific hooks that allow to avoid code duplication.
I'm now going to apply it to ext-whiterabbit, one state at a time.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

struct DSPort now has a void pointer called ext_dsport that is used by
each extension to store its own local data.  White Rabbit, thus,
is not a special case any more in this respect.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This adds the name of structures for all typedefs in the header.
It has no technical effect, but allows developers (i.e. me) to
use "struct DSPort" instead of "DSPort" in new code.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

While the standard protocol doesn't expect master-only opeation, the
flag is in the data strcture and this change, needed for white-rabbit,
has no effect here.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Since proto-standard is used to build a library that can provide
defaults for the extension, include its Makefile regardless.  This has
no technical effect at all until we remove duplicated code (next
commit).

This commit also makes some trivial and small changes in variable
assignements (the ones the user may want to change are now at the top
of the file).
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

You can pass architectures (for example "./MAKEALL spec"),
specific settings (for example "./MAKEALL CONFIG_NO_PRINTF=y")
or "-u" to ask the printout of undefined symbols in each ppsi.o
being built.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

If you "export CONFIG_NO_PRINTF=y", no printf will be built in the
binary. This is needed when ppsi is built within wrpc-sw (which has
its own pp_printf implementation).

This will become a Kconfig item, when Kconfig is added.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Now that we have vprintf, we can save a buffer in the stack and
simplify pp_diag_printf and pp_timed_printf.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This patch-set removes the old diag/printf.c and related files, to use
the "official" pp_printf, which is now published separately and
included here in its entirety with not changes from its upstream repo.

Please note that HAS_DIAG and HAS_FULL_DIAG is not supported any more.

While we have no Kconfig support (coming soon), please see ./MAKEALL
to find how to pass verbosity (and thus size) options to pp_printf.

All commits build for all architectures.