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

Magnus Lilja lilja.magnus at gmail.com
Tue May 12 06:03:29 CEST 2009


2009/5/12 Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>:
> On 21:03 Mon 11 May     , Magnus Lilja wrote:
>> 2009/5/10 Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>:
>> >> >> +
>> >> >> +     }
>> >> >> +
>> >> >> +     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
>>
>> It's not really doable. In order for this to be worthwhile the
>> nand_flash_ids array from drivers/mtd/nand/nand_ids.c would have to
>> fit into the available space (2048 byte in total) and it didn't even
>> though I removed the text-strings.
> as you mention here there is only 2K
> so the start.S was reduced to allow more code
> so CONFIG_xxx_IPL/SPL do not describe this
> so please chance it to a better one

As I said in another email:
"The main purpose of CONFIG_ONENAND_IPL and NAND_SPL in start.S is to
change the behaviour a bit, not so much to save some space."
Given that those CONFIG_s are used in other places as well to indicate
the type of boot I don't think it's good to create a new CONFIG_ that
one has to specify in addition to one of the above.

Regards, Magnus
>
> Best Regards,
> J.
>


More information about the U-Boot mailing list