[U-Boot] [PATCH v2 1/3] MX31: Add NAND SPL for i.MX31.

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sun May 10 22:34:54 CEST 2009


On 21:31 Sun 10 May     , Magnus Lilja wrote:
> 2009/5/9 Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>:
> > On 21:56 Sun 03 May     , Magnus Lilja wrote:
> >> @@ -32,7 +35,7 @@
> >>  #include <version.h>
> >>  .globl _start
> >>  _start: b    reset
> >> -#ifdef CONFIG_ONENAND_IPL
> >> +#if defined(CONFIG_ONENAND_IPL) || defined(CONFIG_NAND_SPL)
> > create a better CONFIG_
> > because we could need for other boot mode
> 
> A better CONFIG_ than what? CONFIG_NAND_SPL is already used elsewhere
> in U-boot when booting from NAND.
IIRC in this case the purpose is to reduce the size of the start.S to allow more
code in the ipl & spl
> 
> >>       ldr     pc, _hang
> >>       ldr     pc, _hang
> >>       ldr     pc, _hang
> >> @@ -156,9 +159,9 @@ relocate:                         /* relocate U-Boot to RAM           */
> >>       adr     r0, _start              /* r0 <- current position of code   */
> >>       ldr     r1, _TEXT_BASE          /* test if we run from flash or RAM */
> >>       cmp     r0, r1                  /* don't reloc during debug         */
> >> -#ifndef CONFIG_ONENAND_IPL
> >> +#if !defined(CONFIG_ONENAND_IPL) && !defined(CONFIG_NAND_SPL)
> >>       beq     stack_setup
> >> -#endif       /* CONFIG_ONENAND_IPL */
> >> +#endif       /* !CONFIG_ONENAND_IPL && !CONFIG_NAND_SPL*/
> >>
> >>  #endif       /* CONFIG_ONENAND_IPL */
> >> +
> >> +     }
> >> +
> >> +     page_count = CFG_NAND_CHIP_SIZE / CFG_NAND_PAGE_SIZE;
> > please use CONFIG_SYS_ or CONFIG_
> 
> Will do.
> 
> > and why not detect it?
> 
> Might be possible, I'll look into it to see if it's doable.
tks

Best Regards,
J.


More information about the U-Boot mailing list