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

We removed the pool of sockets, to only keep a list of pointers.
The caller provides the socket (and the buffer).

This saves from 440 to 1570 bytes in the binary (more in the ones
with no CONFIG_IP, beause sockets are not used.

Now we can move to a caller-defined queue size, when I find the time and
willingness.
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>

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>

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

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

This changes the pfilter rule-set, to make it ready for vlan addition.
Moreover, it removes the choice between three rule-sets and uses one
that works for everyone (we'll add the vlan option later on).

Moreover, we prepare for the option of peer-delay PTP and UDP-based PTP.

As a side effect, one Kconfig option is removed.

We know for sure that what goes to the "streamer" protocol
is re-checked, so the streamer can get some "everything else", exactly
like the 7S wr-nic (that sends to the host everything unused).

This is the current rule set:

     - everything tagged is dropped
     - CPU: arp broadcast, not unicast (i.e. requests only).
     - CPU: PTP ethtype. Any mac address.
     - CPU: ICMP unicast, not broadcast.
     - CPU: UDP (uni/multicast), ports  bootpc, ptp-event, ptp-general.
     - Etherbone (class 7): UDP (uni/multicast) and port 0xebd0
     - Other fabric (class 6): anything not Etherbone (inverted class 7 bit)

This counts up to 28 rules. We have 4 more rules available and we may
add options, either at build time or run time to add udp ports or
other ethertypes.  Or not...

2 caveats:

  - frames for the CPU will have class bits 0x41 instead of 0x01, because
  everything not etherbone will receive bit 6. This is not a problem, as
  the CPU is not checking the class bits.

  - the wr-nic gateware, when using this new sw code base, must be
  changed to use classes 6 and 7 like everybody else, not classes 7
  and 5.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

The description of class management was not completely clear or correct,
so it's now rewritten with some references to gateware files.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Actually, the "new" information is very little.  But now the
explanation of how it works fits in 80 columns (I personally need
this) and is laid out a little better.  Also, it describes the new
situation, activated by the next commits.

No code change at this point.
Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

Otherwise, the macro won't build on 64-bit hosts
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>

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>