[U-Boot] [PATCH 1/2] arm: add a weak function reset_cpu

[Stefano Babic <sbabic@denx.de>]

Hi Peng,

On 04/01/2016 05:19, Peng Fan wrote:
> This commit 4bdcbe60a142b08eefccb0e326a37ba81d3389e8 removes reset_cpu which
> breaks SPL build when DEBUG macro defined.
> 
> Introudce a weak function reset_cpu to avoid build failure when
> building SPL for i.MX with DEBUG enabled.
> "
> arch/arm/lib/built-in.o: In function `do_reset':
> ~/uboot/arch/arm/lib/reset.c:45: undefined reference to `reset_cpu'
> scripts/Makefile.spl:244: recipe for target 'spl/u-boot-spl' failed
> "

I do not like to add a weak function just to fix the build, if this is
not strictly necessary. IMHO the build fails because we moved
reset_cpu() inside the watchdog driver to make it available for all
CPUs. Is imx_watchdog not built together with SPL ?

> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Albert Aribaud <albert.u.boot@aribaud.net>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/lib/reset.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/lib/reset.c b/arch/arm/lib/reset.c
> index 9a95f08..4dd0131 100644
> --- a/arch/arm/lib/reset.c
> +++ b/arch/arm/lib/reset.c
> @@ -27,6 +27,10 @@ __weak void reset_misc(void)
>  {
>  }
>  
> +__weak void reset_cpu(ulong addr)
> +{
> +}
> +
>  int do_reset(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>  {
>  	puts ("resetting ...\n");
> 

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================