[U-Boot] [RFC 4/5] SPL:Defines function required to env read for IFC & env_nand

Prabhakar Kushwaha prabhakar at freescale.com
Wed Sep 25 06:17:47 CEST 2013


On 09/23/2013 11:49 PM, Scott Wood wrote:
> On Wed, 2013-09-18 at 16:40 +0530, Prabhakar Kushwaha wrote:
>> Thanks Scott for taking time and reviewing the RFC patch.
>>
>> Please find my reply in-lined.
>>
>> On 09/17/2013 05:23 AM, Scott Wood wrote:
>>> On Mon, 2013-09-16 at 21:35 +0530, Prabhakar Kushwaha wrote:
>>>> fsl_ifs_spl.c reads data from NAND and store at a memory location in raw mode.
>>>> It does not used MTD layer.
>>>> To read env variable from NAND MTD layer read/write required.
>>>>
>>>> Hence, add mtd_block_isbad & nand_read_skip_bad function required during
>>>> env variable read.
>>>>
>>>> Also, avoid nand_info during env read for SPL
>>>>
>>>> Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
>>>> ---
>>>>    common/env_nand.c              |    7 ++++---
>>>>    drivers/mtd/nand/Makefile      |    2 +-
>>>>    drivers/mtd/nand/fsl_ifc_spl.c |   22 ++++++++++++++++++++++
>>>>    3 files changed, 27 insertions(+), 4 deletions(-)
>>>>
>>>> diff --git a/common/env_nand.c b/common/env_nand.c
>>>> index 7530962..7a7107f 100644
>>>> --- a/common/env_nand.c
>>>> +++ b/common/env_nand.c
>>>> @@ -246,11 +246,13 @@ int readenv(size_t offset, u_char *buf)
>>>>    	u_char *char_ptr;
>>>>    
>>>>    	blocksize = nand_info[0].erasesize;
>>>> +#ifndef CONFIG_SPL_BUILD
>>>>    	if (!blocksize)
>>>>    		return 1;
>>>> -
>>>>    	len = min(blocksize, CONFIG_ENV_SIZE);
>>>> -
>>>> +#else
>>>> +	len = CONFIG_ENV_SIZE;
>>>> +#endif
>>> Use positive logic (ifdef/else, not ifndef/else).
>> I will fix it.
>>
>>> Are you sure that CONFIG_ENV_SIZE will always be appropriate?  Shouldn't
>>> you use CONFIG_SYS_NAND_BLOCK_SIZE in place of nand_info[0].erasesize?
>> I can use CONFIG_SYS_NAND_BLOCK_SIZE . but i can not use in SPL as its
>> is defined as 128K.
> You can't redefine the block size to suit your needs.  It's a property
> of the hardware.  What difference does it make, as long as
> CONFIG_ENV_SIZE is smaller?
Earlier code was like
  len = min(blocksize, CONFIG_ENV_SIZE) as SPL never defines nand_info. 
So the block size is 0.
causing len = 0.

I am avoiding it by setting  = CONFIG_ENV_SIZE. No redefinition of block 
size.

Regards,
Prabhakar






More information about the U-Boot mailing list