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>

Due to a bug in lib/conf.c, the conf file must be ended with a newline char;
if not, the latest line is ignored. The bug will be hopefully solved soon,
in the meantime this simple workaround avoids any problem.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

New function receives a struct pp_globals, coherently with pp_open_globals.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

pp_open_instance received a struct pp_globals, so rename it coherently.
We have no pp_open_instance alone because an instance can live only inside its
pp_globals struct.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

The parser is implemented in lib/conf.c and has no libc dependencies.
It is currently used by arch-gnu-linux, whose main function is here modified
accordingly.
This commit has not been tested yet, here is pushed temporarly to show how
work is going on.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This patch temporarly breaks the proper functionality of arch-gnu-linux,
since pp_parse_conf in lib/conf.c is just stubbed.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

pp_parse_conf will receive the txt data read from conf file and will compile
the ppg->links structs. File open is supposed to be done outside the proper
parser here defined, so that any architecture, even those with no filesystems,
can make use of the parser, supposing the configuration txt data is saved
elsewhere (e.g. in a E2PROM)
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

An array of struct pp_link is defined in struct pp_globals: it will cointain
the result of conf file parsing.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

The ppsi.conf file will define the basic behaviour of ppsi, prior to cmdline
options, which will override conf settings. Here a sample ppsi.conf is
defined, where a mini-switch, driving two slaves, is configured.
User will need to define their own ppsi.conf according to their needs.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

PP_VPRINTF is almost no more used, and anyway replaced by pp_Vprintf
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

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

Default Data Set is common to all ports, so it must be defined globally.
pp_open_instance now gets a pp_globals object; the same should be done for
pp_close_intance, but is not urgent since it is currently used by wrpc arch
only, which is a single port arch.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

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

Due to multilink needs, those variables are no more global but can be defined
as a different setting for each port. They will be set in a /etc/ppsi.conf file,
which is not yet handled, but will be soon.
struct pp_runtime_opts will contain only multiport settings.
If the /etc/ppsi.conf file is not found or not handled (like it is at present),
ppsi accepts and properly handles the same cmd line options as in the past.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

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

This tuning is not mandatory, but permits to have a more stable (though slower)
convergence. This is indeed the default setting which can be seen in original
ptpd daemon.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This patch is temporary and is thought only to guarantee the correct
convergence of the algorithm. Some of the latest changes in servo.c, about
how time is adjusted and how the max value for observed drift is set,
made it unstable. With this patch, the behaviour is restored to the same as
ptpd, though it still needs to be reviewed to increase performance, readability
and easier maintainability.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This patch fixes a bug related to reception order of msgs. Sync msgs
are defined as "Event" messages and FollowUp are "General" messages, and we
need to handle them in the very same order they were sent (i.e. FollowUp is
after Sync). With the implementation prior to this patch, we tried to
serve Event and General alternatively. Now we always check for Event frames
before General ones. I didn't find, in ieee spec, whether they cleary state
that Event class is higher priority, at reception, with respect to General,
but this is likely to be in this way, since Event messages are those needing
the most precise timing.

For more information about PTP message classes, see IEEE 1588v2, pag. 17,
paragraph 6.4.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

This patch, suggested by Alessandro Rubini, fixes a recent bug which made
delay_req message been issued just the first time, because the timer was not
rearmed after the message sending.
Signed-off-by: Aurelio Colosimo <aurelio@aureliocolosimo.it>

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

Additionally, this reverses an if to remove an indentation level for
the good 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>

The __BYTE_ORDER == __LITTLE_ENDIAN was failing on lm32, I don't know
why. So dumping on the spec was wrong: "VERSION: unsupported (0)"

The above symbols are defined by glibc, and I didn't even get
undefined macro errors. So the commit kill bit fields, and I mask at
run time instead.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

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

This commit removes the part of wrpc.h that is not used.  It was there
when we tried to build a main function in ppsi itself, but now we
depend on the exteranl wrpc-sewpackage to provide drivers and headers,
so wrpc.h is now just for local material.

Then, wrc_ptp_ppsi.c is fixed with regard to globals and all the rest.
Moreover, I moved most of the initialization of static variables into
compile-time assignments instead of run-time, since there's nothing
dynamic here in those fields.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

I forgot to fix arch-wrpc when making the time_ops.
Code mainly by Tomasz (wrpc-sw/shell/cmd_time.c)
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>