Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This will let us factorize a lot of very similar config handlers.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

priority2 can be set via the "priority2" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -q cmdline option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

priority1 can be set via the "priority1" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -p cmdline option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Clock class can be set via the clock-class configuration option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Clock accuracy can be set by means of the clock-accuracy config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This has been replaced by the clock-allen-variance config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -v option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

ethernet mode can be set via the "proto" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-y has been replaced by the "announce-interval" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -y command line option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-n has been replaced by the "announce-interval" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -n command line option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-i has been replaced by the "domain-number" config option.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to the -i command line option, which will be removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-l has been replaced by the "latency" config option.
This patch also removes cmd_line_parse_two(), which is now useless.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to -l NUMBER,NUMBER command line option, which will be
removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-a has been replaced by servo-pi in the config file.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This is equivalent to -a NUMBER,NUMBER command line option, which will be
removed.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

This get things ready for introducing the servo-pi config option, which
takes two comma separated integers.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

If interface name is empty, look for a network interface with the same
name of the port.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

-b didn't actually work. Even with a single interface and no config
file, the following error message was printed [arch-unix]:

Can not use -b option in multi-link conf

The bug was caused by a problem with the initialization value
of ppg->nlinks.

Removing this option looks wiser than trying to fix it, because
most users just work with a configuration file or -C on the command line,
so fixing -b would just increase the risk of breaking non-unix architectures
with no actual gain.
Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

For long liks (like 180km) "(signed)s->picos_mu" overflows causing wrong
result of the comparison:
(signed)s->picos_mu < (signed)big_delta_fix)
Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

I need to be more careful before pushing to master...
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

comments used to extend to the next terminator (including the
semicolon), but editors paint comment-color the whole line, so we'd
better fix it rather than documenting an uncommon behaviour.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

Signed-off-by: Davide Ciminaghi <ciminaghi@gnudd.com>

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

Even if we count failure events for SNMP (in the switch), we actually
never generate them.  The code has always been tracking phase for
any offset smaller than one clock tick (8ns or 16ns), but we'd better
jump in phase (and leave TRACK_PHASE servo state) and this is a failure.

With this commit, to loos track_phase in wrpc you can just force
the setpoint far enough from the calculated one: e.g.

  [..] cko:2 setp:929 [...]

  wrc# pll sps 0 500

  [...]  ss:'SYNC_PHASE' [...] cko:431 setp:1367 [...]

then it takes 15 iterations to converge, but that's because the
calculations the first time (as shown) are wrong. A real glitch
loosing track_phase would be recovered in 3 iterations.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>