[U-Boot] [PATCH v2 05/10] spl, nand: add 4bit HW ecc oob first nand_read_page function
Heiko Schocher
hs at denx.de
Thu Oct 6 07:41:19 CEST 2011
Hello Scott,
Scott Wood wrote:
> On 10/05/2011 09:28 AM, Heiko Schocher wrote:
>> similiar to commit dc7cd8e59ba077f3b4c1a4557c9cd86a31b9ab1f, only
>> adapted for the new spl framework.
>>
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>> Cc: Scott Wood <scottwood at freescale.com>
>> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
>> Cc: Sandeep Paulraj <s-paulraj at ti.com>
>> ---
>> drivers/mtd/nand/nand_spl_simple.c | 43 ++++++++++++++++++++++++++++++++++++
>> 1 files changed, 43 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/mtd/nand/nand_spl_simple.c b/drivers/mtd/nand/nand_spl_simple.c
>> index 71491d4..7ab332f 100644
>> --- a/drivers/mtd/nand/nand_spl_simple.c
>> +++ b/drivers/mtd/nand/nand_spl_simple.c
[...]
>> @@ -186,6 +227,7 @@ static int nand_read_page(int block, int page, void *dst)
>>
>> return 0;
>> }
>> +#endif
>>
>> int nand_spl_load_image(uint32_t offs, unsigned int size, void *dst)
>> {
>
> ACK this part
great!
>> @@ -231,6 +273,7 @@ void nand_init(void)
>> nand_chip.IO_ADDR_R = nand_chip.IO_ADDR_W =
>> (void __iomem *)CONFIG_SYS_NAND_BASE;
>> nand_chip.options = 0;
>> + nand_chip.dev_ready = NULL;
>> board_nand_init(&nand_chip);
>>
>> if (nand_chip.select_chip)
>
> This looks unrelated, and if it makes a difference suggests that the BSS
> isn't being cleared.
BSS is cleared at this point, so you are right, above two lines are
not necessary, remove them.
Thanks!
bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list