[U-Boot] [PATCH 4/8 v2] powerpc/85xx: add TPL_BOOT support
Wolfgang Denk
wd at denx.de
Sat Jan 22 23:12:25 CET 2011
Dear Haiying.Wang at freescale.com,
In message <1291218463-4211-1-git-send-email-Haiying.Wang at freescale.com> you wrote:
> From: Haiying Wang <Haiying.Wang at freescale.com>
>
> Signed-off-by: Haiying Wang <Haiying.Wang at freescale.com>
> ---
> Splitted from TPL patch to only address 85xx changes
> arch/powerpc/cpu/mpc85xx/cpu_init_nand.c | 34 ++++++++++-
> arch/powerpc/cpu/mpc85xx/start.S | 12 ++--
> arch/powerpc/cpu/mpc85xx/u-boot-tpl.lds | 99 ++++++++++++++++++++++++++++++
> 3 files changed, 137 insertions(+), 8 deletions(-)
> create mode 100644 arch/powerpc/cpu/mpc85xx/u-boot-tpl.lds
...
> - * Copyright 2009 Freescale Semiconductor, Inc.
> + * Copyright 2010 Freescale Semiconductor, Inc.
You should not undo a previous copyright. I guess you mean:
Copyright 2009-2010 Freescale Semiconductor, Inc.
?
> +DECLARE_GLOBAL_DATA_PTR;
Please move this up to top of file.
> +unsigned long get_tbclk(void)
> +{
> +#ifdef CONFIG_FSL_CORENET
> + return (gd->bus_clk + 8) / 16;
> +#else
> + return (gd->bus_clk + 4UL)/8UL;
> +#endif
This looks inconsistent. Either this should be "... +8UL) / 16UL" or
"... + 4) / 8;"
> -#ifndef CONFIG_NAND_SPL
> +#if !defined(CONFIG_NAND_SPL) && !defined(CONFIG_TPL_BOOT)
> GOT_ENTRY(_start)
> GOT_ENTRY(_start_of_vectors)
> GOT_ENTRY(_end_of_vectors)
> GOT_ENTRY(transfer_to_handler)
> -#endif
> +#endif /* !CONFIG_TPL_BOOT || !CONFIG_NAND_SPL*/
Should that be
#endif /* !CONFIG_TPL_BOOT && !CONFIG_NAND_SPL*/
instead?
And can we please make this consistent, i. e. as you do in the
following:
> +#endif /* !CONFIG_NAND_SPL && !CONFIG_TPL_BOOT */
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
The price of curiosity is a terminal experience.
- Terry Pratchett, _The Dark Side of the Sun_
More information about the U-Boot
mailing list