summaryrefslogtreecommitdiff
path: root/board/digsy_mtc
Commit message (Collapse)AuthorAgeLines
* Make sure that argv[] argument pointers are not modified.Wolfgang Denk2010-07-04-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The hush shell dynamically allocates (and re-allocates) memory for the argument strings in the "char *argv[]" argument vector passed to commands. Any code that modifies these pointers will cause serious corruption of the malloc data structures and crash U-Boot, so make sure the compiler can check that no such modifications are being done by changing the code into "char * const argv[]". This modification is the result of debugging a strange crash caused after adding a new command, which used the following argument processing code which has been working perfectly fine in all Unix systems since version 6 - but not so in U-Boot: int main (int argc, char **argv) { while (--argc > 0 && **++argv == '-') { /* ====> */ while (*++*argv) { switch (**argv) { case 'd': debug++; break; ... default: usage (); } } } ... } The line marked "====>" will corrupt the malloc data structures and usually cause U-Boot to crash when the next command gets executed by the shell. With the modification, the compiler will prevent this with an error: increment of read-only location '*argv' N.B.: The code above can be trivially rewritten like this: while (--argc > 0 && **++argv == '-') { char *arg = *argv; while (*++arg) { switch (*arg) { ... Signed-off-by: Wolfgang Denk <wd@denx.de> Acked-by: Mike Frysinger <vapier@gentoo.org>
* ppc: Remove board-specific command table relocation fixupsPeter Tyser2009-10-03-32/+0
| | | | Signed-off-by: Peter Tyser <ptyser@xes-inc.com>
* digsy_mtc: Add delay in SPI transfers to the companion controller.Detlev Zundel2009-08-09-20/+27
| | | | | | | While at it, remove initialization of variables which will be set before usage in all cases. Signed-off-by: Detlev Zundel <dzu@denx.de>
* digsy_mtc: minor fixes for mtc command helpAnatolij Gustschin2009-08-09-5/+6
| | | | | | | | | | Add mtc state subcommand description to the help of mtc command. Remove some newlines in description of commands for proper help formating. Signed-off-by: Anatolij Gustschin <agust@denx.de>
* digsy_mtc: Add mtc state command.Grzegorz Bernacki2009-08-09-0/+31
| | | | Signed-off-by: Grzegorz Bernacki <gjb@semihalf.com>
* Coding style cleanup; update CHANGELOGWolfgang Denk2009-07-07-15/+15
| | | | Signed-off-by: Wolfgang Denk <wd@denx.de>
* digsy MTC: Add 'mtc' command.Grzegorz Bernacki2009-06-14-1/+413
| | | | | | | | | | New command allows to: o check FW version o set LED status o set digital output status o get digital input status Signed-off-by: Grzegorz Bernacki <gjb@semihalf.com>
* digsy MTC: Add SPI support.Grzegorz Bernacki2009-06-14-0/+10
| | | | Signed-off-by: Grzegorz Bernacki <gjb@semihalf.com>
* Update CHANGELOG, coding style cleanup.Wolfgang Denk2009-04-05-2/+0
|
* Add support for the digsy MTC board.Grzegorz Bernacki2009-03-20-0/+423
This is the InterControl custom device based on the MPC5200B chip. Signed-off-by: Grzegorz Bernacki <gjb@semihalf.com>