[U-Boot] [PATCH v2 2/3] ARM: at91: sama5d3xek: save enviroment as a FAT file in MMC card

[Josh Wu <josh.wu@atmel.com>]

On 1/19/2015 4:18 PM, Bo Shen wrote:
> Hi Josh,
>
> On 01/19/2015 03:06 PM, Josh Wu wrote:
>> This patch will save U-Boot environment as a file: uboot.env, in FAT 
>> partition
>> instead of in raw sector of MMC card.
>>
>> This make us easier to manage the environment file.
>>
>> Signed-off-by: Josh Wu <josh.wu@atmel.com>
>
> Ackey-by: Bo Shen <voice.shen@atmel.com>
>
>> ---
>>
>>   include/configs/sama5d3xek.h | 10 ++++++----
>>   1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/include/configs/sama5d3xek.h b/include/configs/sama5d3xek.h
>> index f2849d7..c29f25b 100644
>> --- a/include/configs/sama5d3xek.h
>> +++ b/include/configs/sama5d3xek.h
>> @@ -215,13 +215,15 @@
>>                   "bootm 0x22000000 - 0x21000000"
>>   #elif CONFIG_SYS_USE_MMC
>>   /* bootstrap + u-boot + env in sd card */
>> -#define CONFIG_ENV_IS_IN_MMC
>> -#define CONFIG_ENV_OFFSET    0x2000
>> -#define CONFIG_ENV_SIZE        0x1000
>> +#define CONFIG_ENV_IS_IN_FAT
>> +#define CONFIG_FAT_WRITE
>
> Nitpick:
> Can you move in the following condition include?
> --->8---
> #if defined(CONFIG_CMD_USB) || defined(CONFIG_CMD_MMC)
> #define CONFIG_CMD_FAT
> #endif
> ---8<---
sure, it's doable. I'll fix it in next version.

Best Regards,
Josh Wu

>
>> +#define FAT_ENV_INTERFACE    "mmc"
>> +#define FAT_ENV_FILE        "uboot.env"
>> +#define FAT_ENV_DEVICE_AND_PART    "0"
>> +#define CONFIG_ENV_SIZE        0x4000
>>   #define CONFIG_BOOTCOMMAND    "fatload mmc 0:1 0x21000000 dtb; " \
>>                   "fatload mmc 0:1 0x22000000 uImage; " \
>>                   "bootm 0x22000000 - 0x21000000"
>> -#define CONFIG_SYS_MMC_ENV_DEV    0
>>   #else
>>   #define CONFIG_ENV_IS_NOWHERE
>>   #endif
>>
>
> Best Regards,
> Bo Shen