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

This also changes ptp-noposix as needed

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>

We include newlib headers (and link part of newlib), so any
locally-defined function must have the same prototype as standard
ones. We'll soon nee a "puts" implementation for pp_printf, so this
adds puts as an alias of uart_write_string (at no size cost) and makes
uart_write_string itself return integer (this costs a few bytes).

While I'm at it, change uart_write_byte to be slightly faster.

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>

Also add the missing prototypes in eeprom.h, to fix warnings in shell/

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

I also turned to gui to void, as nobody checks the return value and
there's no error code to return in any case.

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

When I add prototypes to headers, I ensure the source that
defines the function sees the header, to detect inconsistencies
(here, I add <wrc.h> to the includes).

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

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

This adds all missing prototypes and removes unused variables.
It includes a generic <wrc.h> (which btw includes the printf
prototype, not in uart.h any more) and other needed bits.

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

There was a wrong "static" in a public header that triggered
my interest here. This fixes the warning and makes all functions
that are unused elsewhere really static. It also saves 50 bytes
in the final binary.

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

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

This is massive: 4k lines changed (but only 840 if you ignore
space-only changes).  In this case "git blame -w" won't always find
the right patch, and it may fall on this patch -- because those
800 lines changed in content too.

This has been done with

   find . -name '*.[ch]' | xargs -n 1 ./scripts/Lindent

Statistics: all changes and ignoring blank space:

   morgana% git diff --stat HEAD~1 | tail -1
    77 files changed, 3774 insertions(+), 3709 deletions(-)
   morgana% git diff -w --stat HEAD~1 | tail -1
    61 files changed, 894 insertions(+), 829 deletions(-)

However, after this step I hand-fixed some very ugly long expressions
(leaving them long: no content change at all is there).

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

If you find this patch with "git blame" please use "git blame -w"
to have all white-space ignored while associating lines to commits.

This commit has no practical effect but cleanup. I made it
with sed like this:

  git grep -l '[ \t]$' | xargs sed -i 's/[ \t]*$//'

However, I had to manually restore doc/wrpc_mon.png after the fact.
Similarly, I restored the include/hw/*regs.h files, as they
are (most likely) auto-generated.

Tidy up printing at startup

Read the MAC address either from EEPROM or sensor ID (preferring EEPROM).

rewrote pps_gen driver (to match the libswitchhw one), removed remaining references to UTC timescale