| Message ID | 1510182776-3116-1-git-send-email-praneeth@ti.com |
|---|---|
| State | Accepted |
| Commit | 65d87d6cfc204d6a8fa9e5bbe01a974ef9c3074e |
| Headers | show |
| Series | dra7x: fastboot: Increase recovery partition size | expand |
On 9 November 2017 at 01:12, Praneeth Bajjuri <praneeth@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@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@linaro.org>
On Wed, Nov 08, 2017 at 05:12:56PM -0600, Praneeth Bajjuri 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@ti.com> > Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org> Applied to u-boot/master, thanks! -- Tom
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};" \
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@ti.com> --- include/configs/am57xx_evm.h | 2 +- include/configs/dra7xx_evm.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)