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

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

This removes TRACE_DEV(), turning it into pll_verbose() and wrc_verbose().

The functions are enabled and disabled by kconfig, with defaults matching
the previous behaviour.

So nothing changes, but the binary is smaller because the previous
wrc_debug_printf() was silent because of a run-time conditional
(though hardwired false at build time). The condional is now at build
time.  We save around 2k, between strings and calling code, in
all wrc builds (nothing is saved in wrs, which is verbose by default,
nor in devel_build_test_defconfig where all verbose options are on)).

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

devel_build_test_defconfig: enable all verbose options

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

The makefile included trace.h for everyone, which in turns included
wrc.h.  It's better to include wrc.h straight on (we can't rely on all
files including it, for example pp_printf does not).

Meanwhile, some redundant includes are removed, and some are added
(I prefer wrc.h to be explicitly listed in the files that use it,
even if the command line set in Makefile already prepends it).

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

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

The misleading TRACE thing was only used in wrs builds, and expanded
to pp_printf.

Considering we don't want to remove messages from the softpll in wrpc,
this commit turns trace onto pp_printf.

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

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

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

Otherwise, the first "make" builds tools before spitting the
error about an unconfigured system. I prefer to have the error alone.

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

This quiet=quiet_ has the effect of printing

  HOSTCC  scripts/kconfig/conf.o
  HOSTCC  scripts/kconfig/kxgettext.o
  SHIPPED scripts/kconfig/zconf.tab.c
  SHIPPED scripts/kconfig/lex.zconf.c
  SHIPPED scripts/kconfig/zconf.hash.c
  HOSTCC  scripts/kconfig/zconf.tab.o
  HOSTLD  scripts/kconfig/conf

instead of the full command lines. We could even use quite=silent_ to
have no output at all, but this would introduce a noticeable delay in
the build.

The aim of this change is making more visible the error "your wrpc is
not configured" at the first build. The full compilation of config
stuff makes the error less clear, and the initial error confuses
people (me included).

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

This reverts commit 3a3c3806.

The previous line: every object depending on .config, was better.
We had the problem that a fresh clone was not building because of a
missing autoconf.h.

The depend rule was not working anyways, because it created an empty
.depend the first time over, due to the missing autoconf.h.  I admit I
don't know how to create a depend file after configuration: make
pretends to remake all its inclusions as the first step, so a rule
like ".depend: silentoldconfig" loops forever.

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

* library files can be built even if not used (this gets us more build-time
testing, as for example I usually build with no IP support).

* Some ifdefs are not needed, use HAS_IP instead. This adds a few
bytes to some configurations (not all, can't tell why), but I find it
acceptable.

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

Actually, Etherbone is in gateware, and this option only enables Internet
Protocol in the software.  This fixes a long-standing misunderstanding.

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

This configuration includes all developer options, so we are sure that
./MAKEALL is build-testing all the code (excluding the very-reduced
pp_printf implementations).

If you want to check which config options are used, please run

  grep '^config' Kconfig | tr '[a-z] ' '[A-Z]_' | while read n; do
    echo =========== $n
    grep -rhl "$n=" configs/*
  done

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

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

This type is used by pp_printf-full, and we miss it in lm32.

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

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

This counter can be read through etherbone, to check that the software
is properly running. Feature asked by GSI.

The cost is 84 bytes in code.

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

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

If the Kconfig option is not set, there's no effect on code size.
If set, .bss increases by 256 bytes (the buffer) and .text by
another 200 bytes.

This adds timestamping support, so we know the duration of the
interrupt (because we found it's an important thing when problems
occur).

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

Accessing data structures in the host to get target fields is broken:
we may have different data size or alignment, or whatever. The hack
of using "-m32" to be able to access lm32 fields from x86-64 is not
clean (we imply the i386 and target have the same size/alignment).
Even worse, not everybody has an x86-64 cross-compiler for i386.

Thus, we use OFFSET_OF in the target, with the lm32 cross compiler,
to build a binary table that is then accessed by the host.

In order to link the data structure in the host we use the assembler
instead of objcopy. With objcopy we'd need to spell out the bfd name
of the host, and we'd get horrible names like
   struct dump_info _binary_dump_info_host_bin_start[];

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

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

There was a hole at the beginning of the generated binary, so this
commit includes some data to identify the endianness of the dump file
and host pointers to the important structures that we dump.
For wr-switch, the ppi pointer is just set to zero.

The change is backward compatible, because we just fill 32 bytes of
data where we had a bunch of zeroes.

The next commit will use such information when available in the dump file.

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

SQUASH

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

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

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

Use an environment variable to check for spec byte order, and
fix the buffer before using it.

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

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 commit has no effect on the final binary file.

The state structures for softpll_ng must be accessed by tools/wrpc-dump,
so we need them in an header (not in sofpll_mg.c itself).

Unfortunately, struct softpll_state includes a lot of other structures,
so all dependent headers must be included by softpll_ng.h.  Thus,
all C files should now include softpll_ng.h alone, not the other ones.

I also had to remove inclusion of unneeded <stdint.h> and <stdlib.h>
from spll_common.h, because stdlib pulled the prototype for abs()
which is #defined in ppsi (my bad!), and we would have a build error.

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

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

It's still suboptimal, we need to autodetect with a in-binary helper,
but by now I'm analyzing old logs so I'd better not change wrpc
itself.

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