This commit makes no effect, but it prepares for the following one.
In general, using an intermediate platform-agnostic linker script
when making wrc.o is not bad, so let's do it here.

The main reason why we need it is to force the "KEEP" command for the
".cmd" ELF section, introduced by the next commit.  The alternative to
this would be chaning the main linker script, which is worse in my
opinion.

If we add initcalls or similar stuff later, this bigobj.lds is the
place to do it.

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

The parameter used to be off by default in order to save space in the
generated binay.  Now ppsi is smaller and there's no reason to
prevent diagnostic code to be there.

This parameter, though, will disappear over time, as we have a new
diagnostic system, more modular (which still offers a compile-time
option to remove all diagnostic code).

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

The new configuation option CONFIG_CHECK_RESET adds a function to dump
the stack (to the serial port) if a reset is detected. This happens
when a null pointer is followed or when the program is reloaded
(spec-sw/tools/spec-cl, for example).  For this reason, the feature is
disabled by default (actually, spec-cl may clear the RAM first, but it
currently doesn't do that and the check is triggered).

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

Clearing the screen is in general bad practice: it hides information
for no advantage (ok, I can accept if for the "gui" interface).

In this case it removes from the user the build date.

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

I personally dislike "common" variables, as they hid bugs a few times
for me. Thus, ppsi has "-fno-common" by now.  This commit adds
extern to memlayout.h and creates the variables in dev/sdb.c .
This builds whether or not -fno-common is there and has no effect
on a ptp-noposix build.

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

Alessandro Rubini authored 12 years ago and Grzegorz Daniluk committed 12 years ago

the pp_printf_defconfig was the trial version for pp_pritnf as an
alternative to mprintf, so it is not needed any more.  All other
defconfig were working already, but I'd better regenerate them with
the current defaults.

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

Alessandro Rubini authored 12 years ago and Grzegorz Daniluk committed 12 years ago

mprintf is of uncertain copyright status, and that's why
we added pp_printf (actually, that's why I picked pp_printf
fro ptp-proposal and made it a standalone thing).

Now that everyone involved confirms pp_printf is well tested,
let's remove mprintf.  We still accept mprintf in the callers,
because the name is used by our ptp-noposix submodule, and I'm
sure people has it in their fingers and it will appear again.

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

Alessandro Rubini authored 12 years ago and Grzegorz Daniluk committed 12 years ago

The ppsi submodule (which is being cleaned up) now has a separate
structure for whiterabbit-specific data associated to the port. This
commit includes such change and adapts its own structures accordingly.

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

Alessandro Rubini authored 12 years ago and Grzegorz Daniluk committed 12 years ago

The ppsi subproject now builds pp_printf by default, so the Makefile here
passes ~CONFIG_NO_PRINTF=y". Also, assign_TimeInternal() is not there
any more because it is just an assignment.

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

The minic driver reads the TX timestamp from the device after sending.
Unfortunately, it only waited for the data to be enqueued to be sent.
Sometimes this lead to a race condition where the old TX stamp was read.

As a result, the PTP calculation goes wrong by as much as an entire second.
When in track_phase, this manifests as a bad phase offset sent to softpll.
The softpll takes years to reach that phase offset, so it is always busy.
Thus the clock drifts further and further away, never correcting.

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

Due to previous implementation of net functions, packet buffer address was
badly aligned.

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

ppsi main was still using obsolete eth_ofst when logging incoming packets

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

ppsi module now correctly handles mode shell command

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

This patch makes ppsi use lib/net.c socket functions, whose declaration is
in include/ptpd_netif.h

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

Still missing (or to be fixed) the calls to net-related functions
(minic_poll_rx, spec_recv_packet,...)

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

Aurelio Colosimo authored 12 years ago and Alessandro Rubini committed 12 years ago

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

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

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