[U-Boot] [PATCH v4 4/9] Stop using builtin_run_command()
Michael Walle
michael at walle.cc
Sat Jan 21 17:07:30 CET 2012
Hi Simon,
Am Samstag 14 Januar 2012, 07:45:52 schrieb Simon Glass:
> diff --git a/common/cmd_source.c b/common/cmd_source.c
> index 481241c..32fff5c 100644
> --- a/common/cmd_source.c
> +++ b/common/cmd_source.c
> @@ -179,7 +179,7 @@ source (ulong addr, const char *fit_uname)
> if (*line) {
> debug ("** exec: \"%s\"\n",
> line);
> - if (builtin_run_command(line, 0) < 0)
{
> + if (run_command(line, 0) < 0) {
> rcode = 1;
> break;
> }
> @@ -189,7 +189,7 @@ source (ulong addr, const char *fit_uname)
> ++next;
> }
> if (rcode == 0 && *line)
> - rcode = (builtin_run_command(line, 0) >= 0);
> + rcode = (run_command(line, 0) >= 0);
> }
> #endif
> free (cmd);
Please have a look at this file. There is already a conditional on
CONFIG_SYS_HUSH_PARSER, which can be removed, since it is now handled in
run_command(). Do not forget to remove the hush.h include, too.
Could you move this script handling into an own function into common/main.c?
This way others can easily execute whole scripts without duplicating this
code. I'm planning to submit support for a board which needs exactly that, so
there will be at least one board which make use of this.
Mike proposed source_commands() as a function name. My original patch named it
run_script(). Dunno whats a better name.
Feel free to reuse/incorporate my patch at
http://patchwork.ozlabs.org/patch/137122/
into your patchset ;)
--
michael
More information about the U-Boot
mailing list