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>

eeprom_present() returned a value nobody used. Rename to eeprom_init()
instead. Also, save the two parameters so that they are not needed
in other functions (see later commits).

Also, turn int8_t to int as function arguments (int is better for the
CPU).

This adds 20 bytes to the legacy case, and removes 4 to the sdb case.
The legacy size is reduced by later commits.

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

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

This is the simpler thing: have vendor ID in the data structure.

We have 100 bytes more of data, but we save 50 bytes of code.  Total:
50 more but with simpler code (not bound to cern and gsi, thus
reusable).

Please maintainer choose and squash.

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

[Note: this patch can be preserved or squashed into commit
  replace sdb.c with new library-based code
at the maintainer's best choice]

My initial code assumed the vendor is always VID_CERN, but this is
not true: some cells we use are VID_GSI.  This takes care of the thing.

To save a little space, instead of adding a 64-bit field to the data
structure describing a device, I add an integer field stating whether
the device is CERN or GSI.

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

This adds almost 800 bytes to all configurations, but removes
200 bytes from the gsi_defconfig.  This is because sdb-eeprom
is already managed with this library, and thus the commit is removing
a duplication.

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

It is not mandatory to actually register the tree, but it's
a good move for consistency.

Later we'll need to scan several devices, for example to look for a
mac address in carrier flash or eeprom. To do that we'll need to have
devices properly registered.

This costs 150 bytes of binary size in gsi_defconfig.

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

Pietro Fezzardi authored 10 years ago and Alessandro Rubini committed 10 years ago

Some declaration was int8_t, some int32_t and some didn't match
the function definition. Now everything returns just int.
Anyway the functions return only 0 or -1, so there are no problems
returning int. Moreover in this way the resulting assembler is
smaller for every target architecture (even if here we always
build for lm32)

This reduces the binary by 30 bytes.

Emilio Marín authored 10 years ago and Benoit Rat committed 10 years ago

By default the scb_ver is "33" but it can be modified at runtime in order
to load the config for SCB v3.4 (i.e, load-lm32 rt_cpu.elf scb_ver=34)

Signed-off-by: Benoit Rat <benoit@sevensols.com>

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

This cleans a little the calibrator code: we'll write to eeprom only
when the measured value is different from the previous one.

With this commit, code is also slightly smaller (28 bytes).

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

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

This also removes an unsed function, that I found because it used
timer_delay().

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

We don't want "timer_delay(howlong)" because people won't know the
unit while looking at the header.

Also, this removes an unused prototype and adds void when void is due
in timer code.

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 fixes the makefile to build either wrc.bin or rt_cpu.bin, using
the obj-y convention. At this point Kconfig can't select rt_cpu.bin,
and nothing changes in the ouput binary, in all configurations.

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

The commit is part of the effort in unifying softpll with
wr-switch-sw, and later remove the duplicated code there.

The files added by these commit are going to be built when configuring
wrpc-sw to build wr-switch rt_cpu (which is, basically, the softpll
code alone, with mini-rpc with the host and the basic glue code).  The
files are copied with the original name with two exceptions:

   wr-switch-sw::rt/main.c            becomes wrpc-sw::wrs_main.c
   wr-switch-sw::rt/arch/lm32/ram.ld  becomes wrpc-sw::arch/lm32/ram-wrs.ld

The files are copied from commit FIXME of wr-switch-sw.

This commit has no technical effect, as the files are not built for wrpc-sw.

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

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

Miguel Jimenez Lopez authored 11 years ago and Grzegorz Daniluk committed 11 years ago

	- Added the NIC_PFILTER configuration option in Kconfig to decide which filter rules will be used.
	- Updated the dev/ep_pfilter.c file by using the CONFIG_NIC_PFILTER macro. This is done because packet filter does
	not work correctly if we write all rules in the CONFIG_ETHERBONE section.
	- We do not use the DROP instruction because all other packets go to NIC core.

Notes:

	- The filter rules to the wr-nic do not have the DROP instruction because all traffic
	does not go to the LM32/Etherbone is re-direct to the NIC by default.

	- The NIC_PFILTER configuration option depends on ETHERBONE one.

	- Magic number of Etherbone packets is not checked due to the number of rules. (The packet filter is not able to classify correctly)

Miguel Jimenez Lopez authored 11 years ago and Grzegorz Daniluk committed 11 years ago

- Added CONFIG_WRNIC in Kconfig to decide which filter rules will be use.
- Updated ep_pfilter using CONFIG_WRNIC.
- We do not use drop because all other packet goes to NIC core.

Note: filter rules to wr-nic do not have DROP instruction because all traffic
does not go to LM32/Etherbone is re-direct to NIC by default.

It was too many instructions and for small frames pfilter was not able to
execute them before the end of frame.

Previous one was not working when:
* tR and tF were both inside (0; 4000) and tR < tF
* tR and tF were both inside (4000; 8000) and tF < tR

in those cases it was calculating ttrans around falling edge instead of rising
edge. This commit fixes it.

Wesley W. Terpstra authored 11 years ago and Grzegorz Daniluk committed 11 years ago

t24p calibration finds the rising&falling edges in
dev/rxts_calibrator.c:rxts_calibration_update.  It then computes the
"transition" as (falling+rising)/2.  That is, ttrans points 25% past the
dangerous transition.  Now that value arrives in one of the three copies of
ptpd_netif_linearize_rx_timestamp (depending on build).  In this method
ttrans has +-1/4 period added to it to compute trip_lo and trip_hi.  The
intent as described by the comment (and common sense) is to avoid the rising
edge when the phase is within +-1/4 period.  Unfortunately, this code
assumes that ttrans IS the rising edge, when in fact it is the rising
edge+25%. Thus, the code ACTUALLY tests phase within 0-50%.

Greg's commit to fix the t24p calibration broke that calibration
for devices using sdbfs. The EEPROM write method clobbered the
phase transition value.

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

This is a bigger change than expected, as the width was defined
in two places. So I moved the definition in <wrc.h>, whence everybody
picks it.

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

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