2009/5/12 Jean-Christophe PLAGNIOL-VILLARD <plagn...@jcrosoft.com>: > On 21:03 Mon 11 May , Magnus Lilja wrote: >> 2009/5/10 Jean-Christophe PLAGNIOL-VILLARD <plagn...@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. > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
