u-boot.git/common/main.c, branch v2012.07

net: cosmetic: net.c checkpatch compliance

2012-05-15T22:16:19+00:00

Signed-off-by: Joe Hershberger 
Acked-by: Simon Glass

Fix the behaviour of the 'run' command

2012-04-23T20:07:00+00:00

If one command fails, 'run' command should terminate and not execute
any remaining variables.

Signed-off-by: Timo Ketola 
Tested-by: Wolfgang Denk 
Tested-by: Simon Glass 
Acked-by: Simon Glass

Convert cmd_usage() calls in common to use a return value

2012-03-06T20:09:46+00:00

Change all files in common/ to use CMD_RET_USAGE instead of calling
cmd_usage() directly. I'm not completely sure about this patch since
the code since impact is small (100 byte or so on ARM) and it might
need splitting into smaller patches. But for now here it is.

Signed-off-by: Simon Glass

Add cmd_process() to process commands in one place

2012-03-06T20:09:41+00:00

We currently have the same code in hush.c and main.c. This brings the
code into one place.

As an added feature, if the command function returns CMD_RET_USAGE then
cmd_process() will print a usage message for the command before
returning the standard failure code of 1.

ARM code size increases about 32 bytes with this clean-up.

Signed-off-by: Simon Glass

Create a single cmd_call() function to handle command execution

2012-03-06T20:09:34+00:00

We should aim for a single point of entry to the commands, whichever
parser is used.

Signed-off-by: Simon Glass

Don't include standard parser if hush is used

2012-03-06T20:09:30+00:00

This saves about 1KB of code space on ARM with CONFIG_SYS_HUSH_PARSER
defined.

Signed-off-by: Simon Glass

Stop using builtin_run_command()

2012-03-06T20:09:26+00:00

Boards can select either the 'built-in' parser or the hush parser. We
should not call builtin_run_command() if we are using the hush parser.
We use run_command() instead, since it knows how to call the correct
parser.

Signed-off-by: Simon Glass 
Acked-by: Mike Frysinger

Rename run_command2() to run_command()

2012-03-06T20:09:22+00:00

This is a more sensible name, so rename it.

Signed-off-by: Simon Glass 
Acked-by: Mike Frysinger

Rename run_command() to builtin_run_command()

2012-03-06T20:09:18+00:00

The current run_command() is only one of the parsing options - the other
is hush. We should not call run_command() when the hush parser is being
used. So we rename this function to better explain its purpose.

Signed-off-by: Simon Glass 
Acked-by: Mike Frysinger

Remove CMD_PXE's static on run_command()

2012-03-06T20:09:14+00:00

It really isn't clear why this is here and there is no comment, so
drop it.

Signed-off-by: Simon Glass 
Acked-by: Mike Frysinger