[U-Boot] [PATCH 2/4] common: make abortboot available for pxecfg use

Wolfgang Denk wd at denx.de
Mon Jun 6 21:23:06 CEST 2011


Dear "Jason Hobbs",

In message <1307386599-4256-3-git-send-email-jason.hobbs at calxeda.com> you wrote:
> From: Rob Herring <rob.herring at calxeda.com>
> 
> Signed-off-by: Jason Hobbs <jason.hobbs at calxeda.com>
> ---
>  common/main.c    |    7 +++++++
>  include/common.h |    1 +
>  2 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/common/main.c b/common/main.c
> index 7da6604..4185bfd 100644
> --- a/common/main.c
> +++ b/common/main.c
> @@ -263,6 +263,13 @@ static __inline__ int abortboot(int bootdelay)
>  # endif	/* CONFIG_AUTOBOOT_KEYED */
>  #endif	/* CONFIG_BOOTDELAY >= 0  */
>  
> +int _abortboot(int bootdelay)
> +{
> +#if defined(CONFIG_BOOTDELAY) && (CONFIG_BOOTDELAY >= 0)
> +	return abortboot(bootdelay);
> +#endif
> +}
> +
>  /****************************************************************************/
>  
>  void main_loop (void)
> diff --git a/include/common.h b/include/common.h
> index e659630..fd389e7 100644
> --- a/include/common.h
> +++ b/include/common.h
> @@ -234,6 +234,7 @@ int	readline_into_buffer	(const char *const prompt, char * buffer);
>  int	parse_line (char *, char *[]);
>  void	init_cmd_timeout(void);
>  void	reset_cmd_timeout(void);
> +int	_abortboot(int bootdelay);

Why do you make this a reserved identifier? (Quote standard: " All
identifiers that begin with an underscore and either an uppercase
letter or another underscore are always reserved for any use.").


Also, if this is needed for the PXE code, it should be part of the PXE
patch series.

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
NOTE: The  Most  Fundamental  Particles  in  This  Product  Are  Held
Together  by  a  "Gluing" Force About Which Little is Currently Known
and Whose Adhesive Power Can Therefore Not Be Permanently Guaranteed.


More information about the U-Boot mailing list