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

Since we have a call to bmc inside this function, we do not need to
unpack Announce message locally.

This patch fixes a bug with white rabbit extension, since it did not
update the foreign master record with most recent wrFlags. As a result,
at the end of wr handshaking, the slave (very often) did not handle
correctly the parentWrModeOn flag.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This partially reverts commit 9bd032a6, see wrspec.v2.06-07-2011, page 28
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This fixes a bug in proto-ext-whiterabbit, since the White Rabbit next_state
was overwritten by bmc.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This fixes the correct behaviour of ppsi in arch-wrpc, where adjust_freq
is not available.
Still some more fixes may be needed in servo.c. I'm not sure about the
parameter -adj to be passed both in adjust_freq and adjust_offset case.
Anyway, this has been tested on two SPECs and reaches synchronization.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

if the user passes "-DDIAG_PUTS=sth~ in USER_CFLAGS at build time, al
diagnostic messages are directed to this special puts (through sprintf
to a local buffer) instead of going to the normal printf (and thus the
default puts).

This is going to be used by wrpc-sw, to leave the default output channel
clean for the user shell.

Unfortunately, frame dumps cannot go to this special puts, as
lib/dump-funcs.c just call printf and I don't know how to fix that
without a massive change in 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>

Commit 7373cc02 introduced a workaround for a suboptimal choice in
wrpc-sw about global variables (that were "common").  Commit 9ff25f86
of wrpc-sw fixed it, so this removes the workaround (which, btw,
prevents compilation of ppsi as a submodule of current wrpc-sw).
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

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

call t24p calibration every time PTP is set to Slave mode. This
is a port of commit 279f2c6b of wrc-sw, by Grzegorz Daniluk, since
wrc_ptp_ppsi.c was still part of the supermodule.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

This bugfix is child of the fixed conf startup procedure: read conf, parse
cmd line, and then open pp_globals by using its current rt_opts.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

pp_open_globals must be called after pp_parse_cmdline; ppg->rt_opts is thus
set explicitly before pp_parse_cmdline, since this parsing affect rt_opts.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

rt_opts parameter was always passed as NULL. With this patch, users who do not
want default rt_opts provided by proto-standard need to set their rt_opts
explicitly.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Default Dataset slaveOnly variable is global and cannot be used in multilink
context
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This major patch implements the core of Best Master Clock Algorithm for
multilink context (e.g. Boundary Clock).
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Will be useful for global bmc_state_decision computation
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This partially reverts commit 16707d53, since the IEEE spec states that foreign
masters list must be kept updated for each link, then Ebest search among all
links must be performed.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Two main problems where found when compiling into wrpc-sw, commit
e00d99f82:

- wrpc_sw/include/hw/memlayout.h defines some symbols into a header file
(I think they should be defined there as "extern" and actually declared
somewhere else);
- there was an include to be removed

The simple solution for the first issue is to define __REGS_H where
the compiler complains; the better solution, to me, is to follow include
chain to discover who is including memlayout.h, or define those symbols as
extern.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

posix_main_loop now gets a struct pp_globals ptr and simultaneously
handles all configured links. posix_net_check_pkt has been changed too,
in order to perform a global select on all fd's.

This patch is temporary since it may need debug & test.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

POSIX_ARCH was defined buy never used, so redefine it for pp_globals, since
it will be more useful.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Command line now acts globally. If multilink context (ppsi.conf defined and
cointaining more than one link), the behaviour of -g and -e flags is common
to each link. On the other way, -b flag can not be used in this case.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

GDSDEF macro returns the defaultDS field of struct pp_globals. Actually,
it is the same pointer returned by DSDEF, since the default dataset is unique.
The DSDEF macro is not removed both to preserve commit history and to increase
readability (we do not take ppg pointer around and DSDEF(ppi) is shorter
than GDSDEF(GLOBS(ppi))).
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

GOPTS macro returns the rt_opts field of struct pp_globals. Actually,
in the current implementation it is the same pointer returned by OPTS, since
each pp_instance has no "local" rt_opt, but points to the global one.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Still to be fixed is pp_parse_cmdline
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

nports field is fully replaced by nlinks, so I remove it.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

Three bugs and missing features are fixed here:
- pp_open_globals received a SEGFAULT because its rt_opts parameter was not
correctly handled when NULL;
- use ppg->nlinks instead of obsolete ppg->nports;
- set def->slaveOnly and def->clockQuality.clockClass properly even when
more than one link is defined
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>