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

The HAL process, in the white rabbit switch, is now exporting data
structures in shared memory. Thus, we don't need to "get_port_state"
by RPC, and accessing shared memory is enough.  This change in ppsi
involves the following changes:

arch-wrs/wrs-startup.c: offer global hal_ports[] for others to use
(yes, globals are bad, I know...)

arch-wrs/include/ppsi-wrs.h: provide pp_wrs_lookup_port

time-wrs/wrs-socket.c: use shmem. This means some values are now
hardwired as constants, because they were passed as such from the hal.

arch-wrs/main-loop.c: verify link up or down in the shared mem status

arch-wrs/wrs-calibration.c: wrs_read_calibration_data uses shmem

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

This is respinning a previous commit, based on the new shmem in libwr
after the feedback and patches by Adam. More headers are inserted,
as they are included by the ones we use.

hal_shmem.h has one 1-line modification to account for the different
include path in ppsi, as opposed to wr-switch-sw.

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

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

When using "make -j" we must make silentoldconfig before building
any object. Otherwise we may use the previous autoconf.h, in error
if we just changed the configuration.

This was exposed by ./MAKEALL with the properly working wr/non-wr build
for wrs: previous wrcp_defconfig left CONFIG_EXT_WR active while
building some file with wrs-nonwr_defconfig.

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

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

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

All functions in msg.c that are used locally are now static.
There is one that is never used at all (msg_unpack_delay_req)
but I fear I need more auditing before removing it. It may
be something we need to do, I'm not sure.

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

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

This field is not needed, because we issue the response immediately,
using the last received header. The copy is not needed, nor the field.

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

We always send them in pair, so use a single function to do it.
This simplifies the code a little, but most of al it allows
later interoduction of mandated masters on multiple vlans.

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

Cesar Prados authored 10 years ago and Alessandro Rubini committed 10 years ago

Signed-off-by: C.Prados <c.prados@gsi.de>

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

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

In the next commits we'll need CONFIG_ symbols and ARRAY_SIZE

Unfortunately, we can't use "-include include/generated/autoconf.h"
in CFLAGS, because pp_printf is built with our CFLAGS from its own
directory. And I'd better not include the full pathname as our make
is verbose

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

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

This paves the way to the addition of a third protocol. Besides the
code is slightly smaller (around 50 bytes on all archs).

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

This reverts commit 9a3e00784afea6ab46510315b9c50cb510175f2d.

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

Bitfields are bad, so this turns them into flags and constants.
There should be no change at all in behaviour (although the code
is a little smaller).

The next commit does a change for real.

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

I'm going to introduce a "flags" field in pp_instance, so diagnostic
flags are better called d_flags. The same applies, for consistency,
to the global variable.

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

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

Bitfields are a bad practice. And this removes a few bytes in the wrpc
builds (no change in other builds).

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 reverts commit 5ac225a4.
Otherwise, in the case of WRS (when wrp->calibrated is always true) we never
call calib_poll() and we don't provide correct values of fixed Tx/Rx deltas.

The port number used to be 1, but we are multi-port, and forgot to
update this item (no bad effects were seen so far, but the standard
requires each port to use a different port number so loops can be
managed, and, for example, having both Ethernet and UDP on the same
wire is a loop.

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

Otherwise, if we have two PTP ports for the same interface, like
Ethernet and UDP, we are not able to tell which message is for which.

This now works as expected:

./ppsi -C "port eth0;  iface eth0; role master; proto raw; diagnostics 111" \
       -C "port eth0u; iface eth0; role master; proto udp; diagnostics 111"

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

This adds the "adjust_frequecy" method to time-wrs. The WR servo is
not using this method, which is only called by the normal PTP servo.

With this new method, a non-WR slave is able to trim frequency,
whereas before this commit we were only able to jump our time.  The
code move straight the FPGA register, by having some hardware-specific
constants explicit in the code. Thus, it is not very beautiful, but
it works fine on my systems.

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

All adjustments are passed as part-per-billion, so after getting mad
in understanding what freq_ppm was, I renamed all of them to freq_ppb.

While making changes I verified this is what it is. Thus I fixed the comment
in ppsi.h (that was wrong by a factor of 64) and the adjustment code in
bare-time.c that made the same error (I admit I don't test arch-bare often).

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

This bug was present in previous versions too: when not sending the
calibration pattern, we had no timeout, so a lost frame was fatal.

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

This commits adds retry in most wr states. This allows to deal with
lost frames. There a few missing items, and a few rough situation.
The code itself shows a number of repetitions, which I dislike, but
over time we'll fix it properly.

By now this should be enough to deal with transient data loss
during calibration, that happens at some users' site.

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

WR_DEFAULT_INIT_REPEAT is going to be used instead (we had both, none
of them used; now we'll use one).

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

Currently, a wrpc slave taks almost 10s to do the calibration step
(uncalibrated/wr-s-lock), because, among other things, of eeprom
access.

Thus, a 10s timeout is really unsuitable. Turn it to 15 instead.

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

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

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