[U-Boot] [PATCH v3 1/4] Add run_command_list() to run a list of commands

Wolfgang Denk wd at denx.de
Thu Aug 9 22:06:15 CEST 2012


Dear Simon Glass,

In message <1333179058-19598-1-git-send-email-sjg at chromium.org> you wrote:
> This new function runs a list of commands separated by semicolon or newline.
> We move this out of cmd_source so that it can be used by other code. The
> PXE code also uses the new function.
> 
> Suggested-by: Michael Walle <michael at walle.cc>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> Changes in v3:
> - Added a few more comments on the need for malloc()
> - Change PXE's printf() to a debug()
> - Fix bug in built-in run_command()
> - Fix commit message to mention ; and \n
> - Move run_command_list() comment to header file
> 
>  common/cmd_pxe.c    |   20 ++----------
>  common/cmd_source.c |   49 +----------------------------
>  common/main.c       |   85 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  include/common.h    |   13 ++++++++
>  4 files changed, 102 insertions(+), 65 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Swap read error.  You lose your mind.


More information about the U-Boot mailing list