It's a preparation for flash support, as there we cannot update
bytes/entries count without erasing entire flash sector.

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

- Turn the copyright header to the local convention
- fix a little white space
- fix flash_sdb_check (must return error -- but it's unused)

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

Theodor-Adrian Stana authored 11 years ago and Grzegorz Daniluk committed 10 years ago

The optimization is the removal of the retval variable in
bbspi_transfer() and using the val parameter as a "shift register"
that is returned at the end of the function.

Signed-off-by: Theodor Stana <t.stana@cern.ch>

Theodor-Adrian Stana authored 11 years ago and Grzegorz Daniluk committed 10 years ago

Signed-off-by: Theodor Stana <t.stana@cern.ch>

Theodor-Adrian Stana authored 11 years ago and Grzegorz Daniluk committed 10 years ago

For this to work, I also had to add (in flash.c) setting the
entry-point to the right value (the one where the SDB magic is
detected).

The rest of the commit files are just toying around for testing
purposes.

Signed-off-by: Theodor Stana <t.stana@cern.ch>

Theodor-Adrian Stana authored 11 years ago and Grzegorz Daniluk committed 10 years ago

Signed-off-by: Theodor Stana <t.stana@cern.ch>

Theodor-Adrian Stana authored 11 years ago and Grzegorz Daniluk committed 10 years ago

wrc_main still in test-mode, currently doing a read of the first two
sectors (in the purpose of testing flash-write in tools -- see next
commit)

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>