[U-Boot] [PATCH] dra7x: fastboot: Increase recovery partition size

[Bajjuri, Praneeth]

Hi Tom,

if there is nothing waiting to be addressed, Can you pull this patch.

-- Praneeth

On 11/10/2017 1:22 PM, Sam Protsenko wrote:
> On 9 November 2017 at 01:12, Praneeth Bajjuri <praneeth at ti.com> wrote:
>> As per current android recommendation
>> https://source.android.com/devices/architecture/kernel/modular-kernels
>>
>> 1. Android recovery mode should contain both SOC and ODM
>> kernel modules in the recovery partition.
>>
>> 2. If a kernel module is required both in recovery and normal boot
>> mode,  the module has to be located in recovery and vendor
>> partition seperately.
>>
>> 3. Kernel modules used in recovery mode should be independent
>> of vendor and odm partition
>>
>> 4. Recovery image should contain atleast
>> storage, display, keypad, battery and pmic modules.
>>
>> Due to these requirements, recovery image size has increased
>> to >10MB.
>>
>> This patch is to increase recovery partition size for TI devices
>> so that we dont see such flashing error
>>
>> log:
>> sending 'recovery' (12560 KB)...
>> OKAY [  0.436s]
>> writing 'recovery'...
>> FAILED (remote: too large for partition)
>> finished. total time: 0.458s
>>
>> Signed-off-by: Praneeth Bajjuri <praneeth at ti.com>
>> ---
>>   include/configs/am57xx_evm.h | 2 +-
>>   include/configs/dra7xx_evm.h | 2 +-
>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
>> index ebb0474..dc05bea 100644
>> --- a/include/configs/am57xx_evm.h
>> +++ b/include/configs/am57xx_evm.h
>> @@ -56,7 +56,7 @@
>>          "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
>>          "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
>>          "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
>> -       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
>> +       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
>>          "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
>>          "name=system,size=768M,uuid=${uuid_gpt_system};" \
>>          "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
>> diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
>> index 34a4555..f84c1f0 100644
>> --- a/include/configs/dra7xx_evm.h
>> +++ b/include/configs/dra7xx_evm.h
>> @@ -63,7 +63,7 @@
>>          "name=reserved,size=256K,uuid=${uuid_gpt_reserved};" \
>>          "name=efs,size=16M,uuid=${uuid_gpt_efs};" \
>>          "name=crypto,size=16K,uuid=${uuid_gpt_crypto};" \
>> -       "name=recovery,size=10M,uuid=${uuid_gpt_recovery};" \
>> +       "name=recovery,size=40M,uuid=${uuid_gpt_recovery};" \
>>          "name=boot,size=10M,uuid=${uuid_gpt_boot};" \
>>          "name=system,size=768M,uuid=${uuid_gpt_system};" \
>>          "name=vendor,size=256M,uuid=${uuid_gpt_vendor};" \
>> --
>> 1.9.1
>>
> Reviewed-by: Sam Protsenko <semen.protsenko at linaro.org>