[U-Boot] [PATCH] Allow the "reset" command to be omitted with CONFIG_CMD_RESET

Mike Frysinger vapier at gentoo.org
Thu Oct 20 21:53:50 CEST 2011


On Thursday 20 October 2011 15:05:50 Kyle Moffett wrote:
> This new #define is set by default in config_cmd_defaults.h, and
> config_cmd_all.h, but this allows boards to conditionally omit the
> "reset" command if necessary.

ignoring the related issues as i'm sure Wolfgang will chime in ...

> --- a/common/cmd_boot.c
> +++ b/common/cmd_boot.c
> @@ -71,8 +71,10 @@ U_BOOT_CMD(
> 
>  #endif
> 
> +#ifdef CONFIG_CMD_RESET
>  U_BOOT_CMD(
>  	reset, 1, 0,	do_reset,
>  	"Perform RESET of the CPU",
>  	""
>  );
> +#endif

would be a good time to split this into a dedicated common/cmd_reset.c

> --- a/include/config_cmd_all.h
> +++ b/include/config_cmd_all.h
>
> --- a/include/config_cmd_defaults.h
> +++ b/include/config_cmd_defaults.h

updating these files is not sufficient and will break boards.  drop the hunks to 
these files and update include/config_defaults.h instead.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111020/33e75fe2/attachment.pgp 


More information about the U-Boot mailing list